Skip to content

[BI-2911] Remove rest of CRUD operations for ObservationDAO#532

Open
jloux-brapi wants to merge 4 commits into
bug/BI-2944from
feature/BI-2911
Open

[BI-2911] Remove rest of CRUD operations for ObservationDAO#532
jloux-brapi wants to merge 4 commits into
bug/BI-2944from
feature/BI-2911

Conversation

@jloux-brapi

@jloux-brapi jloux-brapi commented Jul 8, 2026

Copy link
Copy Markdown
Collaborator

Description

Story: BI-2911

  • References to cache.post() were removed and replaced with brapiDAOUtil.post()
  • References to cache.postThese() were removed and replaced with brapiDAOUtil.put()
  • References to repopulate the observation cache were removed
  • Extension of parent CachedDAO class removed

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 HMS17 and removed request for a team July 8, 2026 17:27
@jloux-brapi jloux-brapi added on hold Review, but do not merge until removed and removed feature labels Jul 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

on hold Review, but do not merge until removed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant