-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Force URL trailing slash when loading CMS UI #332
Comments
What about recommending a |
That could help, but it isn't always appropriate, and may actually add confusion for some users. Still feels like we're better off adding the trailing slash and resolving this automatically for all users. We'd probably need to add a config option as well to turn off for odd use cases, but that should be a pretty rare need. |
Related - if you type in the URL with a double |
Does this actually break anything except loading the |
React Router's Redirect could be used to redirect |
@lol-russo my concern on this has been that the CMS has no control over it's environment, so I'd hate to cause regressions by enforcing this. The number one issue behind this symptom is a site generator unexpectedly producing both Thoughts? |
Pretty sure this isn't something the cms should handle, closing. |
If the CMS is provided through a file at
example.com/admin/index.html
, and a user loads it viaexample.com/admin
, the location ends up atexample.com/admin#/
, and relative paths are relative to root instead of/admin/
. This stops the config file from loading, among other things.React Router may have a config to help with this, but we'll probably just need to force a trailing slash before loading the app.
The text was updated successfully, but these errors were encountered: