bulk_scan barcode #512
bulk_scan barcode #512
Conversation
Master overhaul merge to my fork
| return response | ||
|
|
||
|
|
||
| def get_sample_from_rack(token_info, rack_id): |
There was a problem hiding this comment.
The mapping between samples and racks needs to be sensitive to the fact that samples can move from rack to rack. Right now, the code just assumes that there's a stable relationship, which results in issues when you test using different sample barcodes with the same rack ID. Retrieving the samples from a rack should only reflect the samples that were present at the most recent scan, not every sample that has ever been on the rack.
Also, please pluralize the function name to reflect that we expect multiple samples to exist on a rack.
There was a problem hiding this comment.
Resolved the issue!
|
@raginirai553 We just merged a big set of changes into master-overhaul. Can you please back-out the changes to |
No description provided.