Skip to content

BDMS 296: increase limit size for search#259

Merged
jirhiker merged 2 commits into
stagingfrom
jab-search-update
Nov 25, 2025
Merged

BDMS 296: increase limit size for search#259
jirhiker merged 2 commits into
stagingfrom
jab-search-update

Conversation

@jacob-a-brown

Copy link
Copy Markdown
Contributor

Why

This PR addresses the following problem / context:

  • A pagination size of 25 is too restrictive

How

Implementation summary - the following was changed / added / removed:

  • Increased the size to 100

Notes

Any special considerations, workarounds, or follow-up work to note?

  • I haven't tested a pagination in the search tests, but from the research I've done setting a query parameter called size should do the trick.

@codecov-commenter

codecov-commenter commented Nov 25, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

Files with missing lines Coverage Δ
api/search.py 97.82% <ø> (ø)

... and 23 files with indirect coverage changes

@TylerAdamMartinez TylerAdamMartinez 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.

lgtm

@jirhiker jirhiker merged commit 3e5864f into staging Nov 25, 2025
6 checks passed
@TylerAdamMartinez TylerAdamMartinez deleted the jab-search-update branch February 26, 2026 18:29
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.

6 participants