Skip to content

[BI-2914] Swap OU exref usages with brapi OU dbId - #536

Merged
jloux-brapi merged 2 commits into
epic/BI-2862from
feature/BI-2914
Aug 25, 2026
Merged

[BI-2914] Swap OU exref usages with brapi OU dbId#536
jloux-brapi merged 2 commits into
epic/BI-2862from
feature/BI-2914

Conversation

@jloux-brapi

@jloux-brapijloux-brapi commented Jul 23, 2026

Copy link
Copy Markdown
Collaborator

Description

Story:BI-2914

  • Swapped exref usages on BI observation usages with brapi observation unit dbIds.
  • Fixed some remaining trial and observation usages as well

Dependencies

Testing

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 humsika and removed request for a teamJuly 23, 2026 18:24
@jloux-brapijloux-brapi added the on hold Review, but do not merge until removed label Jul 23, 2026

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

Code looks good to me. No changes needed.

Base automatically changed from feature/BI-2913 to epic/BI-2862August 25, 2026 19:32
@jloux-brapijloux-brapi removed the on hold Review, but do not merge until removed label Aug 25, 2026
@jloux-brapi
jloux-brapi merged commit bce57fc into epic/BI-2862Aug 25, 2026
6 of 8 checks passed
@jloux-brapi
jloux-brapi deleted the feature/BI-2914 branch August 25, 2026 20:07
Sign up for freeto 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

@jloux-brapi@humsika