-
Notifications
You must be signed in to change notification settings - Fork 2
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
Add INSDC accession to results of ena submission and upload to backend. #2845
base: main
Are you sure you want to change the base?
Conversation
ena_submission_url: https://wwwdev.ebi.ac.uk/ena/submit/drop-box/submit # TODO(https://github.com/loculus-project/loculus/issues/2425): update in production | ||
github_url: https://raw.githubusercontent.com/pathoplexus/ena-submission/main/test/approved_ena_submission_list.json # TODO(https://github.com/loculus-project/loculus/issues/2425): update in production | ||
ena_reports_service_url: https://wwwdev.ebi.ac.uk/ena/submit/report # TODO(https://github.com/loculus-project/loculus/issues/2425): update in production | ||
submit_to_ena_prod: False # TODO(https://github.com/loculus-project/loculus/issues/2425): update in production |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In a later PR I will add a corresponding field in the values.yaml and update the ena-submission template to make sure this can be set to true via the values.yaml for Pathoplexus. In the same PR I will add a github workflow to prevent us setting this to true for a preview instance without an override as a safety mechanism.
cdf855d
to
008328e
Compare
resolves #
preview URL:
Summary
--test
flag (determines if ENA dev or prod is used) that is set at config level to make code clearer.Screenshot
PR Checklist