fix(website): only put fasta Id into fastaIds in edit page#5636
fix(website): only put fasta Id into fastaIds in edit page#5636corneliusroemer wants to merge 1 commit intoedit-page-anyafrom
Conversation
There was a problem hiding this comment.
could you add some of the integration test changes I made in https://github.com/loculus-project/loculus/pull/5629/files?
Update: I cherrypicked your changes into #5629, added tests and made the difference between header and ID clearer on the edit page
bce2b8a to
2213cec
Compare
4c94ba7 to
247b990
Compare
They don't matter? This fix here is contained and should be good to go? |
|
In https://github.com/loculus-project/loculus/pull/5629/files I also improve the error message - we were checking if the full fasta header was the same not the fasta ID. If we only merge these changes here a user could submit a sequence with the same fastaID and a different description - causing submission issues when this is sent to the backend |
|
superseded by #5629 |
resolves #5627
🚀 Preview: Add
previewlabel to enable