Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat!: rename and surface sraRunAccession #3524

Merged
merged 2 commits into from
Jan 20, 2025
Merged

feat!: rename and surface sraRunAccession #3524

merged 2 commits into from
Jan 20, 2025

Conversation

fhennig
Copy link
Contributor

@fhennig fhennig commented Jan 14, 2025

BREAKING: value.yaml needs to be updated (see changed files).

resolves pathoplexus/pathoplexus#331

preview URL: http://rename-srarunacc.loculus.org

Summary

  • rename the field as discussed.
  • change display name to 'Raw reads accession'

Screenshot

n/a

PR Checklist

  • All necessary documentation has been adapted.
  • The implemented feature is covered by an appropriate test.

@fhennig fhennig added the preview Triggers a deployment to argocd label Jan 14, 2025
@fhennig fhennig self-assigned this Jan 14, 2025
@rneher
Copy link

rneher commented Jan 19, 2025

anything blocking here?

@fhennig
Copy link
Contributor Author

fhennig commented Jan 20, 2025

The metadata docs on the website are generated, also the template file is generated, so these should be fine too.

Not sure about this: https://github.com/pathoplexus/pathoplexus/blob/main/monorepo/website/public/metadata_descriptions.csv @anna-parker ?

@fhennig fhennig marked this pull request as ready for review January 20, 2025 10:19
@anna-parker anna-parker self-requested a review January 20, 2025 13:28
Copy link
Contributor

@anna-parker anna-parker left a comment

Choose a reason for hiding this comment

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

lgtm! we just need to make a PR in Pathoplexus as well with the values.yaml changes - it would also be great to mark this as a breaking change so we remember to rollout those changes together!

@fhennig fhennig changed the title feat: rename and surface sraRunAccession feat!: rename and surface sraRunAccession Jan 20, 2025
@fhennig fhennig merged commit d682371 into main Jan 20, 2025
18 checks passed
@fhennig fhennig deleted the rename-srarunacc branch January 20, 2025 14:54
@fhennig
Copy link
Contributor Author

fhennig commented Jan 20, 2025

yes good point! I've updated the title and description accordingly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
preview Triggers a deployment to argocd
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEAT] Rename and surface sraRunAccession
3 participants