Skip to content

[BI-1638] - Cache germplasm data immediately after saving to BrAPI#218

Merged
timparsons merged 1 commit intorelease/0.7from
feature/BI-1638
Oct 17, 2022
Merged

[BI-1638] - Cache germplasm data immediately after saving to BrAPI#218
timparsons merged 1 commit intorelease/0.7from
feature/BI-1638

Conversation

@timparsons
Copy link
Member

Description

Story: https://breedinginsight.atlassian.net/browse/BI-1638

Updated the post method in ProgramCache to add records to the cache before calling the updateCache method. This will allows the cache to have the records that were just saved, while the new data to refresh the cache with is pulled in the background.

Dependencies

bi-web: release/0.7

Testing

  1. Upload a germplasm file (preferably a larger file so that refreshing the cache takes a few minutes)
  2. After the upload is complete, immediatly navigate to the All Germplasm table, and verify that the new records are there
  3. Switch to the Germplasm List tab, and verify the new list is there, and that the page loads without error

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>

…updateCache method so that records are immediately available
@timparsons timparsons requested review from a team, dmeidlin and nickpalladino and removed request for a team October 13, 2022 21:14
@timparsons timparsons merged commit c7bdeec into release/0.7 Oct 17, 2022
@timparsons timparsons deleted the feature/BI-1638 branch October 17, 2022 20:11
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