Skip to content

feat: remove default search params - #116

Merged
vnepogodin merged 1 commit into
CachyOS:developfrom
azdanov:remove-search-defaults
Apr 25, 2026
Merged

feat: remove default search params#116
vnepogodin merged 1 commit into
CachyOS:developfrom
azdanov:remove-search-defaults

Conversation

@azdanov

@azdanov azdanov commented Apr 23, 2026

Copy link
Copy Markdown
Member

@azdanov
azdanov requested review from Copilot and vnepogodin April 23, 2026 17:50

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds TanStack Router search middleware to keep the URL clean by stripping default search parameters on the home/package search route.

Changes:

  • Introduces a defaultValues object for the / route’s search params.
  • Simplifies toQueryParams by merging defaultValues with the validated search object.
  • Adds stripSearchParams(defaultValues) middleware to remove defaults from the URL search string.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@azdanov
azdanov force-pushed the remove-search-defaults branch from c405f22 to bcc5025 Compare April 25, 2026 17:58

@vnepogodin vnepogodin left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Many thanks

@vnepogodin
vnepogodin merged commit 13274b6 into CachyOS:develop Apr 25, 2026
1 check passed
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.

3 participants