-
Notifications
You must be signed in to change notification settings - Fork 177
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
refactor(app): update Labware Help Modal #8558
Conversation
Codecov Report
@@ Coverage Diff @@
## edge #8558 +/- ##
==========================================
+ Coverage 74.42% 74.45% +0.02%
==========================================
Files 1685 1685
Lines 45518 45564 +46
Branches 4551 4564 +13
==========================================
+ Hits 33878 33924 +46
Misses 10847 10847
Partials 793 793
Flags with carried forward coverage won't be shown. Click here to find out more.
|
const ROBOT_CAL_HELP_ARTICLE = | ||
'https://support.opentrons.com/en/articles/3499692-how-positional-calibration-works-on-the-ot-2' | ||
|
||
const OFFSET_DATA_HELP_ARTICLE = '#' // REPLACE WITH ACTUAL LINK |
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.
Emily said the article this is supposed to link to doesn't exist yet
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.
I'd add a TODO: IMMEDIATELY to this comment as a reminder that this needs to be addressed before release.
Hi @emilywools, please review the following and let me know how everything looks! Thanks 😄 > |
hey Jethary, this looks great! One thing: the spacing before the "learn more about xx" and after should be different. Before the link, the paragraph spacing should be a little bit less so it looks like it belongs with the paragraph above it. |
@emilywools, thanks for noticing that! I did have extra spacing. How is this? |
LGTM!!! |
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 to me. Let's make sure to flag the missing article link as an immediate TODO
const ROBOT_CAL_HELP_ARTICLE = | ||
'https://support.opentrons.com/en/articles/3499692-how-positional-calibration-works-on-the-ot-2' | ||
|
||
const OFFSET_DATA_HELP_ARTICLE = '#' // REPLACE WITH ACTUAL LINK |
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.
I'd add a TODO: IMMEDIATELY to this comment as a reminder that this needs to be addressed before release.
Overview
closes #8325
Changelog
Labware Position Check and Offset Data
section in Labware setup accordionLabwareSetupModal
toLabwareOffsetModal
to describe the information in the new modal betterReview requests
Risk assessment
low, behind ff