Closed ipediez closed 2 years ago
Reviewer's: @aaclan-ebi & @yusra-haider
This issue is 2 fold:
primaryWrangler: null
, ingest-ui strips out all null values before sending the data to ingest core.
Work in progress solution for problem 1: "there is no interface (clickable) to allow clearing a wrangler assignment":
Current Sequence Diagram: https://miro.com/app/board/uXjVOSWgtMw=/
@gabsie @ipediez and @ofanobilbao to take a look at the design and the functionality
Hi, @MightyAx - I am good with the designs shown above !
@ofanobilbao you wanted to know the other fields that can't currently be un-set, that should be resolved by current intended solution:
Awaiting PR Reviews
Jacob is reviewing
Review comments resolved, merged and deployed to dev. merged to master and deployed to staging. Will sit with @Wkt8 this afternoon to verify.
Have sat with @MightyAx and tested it - looks great. Clear, and fulfils what we are trying to do. I think we can put this into production.
Deployed to Production. Verified functionality is on production
Description of the task:
The project Multiomics single-cell analysis of human pancreatic islets reveals novel cellular states in health and type 1 diabetes is not wrangleable right now, as it is a pre-print with no data or metadata available at the moment. To avoid this affecting our way of filtering and triaging datasets in Ingest, it would be great to set back the value of the "Primary Wrangler" field to its default. This cannot be done from the UI, where the last value of the field gets automatically assigned when left blank, so a dev action is needed.
Acceptance criteria for the task:
Other fields that can't currently be un-set, should be resolved by current intended solution: