Skip to content

BI-1680 - Updated Germplasm Template To Remove "Somoclone"#370

Closed
mlm483 wants to merge 1 commit intodevelopfrom
feature/BI-1680
Closed

BI-1680 - Updated Germplasm Template To Remove "Somoclone"#370
mlm483 wants to merge 1 commit intodevelopfrom
feature/BI-1680

Conversation

@mlm483
Copy link
Contributor

@mlm483 mlm483 commented Apr 1, 2024

Description

Story: BI-1680

I updated the germplasm import template to remove the "Somoclone" breeding method from the Breeding Methods worksheet.

This bi-api PR contains the corresponding database migration.

Dependencies

bi-api branch: feature/BI-1680.

Testing

  1. Checkout this branch, git checkout feature/BI-1680.
  2. Run DeltaBreed.
  3. Download the germplasm import template.
    • Ensure the template version is v10.
    • Ensure the template Breeding Methods worksheet does not contain the "Somoclone" method.

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 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: <link to TAF run>

@mlm483 mlm483 marked this pull request as ready for review April 1, 2024 16:50
@mlm483 mlm483 requested review from davedrp and nickpalladino April 1, 2024 16:53
@mlm483 mlm483 changed the title BI-1680 - Updated Germplasm Template BI-1680 - Updated Germplasm Template To Remove "Somoclone" Method Apr 1, 2024
@mlm483 mlm483 changed the title BI-1680 - Updated Germplasm Template To Remove "Somoclone" Method BI-1680 - Updated Germplasm Template To Remove "Somoclone" Apr 1, 2024
Copy link
Contributor

@davedrp davedrp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Passed developer testing

@mlm483
Copy link
Contributor Author

mlm483 commented Apr 10, 2024

Closing in favor of #373, which is based on release/0.9.

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