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

Add query to SvelteKit Routing docs #4129

Closed
bluepuma77 opened this issue Feb 25, 2022 · 2 comments · Fixed by #4171
Closed

Add query to SvelteKit Routing docs #4129

bluepuma77 opened this issue Feb 25, 2022 · 2 comments · Fixed by #4171
Labels
documentation Improvements or additions to documentation

Comments

@bluepuma77
Copy link

Describe the problem

I'm always frustrated when I try to rebuild my NodeJS Express project in SvelteKit. Like most developers I use params, query and body to pass data to the server. I can find params and body in the docs, but not query.

Describe the proposed solution

Please add a sentence about query parameters in Routing.

Alternatives considered

No response

Importance

i cannot use SvelteKit without it

Additional Information

No response

@bluepuma77 bluepuma77 changed the title Add query to SvelteKit Endpoints docs Add query to SvelteKit Routing docs Feb 25, 2022
@dansvel
Copy link

dansvel commented Feb 25, 2022

they replace request.origin/path/query with request.url , you may want check it here #3126

yeah, its true, the docs did not mention a single word of query in https://kit.svelte.dev/docs/loading#input-url

@dimfeld
Copy link
Contributor

dimfeld commented Feb 26, 2022

On a URL object the searchParams field exposes the parsed querystring, or search is the fully-expanded querystring. See https://developer.mozilla.org/en-US/docs/Web/API/URL

@benmccann benmccann added the docs label Feb 27, 2022
@Rich-Harris Rich-Harris added documentation Improvements or additions to documentation and removed docs labels Mar 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants