Skip to content

fix(web): preserve regex and case sensitivity params in search load-more#972

Merged
brendan-kellam merged 2 commits intomainfrom
brendan-kellam/fix-search-load-more
Mar 2, 2026
Merged

fix(web): preserve regex and case sensitivity params in search load-more#972
brendan-kellam merged 2 commits intomainfrom
brendan-kellam/fix-search-load-more

Conversation

@brendan-kellam
Copy link
Contributor

@brendan-kellam brendan-kellam commented Mar 2, 2026

When clicking the "load more results" button in /search, the isRegexEnabled and isCaseSensitivityEnabled query parameters were being dropped. This fix includes these parameters in the URL built by onLoadMoreResults, matching the behavior of the initial search submission.

Summary by CodeRabbit

Bug Fixes

  • Fixed an issue where regex and case-sensitivity query parameters were not preserved when loading additional search results. These options are now maintained throughout pagination.

brendan-kellam and others added 2 commits March 2, 2026 10:11
When clicking the "load more results" button in /search, the isRegexEnabled and isCaseSensitivityEnabled query parameters were being dropped. This fix includes these parameters in the URL built by onLoadMoreResults, matching the behavior of the initial search submission.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions

This comment has been minimized.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 2, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 4483213 and 00478bb.

📒 Files selected for processing (2)
  • CHANGELOG.md
  • packages/web/src/app/[domain]/search/components/searchResultsPage.tsx

Walkthrough

The changes add functionality to preserve regex and case-sensitivity query parameters when users load additional search results. The searchResultsPage component now includes these flags in the URL during pagination, and the changelog documents this behavioral update.

Changes

Cohort / File(s) Summary
Search Parameters Preservation
CHANGELOG.md, packages/web/src/app/[domain]/search/components/searchResultsPage.tsx
Adds query parameter preservation for isRegexEnabled and isCaseSensitivityEnabled flags when loading more search results, along with documentation of this change.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and specifically describes the main change: preserving regex and case-sensitivity parameters in the search load-more functionality.
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 docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch brendan-kellam/fix-search-load-more

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

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

@brendan-kellam brendan-kellam merged commit 4004933 into main Mar 2, 2026
8 checks passed
@brendan-kellam brendan-kellam deleted the brendan-kellam/fix-search-load-more branch March 2, 2026 18:19
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.

1 participant