Skip to content

Conversation

@nickpalladino
Copy link
Member

@nickpalladino nickpalladino commented Dec 5, 2025

Description

Story: BI-2743

  • Create new /observationlevelnames when creating subentity dataset
  • Added tests

Dependencies

  • bi-web feature/BI-2758
  • brapi-server breedinginsight/brapi-java-server:pr-17 docker image

Testing

Acceptance criteria in BI-2758 & BI-2743

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 tested that my code works with both the brapi-java-server and BreedBase
  • I have create/modified unit tests to cover this change
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to documentation
  • I have run TAF: <please include a link to TAF run>

Copy link
Contributor

@HMS17 HMS17 left a comment

Choose a reason for hiding this comment

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

Code looks good. Like with BI-2758, needs to be tested again once brapi capitalization fix implemented.

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.

3 participants