Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Upgrade React Router to allow the declaration of array of string in Route path #1104

Closed
Skylsmoi opened this issue Nov 2, 2018 · 2 comments
Closed

Upgrade React Router to allow the declaration of array of string in Route path #1104

Skylsmoi opened this issue Nov 2, 2018 · 2 comments
Assignees
@Skylsmoi
Labels
draft Issue requiring a better description / specification, or needing research enhancement frontend issue related to frontend development / ux / ui / apps technical debt Rework caused by choosing an easy solution or obsolete code over time. Transparent to the user.

Comments

@Skylsmoi
Copy link
Collaborator

Skylsmoi commented Nov 2, 2018

see remix-run/react-router#5889

refactor should happen in frontend/src/container/Tracim.jsx files
@Skylsmoi Skylsmoi added enhancement frontend issue related to frontend development / ux / ui / apps technical debt Rework caused by choosing an easy solution or obsolete code over time. Transparent to the user. to sort need first level analyse and release association labels Nov 2, 2018
@Skylsmoi Skylsmoi self-assigned this Nov 2, 2018
@tracim tracim changed the title Updagrade React Router to allow the declaration of array of string in Route path Upgrade React Router to allow the declaration of array of string in Route path Nov 19, 2018
@tracim tracim removed the to sort need first level analyse and release association label Nov 19, 2018
@tracim tracim added this to the TracimV2 - V2.2 milestone Nov 19, 2018
@PhilippeAccorsi PhilippeAccorsi added this to To Do in Tracim V2.2 Nov 20, 2018
@lebouquetin lebouquetin removed this from the TracimV2 - V2.2 milestone Feb 20, 2019
@PhilippeAccorsi PhilippeAccorsi added the draft Issue requiring a better description / specification, or needing research label Apr 24, 2019
@PhilippeAccorsi PhilippeAccorsi removed this from To Do in Tracim V2.2 Apr 24, 2019
@PhilippeAccorsi PhilippeAccorsi added this to To Do in Tracim V2.3 Apr 24, 2019
@lebouquetin lebouquetin added the to sort need first level analyse and release association label May 21, 2019
@lebouquetin lebouquetin removed this from To Do in Tracim V2.3 May 21, 2019
@Skylsmoi
Copy link
Collaborator Author

after 2.4, tracim.jsx will uses react fragments to allow a list of routes.

This issue might then be outdated

@Skylsmoi
Copy link
Collaborator Author

it is, I'm closing this issue

@PhilippeAccorsi PhilippeAccorsi removed the to sort need first level analyse and release association label Sep 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Skylsmoi Skylsmoi

Labels
draft Issue requiring a better description / specification, or needing research enhancement frontend issue related to frontend development / ux / ui / apps technical debt Rework caused by choosing an easy solution or obsolete code over time. Transparent to the user.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@lebouquetin @Skylsmoi @PhilippeAccorsi