-
Notifications
You must be signed in to change notification settings - Fork 8
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
Y24-487: Add tooltips to racked tube icons #2113
base: y24-088-tuberacks-epic
Are you sure you want to change the base?
Y24-487: Add tooltips to racked tube icons #2113
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## y24-088-tuberacks-epic #2113 +/- ##
==========================================================
- Coverage 80.78% 80.68% -0.11%
==========================================================
Files 482 483 +1
Lines 18295 18299 +4
Branches 269 268 -1
==========================================================
- Hits 14780 14764 -16
- Misses 3513 3533 +20
Partials 2 2
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
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.
Do you have a full page screenshot as well? I am trying to figure out how Fail screen and tooltips would look like. I can't figure out if there is any interference between these tooltips and QC tooltips. Thanks.
Hi @yoldas, Here is a full screenshot as requested: |
Closes #2098
Changes proposed in this pull request
well
totube
It currently shows:
If more fields are required, another (or different) request will need to be made to SS to retreive the additional fields
Screenshot
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