-
-
Notifications
You must be signed in to change notification settings - Fork 10.5k
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.
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
Case sensitive routes #5438
Case sensitive routes #5438
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In #5435, I implied that there were five lines to change for this to work, but upon further review that number is actually eight. This has three of them, but is missing the changes to Route.js
and Switch.js
to support the sensitive
prop. 😄
@pshrmn Updated, sir. Thank you for giving the 8 lines hint. :) |
@timdorr It looks like this got missed in the |
Well, crap, everything is on my laptop at home. I'll take a look at it tonight. |
For this enhancement: #5435