Skip to content

fix(providers): sync search query to URL query param (#8624) - #8652

Merged
diegosouzapw merged 1 commit into
cheaper-inference:release/v3.8.49from
Dingding-leo:pr/fix-sync-search-to-url-8624
Jul 27, 2026
Merged

diegosouzapw merged 1 commit into
cheaper-inference:release/v3.8.49from
Dingding-leo:pr/fix-sync-search-to-url-8624

Conversation

@Dingding-leo

Copy link
Copy Markdown
Contributor

Summary

Fixes #8624.

Changes

  • Export syncSearchToUrl in providerPageUtils.ts to update or delete the ?search= URL query parameter via window.history.replaceState.
  • Call syncSearchToUrl(searchQuery) inside a useEffect in src/app/(dashboard)/dashboard/providers/page.tsx.
  • Added unit test coverage in tests/unit/sync-search-to-url-8624.test.ts.

Add syncSearchToUrl helper in providerPageUtils and hook it up in providers page useEffect so searching updates the ?search= URL param seamlessly.
Copilot AI review requested due to automatic review settings July 26, 2026 08:44

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@diegosouzapw

Copy link
Copy Markdown
Collaborator

Validated in local merge-train /tmp/train1d-20260727-090022-suite.log on .113 @ 029cdf4215cf465f0e1716ac9f84a84692b1e881 (full unit suite green on CI-equivalent host)

@diegosouzapw
diegosouzapw merged commit 7e2c687 into cheaper-inference:release/v3.8.49 Jul 27, 2026
14 of 15 checks passed
@diegosouzapw diegosouzapw mentioned this pull request Jul 28, 2026
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