Skip to content

[BI-2806] Route Experiments using trialDbId, use trialDbId to get metadata#466

Merged
jloux-brapi merged 1 commit into
epic/BI-2862from
feature/BI-2806
Jun 22, 2026
Merged

[BI-2806] Route Experiments using trialDbId, use trialDbId to get metadata#466
jloux-brapi merged 1 commit into
epic/BI-2862from
feature/BI-2806

Conversation

@jloux-brapi

@jloux-brapi jloux-brapi commented May 6, 2026

Copy link
Copy Markdown
Contributor

Description

Story: BI-2806

These changes support the removal of BrAPITrials from the ProgramCache.

The Experiment table will now route experiments using their BrAPITrialDbId, and the metadata grab associated will also use the BrAPITrialDbId.

Dependencies

bi-api PR

Testing

Regression on just about anything Experiment related, including:

  • Experiment creation
  • Experiment deletion
  • Experiment updates
  • Experiment Collaborator checks
  • Experiment viewing
  • Dataset viewing
  • Dataset creation
  • Dataset appending

Verify there are no log messages related to caching trials for a program by looking at logs once above tests have been completed.

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 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
  • I have run SiteImprove on pages impacted by changes

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

Everything looks good to me.

Comment thread src/components/experiments/ExperimentsObservationsTable.vue
Comment thread src/breeding-insight/service/ExperimentService.ts
Comment thread src/components/experiments/ExperimentsObservationsTable.vue
@jloux-brapi jloux-brapi added the on hold Don't merge until this label is removed label Jun 10, 2026
@jloux-brapi jloux-brapi removed the on hold Don't merge until this label is removed label Jun 22, 2026
@jloux-brapi jloux-brapi merged commit 7a41bf5 into epic/BI-2862 Jun 22, 2026
2 checks passed
@jloux-brapi jloux-brapi deleted the feature/BI-2806 branch June 22, 2026 21:44
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.

5 participants