-
Notifications
You must be signed in to change notification settings - Fork 34
Y25-586 - Part 2 - Ultima AMP plate layout verification #5349
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
base: develop
Are you sure you want to change the base?
Conversation
…e layout verification but with different format barcodes
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #5349 +/- ##
===========================================
- Coverage 87.13% 87.13% -0.01%
===========================================
Files 1452 1452
Lines 32404 32436 +32
Branches 3382 3386 +4
===========================================
+ Hits 28236 28263 +27
- Misses 4154 4159 +5
Partials 14 14 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
…ameter to distinguish between tubes and AMP plates - neater
…s probably a bit off.
…eneralise success message
yoldas
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good.
| <%= page_title "Verify #{@verification_flavour.to_s.humanize} Layout", "Batch #{@batch.id}" %> | ||
| <div class="info"> | ||
| Please scan the barcodes from the sample tubes into the table below. | ||
| <%= "Please scan the barcodes from the #{@verification_flavour.to_s.humanize.downcase.pluralize} into the table below." %> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
note: AMQ will be lowercase in ... from the amq plates into the ... , but OK.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes I know... was hoping it wasn't too important 😅
Closes #5272
Changes proposed in this pull request
Instructions for Reviewers
[All PRs] - Confirm PR template filled
[Feature Branches] - Review code
[Production Merges to
main]- Check story numbers included
- Check for debug code
- Check version