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

warn and not exit for route conflicts, overhaul messaging #5471

Closed
mcansh opened this issue Feb 15, 2023 · 1 comment · Fixed by #5459
Closed

warn and not exit for route conflicts, overhaul messaging #5471

mcansh opened this issue Feb 15, 2023 · 1 comment · Fixed by #5459
Assignees
@mcansh
Labels
package:dev
Milestone
v1.14

Comments

@mcansh
Copy link
Collaborator

mcansh commented Feb 15, 2023 •
edited
Loading

PR: https://github.com/remix-run/remix/pull/5459/files
@mcansh mcansh linked a pull request Feb 15, 2023 that will close this issue
fix(remix-dev): normalize /route routeId, update route collision warnings #5459
Merged
2 tasks
@mcansh mcansh self-assigned this Feb 15, 2023
@mcansh mcansh mentioned this issue Feb 17, 2023
Merged
2 tasks
@mcansh mcansh added this to the v1.14 milestone Feb 21, 2023
@mcansh mcansh closed this as completed Feb 21, 2023
@github-actions
Copy link
Contributor

🤖 Hello there,

We just published version v0.0.0-nightly-aecf731-20230222 which involves this issue. If you'd like to take it for a test run please try it out and let us know what you think!

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mcansh mcansh

Labels
package:dev
Projects
Roadmap
Archived in project
Milestone
v1.14
Development

Successfully merging a pull request may close this issue.

fix(remix-dev): normalize /route routeId, update route collision warnings remix-run/remix
1 participant
@mcansh