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.

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

Enhancement to support case-insensitive URL parameters #8

Closed
MikeTschudi opened this issue Apr 27, 2017 · 2 comments
Closed

Enhancement to support case-insensitive URL parameters #8

MikeTschudi opened this issue Apr 27, 2017 · 2 comments

Comments

@MikeTschudi
Copy link
Member

ApplicationBase.json and the app have case-sensitive URL parameters. We’ve had user problems because people are used to generally ignoring case in the URL, so case-sensitive query parameters get mistyped.

Can we change map[key] in _urlToObject() to map[key.toLowerCase()] and the urlParams list in ApplicationBase.json to all lowercase?

@driskull
Copy link
Member

👍

@driskull
Copy link
Member

Actually, I don't think we want to do this because these values get mixed into the config. things like basemapUrl need to be the correct case to be easily mixed in.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants