Make query string parameters work when using hashbang routing #551
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
When
hashbang
istrue
, urls like http://localhost/?foo=bar gotredirected to http://localhost/?foo=bar#!?foo=bar.
This duplicated the query string and made the url ugly.
This commit fixed this problem by making urls like
http://localhost/?foo=bar redirect to http://localhost/#!?foo=bar
I made some nightwatch tests to make sure all use cases are covered.
The tests can be found here: https://github.com/S2-/page.js-nightwatch-test
All urls work as expected: