Skip to content

Generalize BrAPI Entity GET code, and pagination/sorting/filtering code#527

Open
jloux-brapi wants to merge 3 commits into
feature/BI-2861from
feature/BI-2932
Open

Generalize BrAPI Entity GET code, and pagination/sorting/filtering code#527
jloux-brapi wants to merge 3 commits into
feature/BI-2861from
feature/BI-2932

Conversation

@jloux-brapi

@jloux-brapi jloux-brapi commented Jun 17, 2026

Copy link
Copy Markdown
Collaborator

Description

Story: BI-2932
Added generalized code for usage by other developers in brapi entity cache removal processes for the following usages:

  • Generalized code for pagination/sorting/filtering in BrAPIDAOUtil
  • Generalized code for BrAPI entity GET in BrAPIDAOUtil. This code will set default pagination parameters if they are not set in submitted BrapiQueryParams

Dependencies

brapi

Testing

Same regression as BI-2861/2806

Checklist:

  • I have performed a self-review of my own code
  • I have tested my code and ensured it meets the acceptance criteria of the story
  • I have create/modified unit and/or integration tests to cover this change or tests are not applicable
  • I have commented my code, particularly in hard-to-understand areas
  • I have either updated the source of truth or arranged for update with product owner if needed https://breedinginsight.atlassian.net/wiki/spaces/BI/pages/1559953409/Source+of+Truth

@jloux-brapi jloux-brapi requested review from a team and HMS17 and removed request for a team June 17, 2026 00:17
@jloux-brapi jloux-brapi requested review from davedrp and removed request for HMS17 June 17, 2026 00:53
@jloux-brapi jloux-brapi added the on hold Do not merge until this label is removed label Jun 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature on hold Do not merge until this label is removed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants