Skip to content

[BI-2916] Remove remaining CRUD operations and references to OU programCache - #537

Merged
jloux-brapi merged 1 commit into
epic/BI-2862from
feature/BI-2916
Aug 25, 2026
Merged

[BI-2916] Remove remaining CRUD operations and references to OU programCache#537
jloux-brapi merged 1 commit into
epic/BI-2862from
feature/BI-2916

Conversation

@jloux-brapi

@jloux-brapijloux-brapi commented Jul 23, 2026

Copy link
Copy Markdown
Collaborator

Description

Story:BI-2916

Removed references to programCache in BrAPIObservationUnitDAO, and in other classes. The DAO is now a standalone class since it doesn't need any methods from its parent cacheDAO class.

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 nickpalladino and removed request for a teamJuly 23, 2026 21:00
@jloux-brapijloux-brapi changed the title Remove remaining CRUD operations and references to OU programCache[BI-2916] Remove remaining CRUD operations and references to OU programCacheJul 23, 2026
@jloux-brapijloux-brapi added the on hold Review, but do not merge until removed label Aug 25, 2026
Base automatically changed from feature/BI-2914 to epic/BI-2862August 25, 2026 20:07
@jloux-brapi
jloux-brapi merged commit 19ae9d0 into epic/BI-2862Aug 25, 2026
7 of 8 checks passed
@jloux-brapi
jloux-brapi deleted the feature/BI-2916 branch August 25, 2026 20:23
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

featureon holdReview, but do not merge until removed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@jloux-brapi@nickpalladino