fix: add input search field on show pages list page

[tomrndom]

ref: https://app.clickup.com/t/86b96pbpc

Signed-off-by: Tomás Castillo tcastilloboireau@gmail.com

Summary by CodeRabbit

Release Notes

• New Features
  • Added search functionality to the show pages list. Users can now enter search terms to filter results while preserving existing filters and sorting preferences.

[coderabbitai]

📝 Walkthrough

Walkthrough

A search input field has been added to the show pages list interface, enabling users to filter results by a search term while maintaining existing pagination and filter parameters.

Changes

Cohort / File(s)	Summary
Search Functionality Addition src/pages/sponsors/show-pages-list-page/index.js	Added SearchInput component to page header, implemented handleSearch handler that calls getShowPages() with search term while resetting pagination to DEFAULT_CURRENT_PAGE and preserving filter/sort parameters.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

• fix: adjust current page param after add new sponsor to summit #803: Both PRs reset pagination to DEFAULT_CURRENT_PAGE in sponsor-related contexts, suggesting similar pagination handling patterns across sponsor operations.

Suggested reviewers

• smarcet

Poem

🐰 A search box hops into view,
Through sponsor pages, hopping through,
With filters held and pagination new,
The search term leads us, crisp and true! 🔍

🚥 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 'fix: add input search field on show pages list page' directly and clearly describes the main change: adding a search input field to the show pages list page.
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

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/show-pages-search

---

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

[coderabbitai]

🧹 Nitpick comments (1)

src/pages/sponsors/show-pages-list-page/index.js (1)

119-124: Consider preserving search context after save.

After a page is saved, getShowPages() is called without parameters, which resets the search term. If users search, edit a page, and save, they'll lose their search filter. Consider passing the current search state:

♻️ Suggested improvement

 const handleSaveShowPage = (entity) => {
   saveShowPage(entity).then(() => {
     setOpenPopup(null);
-    getShowPages();
+    getShowPages(term, currentPage, perPage, order, orderDir, hideArchived);
   });
 };

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@src/pages/sponsors/show-pages-list-page/index.js` around lines 119 - 124,
handleSaveShowPage currently calls saveShowPage(entity) and then getShowPages()
which clears the user's active search; preserve the current search/filter by
passing the existing search state into getShowPages after saving. Modify
handleSaveShowPage to read the current searchTerm (or filters/pagination state
used by getShowPages) and call getShowPages(searchTermOrFilters) instead of bare
getShowPages(), keeping the setOpenPopup(null) behavior and ensuring
getShowPages accepts and applies the passed search context (update getShowPages
signature if needed).

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@src/pages/sponsors/show-pages-list-page/index.js`:
- Around line 119-124: handleSaveShowPage currently calls saveShowPage(entity)
and then getShowPages() which clears the user's active search; preserve the
current search/filter by passing the existing search state into getShowPages
after saving. Modify handleSaveShowPage to read the current searchTerm (or
filters/pagination state used by getShowPages) and call
getShowPages(searchTermOrFilters) instead of bare getShowPages(), keeping the
setOpenPopup(null) behavior and ensuring getShowPages accepts and applies the
passed search context (update getShowPages signature if needed).

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 9d81b39e-b715-4abd-a697-5ee6e7bb0c18

📥 Commits

Reviewing files that changed from the base of the PR and between 2fa7555 and d68307f.

📒 Files selected for processing (1)

• src/pages/sponsors/show-pages-list-page/index.js