-
Notifications
You must be signed in to change notification settings - Fork 0
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
submission portal UI updates - merge context and multi-omics pages #1512
Comments
Mary will start on this tomorrow |
Expect this to be done this sprint :) |
is it done? |
@marySalvi would you please link the relevant PR? |
Still need to add links for the material processing references & some of the expansion options still needs work. Mary will share the branch with Patrick. |
@aclum @marySalvi is this issue still actively being worked on? |
@marySalvi can we say this is in review? I know things won't be merged until Patrick is done but I believe this part is ready. |
Will move to next sprint and move to in review for when Patrick is back |
I'm closing this assuming it will be in the release today. @marySalvi @aclum |
Changes haven't been merged into main yet. |
Discussed and this at infra sync meeting. Will be part of April release, 4/21/25. |
As part of the squad to accept external fastq data we need some updates to the UI. Please implement merging the context and multi-omics pages following the logic designed in https://drive.google.com/file/d/1rf1ejqxq4Xj1A-AkY1QvRir5CABfpHa7/view?usp=drive_link
cc @pkalita-lbl
The text was updated successfully, but these errors were encountered: