Skip to content

BI-1975 - "Unknown" Breeding Method not recognized - #369

Merged
nickpalladino merged 2 commits into
developfrom
bug/BI-1975
Jul 2, 2024
Merged

BI-1975 - "Unknown" Breeding Method not recognized#369
nickpalladino merged 2 commits into
developfrom
bug/BI-1975

Conversation

@nickpalladino

@nickpalladinonickpalladino commented Jun 26, 2024•
edited by atlassian Bot
Loading

Copy link
Copy Markdown
Member

Description

Story:BI-1975

  • Added db migration to change UNK breeding method class to Increase

Dependencies

Testing

  • Germplasm import accepts UNK and Unknown breeding methods for germplasm records
  • Breeding methods in program administration page shows Increase for the UNK method class

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>

@github-actionsgithub-actionsBot added the bug Something isn't working label Jun 26, 2024
@nickpalladino
nickpalladino requested review from a team, dmeidlin and mlm483 and removed request for a teamJune 26, 2024 21:00
@nickpalladino
nickpalladino requested review from HMS17 and removed request for dmeidlinJuly 1, 2024 15:10
@mlm483mlm483 self-assigned this Jul 1, 2024
mlm483
mlm483 approved these changes Jul 1, 2024
View reviewed changes

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

Tested, working.

@nickpalladino
nickpalladino merged commit 4e9e1b8 into developJul 2, 2024
@nickpalladino
nickpalladino deleted the bug/BI-1975 branch July 2, 2024 15:31
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@HMS17HMS17HMS17 approved these changes
@mlm483mlm483mlm483 approved these changes

Assignees

@mlm483mlm483

Labels

bugSomething isn't working

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@nickpalladino@HMS17@mlm483