Skip to content

BI-2964: BrAPI Observation GET: Apply pagination and filtering on BrAPI side - #32

Open
humsika wants to merge 1 commit into
epic/BI-2862from
feature/BI-2964
Open

humsika wants to merge 1 commit into
epic/BI-2862from
feature/BI-2964

Conversation

@humsika

@humsika humsika commented Oct 12, 2026 •

Copy link
Copy Markdown

Description

Story - BI-2964
Fixed an edge case in BrAPI pagination where the first invalid page returned 200 OK when the total record count was exactly divisible by the page size. The shared validator now correctly returns 400 Bad Request when page >= totalPages.

Dependencies

bi-web: epic/BI-2862
bi-api: feature/BI-2964
brapi-Java-ProdServer: feature/BI-2964

Testing

  • Open Experiments & Observations and select an experiment containing observations.
  • Verify that the observations load successfully.
  • Apply filters and confirm the displayed observations match the selected filters.
  • Navigate between different pages and confirm the correct data loads on each page without errors.
  • Change the page size, if available, and confirm pagination continues to work correctly.

Checklist:

@humsika
humsika requested a review from jloux-brapi October 12, 2026 21:27
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