Skip to content

BI-2805 - Fix autocomplete to work with new BrAPI levelNames#505

Merged
davedrp merged 6 commits into
developfrom
feature/BI-2805
Apr 16, 2026
Merged

BI-2805 - Fix autocomplete to work with new BrAPI levelNames#505
davedrp merged 6 commits into
developfrom
feature/BI-2805

Conversation

@nickpalladino
Copy link
Copy Markdown
Member

@nickpalladino nickpalladino commented Mar 30, 2026

Description

Story: BI-2805

  • Merged with develop and removed old brAPIDAOUtil in BrAPIObservationLevelDAO code now that we have brapi client code
  • Added new GET /programs/{programId}/experiments/{experimentId}/recommended-sub-entity-dataset-names endpoint
  • Updated and re-enabled old sub entity test and added new tests for this card

Dependencies

Testing

  • See acceptance criteria in BI-2805

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

@nickpalladino nickpalladino changed the title Feature/bi 2805 BI-2805 - Fix autocomplete to work with new BrAPI levelNames Apr 10, 2026
@nickpalladino nickpalladino marked this pull request as ready for review April 16, 2026 13:51
@nickpalladino nickpalladino requested a review from davedrp April 16, 2026 15:22
Copy link
Copy Markdown
Contributor

@davedrp davedrp left a comment

Choose a reason for hiding this comment

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

Passed Developer test.
Code looks good

@davedrp davedrp merged commit 4d0add5 into develop Apr 16, 2026
1 check passed
@davedrp davedrp deleted the feature/BI-2805 branch April 16, 2026 19:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants