-
Notifications
You must be signed in to change notification settings - Fork 8
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
Optional route segments #101
Comments
Are we happy with just repeating execution of Replace("//", "/") until there's no "//" left, or will that break anything? |
It could potentially break something i guess if someone is actually using |
my browser accepts it. Let's just make it opt-in |
this issue kind of also raises the question about what should happen if a route segment value is null. |
Also, about normalizing urls. We could implement a lot of stuff: |
There ya go! |
@all-contributors please add @carl-berg for bug reports |
I've put up a pull request to add @carl-berg! 🎉 |
Would be nice to be able to insert a route segment optionally. Consider this api
If
version=v1
you would getapi/v1/user
... but if
version=""
, you would getapi//user
so it would be nice if that could be handled somehow by WebAnchorThe text was updated successfully, but these errors were encountered: