Closed cosa65 closed 5 months ago
Removed the goal:
- Update the programmatic interface to fit the new api endpoints
Because there is already a ticket to make programmatic interface work again (it is already broken). I will update this other ticket with this change too
Background There was a bug that was fixed here, in that PR's description there is an explanation about the problem and the fix.
It is clear that checks need to be established to make sure this can't happen again.
One way we could ensure this doesn't happen again is to change the flow to upload a sample file:
/experiments/{experimentId}/samples/{sampleId}/sampleFiles/{sampleFileType}
)/experiments/{experimentId}/samples/{sampleId}/sampleFiles/{sampleFileType}
with/sampleFiles/{sampleFileId}
).sampleFileId
is only used duringprocessUpload
, it shouldn't be stored anywhere in redux. We want the browser to only be able to patch the uploadStatus while the upload is in progress, not after.Goal