Skip to content

[BI-2574] - Migrate documentation links to the new BOX#432

Merged
mlm483 merged 2 commits intodevelopfrom
task/BI-2574
Mar 31, 2025
Merged

[BI-2574] - Migrate documentation links to the new BOX#432
mlm483 merged 2 commits intodevelopfrom
task/BI-2574

Conversation

@mlm483
Copy link
Contributor

@mlm483 mlm483 commented Mar 24, 2025

Description

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

Updated the download links for all template files to the new Box folder.

Note that TraitImportTemplateMessageBox was updated for completeness, it is not used anywhere.

Dependencies

Please include any dependencies to other code branches, testing configurations, scripts to be run, etc.

Testing

For each import template (germplasm, experiment, ontology and sample submission), ensure that

  1. the download works,
  2. the most recent template file is downloaded, and
  3. the download link matches the share link in the new Box folder.

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>
  • I have run SiteImprove on pages impacted by changes

@mlm483 mlm483 marked this pull request as ready for review March 24, 2025 21:32
@mlm483 mlm483 requested review from a team, davedrp and dmeidlin and removed request for a team March 24, 2025 21:32
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 test & Code Review

@mlm483 mlm483 merged commit a4a854e into develop Mar 31, 2025
1 check passed
@mlm483 mlm483 deleted the task/BI-2574 branch March 31, 2025 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants