-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Fixed discrepancies in page store between server and client #119
Conversation
}); | ||
} | ||
|
||
search.slice(1).split('&').filter(Boolean).forEach(searchParam => { |
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.
there was a teeny tiny bug in here: the query string ?a=b&
would become translated to {a: 'b', '': ''}
as for the rest of the function i refactored it because i think this is more readable and also shorter
Looks like this one has a merge conflict |
eb13ad8
to
f700856
Compare
rebased |
You can probably do that now that a couple of the other PRs have been checked in |
f700856
to
4676fcb
Compare
I moved Would be good to get this merged now; there will quickly be new conflicts in the types. |
Co-authored-by: Simon H <5968653+dummdidumm@users.noreply.github.com>
Host was
null
on the server side using dev server. The query was aURLSearchParams
on the server and a map on the client.(This should be safeguarded by types; I'll add that once my other PRs regarding types are merged)
Fixes #118 (except for
error
beingnull
vsundefined
; that's fixed by #108)