paoloricciuti / sveltekit-search-params

The easiest way to read and WRITE from query parameters in sveltekit.
https://sveltekit-search-params.netlify.app
MIT License
478 stars 13 forks source link

Fix undefined not removing params (#31) #32

Closed paoramati closed 1 year ago

paoramati commented 1 year ago

Fixes https://github.com/paoloricciuti/sveltekit-search-params/issues/31

stackblitz[bot] commented 1 year ago

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

changeset-bot[bot] commented 1 year ago

🦋 Changeset detected

Latest commit: 6a881393f010ac8fbfd0b885e0767603564b26f8

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package | Name | Type | | ----------------------- | ----- | | sveltekit-search-params | Patch |

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

netlify[bot] commented 1 year ago

Deploy Preview for sveltekit-search-params ready!

Name Link
Latest commit 6a881393f010ac8fbfd0b885e0767603564b26f8
Latest deploy log https://app.netlify.com/sites/sveltekit-search-params/deploys/64c455184e56100008989312
Deploy Preview https://deploy-preview-32--sveltekit-search-params.netlify.app/
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

paoloricciuti commented 1 year ago

Can you please regenerate the lockfile with pnpm version 8? I think netlify is using that now and the deploy would fail without it. If that's a problem I can do it myself later

paoloricciuti commented 1 year ago

Thanks for contributing 🙏🏻