fix: preserve array query parameters in user redirect

[aayushbaluni]

Summary

When redirecting dynamic groups to /.../dynamic, the query string was built with URLSearchParams(context.query as Record<string, string>). Next.js passes string | string[] | undefined per key; array values were not serialized as repeated keys and broke multi-value params.

This change builds URLSearchParams by appending each value (and each array element) explicitly.

Fixes #28687.

Made with Cursor

[github-actions]

Welcome to Cal.diy, @aayushbaluni! Thanks for opening this pull request.

A few things to keep in mind:

• This is Cal.diy, not Cal.com. Cal.diy is a community-driven, fully open-source fork of Cal.com licensed under MIT. Your changes here will be part of Cal.diy — they will not be deployed to the Cal.com production app.
• Please review our Contributing Guidelines if you haven't already.
• Make sure your PR title follows the Conventional Commits format.

A maintainer will review your PR soon. Thanks for contributing!

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 1a5e334d-08c0-4d2b-bf59-4613febfad9e

📥 Commits

Reviewing files that changed from the base of the PR and between d278d6c and 82a0734.

📒 Files selected for processing (1)

• apps/web/server/lib/[user]/getServerSideProps.ts

---

📝 Walkthrough

Walkthrough

The change modifies how query string parameters are preserved during redirects for dynamic groups. Instead of directly converting context.query into URLSearchParams, the code now iterates through query entries individually, appending array values separately and filtering out undefined values. This alters the serialization behavior for multi-valued query parameters and handles undefined values explicitly.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title 'fix: preserve array query parameters in user redirect' directly and clearly describes the main change, which is fixing the handling of array query parameters in redirect logic.
Description check	✅ Passed	The description explains the problem (incorrect serialization of array query parameters due to casting context.query), the solution (explicitly appending each value), and references the fixed issue #28687.
Linked Issues check	✅ Passed	The pull request addresses all coding objectives from issue #28687: it modifies URLSearchParams construction to handle string | string[] | undefined values correctly, preserving repeated query parameters instead of serializing them as comma-separated values.
Out of Scope Changes check	✅ Passed	The changes are focused on fixing the query parameter serialization in getServerSideProps, which is directly related to the stated objective of issue #28687 with no out-of-scope modifications present.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Review rate limit: 7/8 reviews remaining, refill in 7 minutes and 30 seconds.}

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[romitg2]

mind adding video demo showing before fix and after fix behavior.

[CLAassistant]

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}