Skip to content

Add bulk search link to advanced search#12396

Open
Shikhar-395 wants to merge 1 commit intointernetarchive:masterfrom
Shikhar-395:feature/add-bulk-search-link-to-advanced-search
Open

Add bulk search link to advanced search#12396
Shikhar-395 wants to merge 1 commit intointernetarchive:masterfrom
Shikhar-395:feature/add-bulk-search-link-to-advanced-search

Conversation

@Shikhar-395
Copy link
Copy Markdown
Contributor

Closes #9715

Summary

Adds a Bulk Search (beta) link to the advanced search page so the bulk search feature is discoverable without exposing it broadly in the main search UI.

Screenshot

Before
Screenshot 2026-04-17 at 05 19 37
After
Screenshot 2026-04-17 at 05 12 35

Stakeholders

@RayBB

@mekarpeles
Copy link
Copy Markdown
Member

Thanks for the PR, @Shikhar-395!

🤖 Copilot has been assigned for an initial review.

@RayBB is assigned to this PR and currently has:

  • 1 open PR(s) of equal or higher priority to review first
PR triage checklist (maintainers / Pam)
  • PR description — not empty; explains what the change does and how to verify it
  • References an issue — PR body contains a #NNN reference
    • Linked issue is triaged — has a Priority: * label (not just Needs: Triage)
    • Linked issue is assigned — has at least one assignee
  • Commit history clean — no WIP/fixup/conflict noise; commit messages are meaningful
  • CI passing — no failing check-runs
  • Test cases present — if the change touches substantive logic, test coverage exists or is explained
  • Proof of testing — PR body includes a description of what was tested, a screenshot, or a video

Note

This comment was automatically generated by Pam, Open Library's Project AI Manager, on behalf of @mekarpeles. Pam is designed to provide status visibility, perform basic project management functions and relevant codebase research, and provide actionable feedback so contributors aren't left waiting.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a discoverability link for the Bulk Search feature by surfacing “Bulk Search (beta)” on the Advanced Search page (per #9715), without adding it to the main search UI.

Changes:

  • Add “Bulk Search (beta)” link to the Advanced Search page’s helper links.
  • Update i18n message references to include the Advanced Search template as a source for the existing “Bulk Search (beta)” string.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
openlibrary/templates/search/advancedsearch.html Adds a /search/bulk link labeled “Bulk Search (beta)” alongside existing search helper links.
openlibrary/i18n/messages.pot Updates source references for the existing “Bulk Search (beta)” msgid to include search/advancedsearch.html.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Waiting Review/Merge from Staff

Development

Successfully merging this pull request may close these issues.

Add links to bulk search from search dropdown, search toolbar

4 participants