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

fix: query params not appending with the same key #380

Merged
merged 3 commits into from
Jul 12, 2023

Conversation

james-elicx
Copy link
Contributor

This PR does the following:

  • Changes the applySearchParams function to append on unique key-value combos, instead of overriding them.
  • Ensures that internal Next.js search params from matches in the build output config always override so routes function properly.

From what I understand, this is common behaviour and is how it should have been implemented originally.

@changeset-bot
Copy link

changeset-bot bot commented Jul 11, 2023

🦋 Changeset detected

Latest commit: f7acb60

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

This PR includes changesets to release 2 packages
Name Type
@cloudflare/next-on-pages Patch
eslint-plugin-next-on-pages 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

@github-actions
Copy link
Contributor

github-actions bot commented Jul 11, 2023

🧪 Prereleases are available for testing 🧪

@cloudflare/next-on-pages

You can install this latest build in your project with:

npm install --save-dev https://prerelease-registry.devprod.cloudflare.dev/next-on-pages/runs/5524960917/npm-package-next-on-pages-380

@cloudflare/eslint-plugin-next-on-pages

You can install this latest build in your project with:

npm install --save-dev https://prerelease-registry.devprod.cloudflare.dev/next-on-pages/runs/5524960917/npm-package-eslint-plugin-next-on-pages-380

@james-elicx james-elicx force-pushed the james/fix-query-param branch 3 times, most recently from ed046ab to 4d65fc0 Compare July 11, 2023 21:12
Copy link
Member

@dario-piotrowicz dario-piotrowicz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great! thanks a lot! 😄

the only thing is those tests that look a bit awkward 🤔

@dario-piotrowicz dario-piotrowicz merged commit 76a8bb4 into cloudflare:main Jul 12, 2023
@james-elicx james-elicx deleted the james/fix-query-param branch July 12, 2023 22:30
Kebab11noel pushed a commit to Likespace-Kft/next-on-pages that referenced this pull request Aug 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants