Skip to content

[BI-1224] - Show list creator in Germplasm lists table#194

Merged
HMS17 merged 2 commits intodevelopfrom
feature/BI-1224
Jul 5, 2022
Merged

[BI-1224] - Show list creator in Germplasm lists table#194
HMS17 merged 2 commits intodevelopfrom
feature/BI-1224

Conversation

@HMS17
Copy link
Contributor

@HMS17 HMS17 commented Jun 23, 2022

Description

Story: BI-1224 - Show list creator in Germplasm lists table

Retrieves list creator and displays for germplasm lists.

As the brapi listOwnerName is not stored in breedbase, the list creator is retrieved from the first germplasm in the list, which requires multiple calls to retrieve alas. Trying to store listOwnerName in additionalInfo on list create does not work alas as additionalInfo for lists is also not saved on breedbase.

Dependencies

bi-web/BI-1224

Testing

see bi-web

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>

@HMS17 HMS17 requested review from a team, davedrp and timparsons and removed request for a team and timparsons June 27, 2022 20:58
@HMS17 HMS17 requested a review from ctucker3 June 27, 2022 20:59
@HMS17 HMS17 marked this pull request as ready for review June 27, 2022 20:59
@HMS17 HMS17 merged commit f0f441a into develop Jul 5, 2022
@HMS17 HMS17 deleted the feature/BI-1224 branch July 5, 2022 14:43
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