-
Notifications
You must be signed in to change notification settings - Fork 178
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
fix(app): Copy fixes in app flows #14167
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## chore_release-7.1.0 #14167 +/- ##
====================================================
Coverage 70.45% 70.46%
====================================================
Files 2512 2512
Lines 71217 71216 -1
Branches 8971 8970 -1
====================================================
+ Hits 50177 50180 +3
+ Misses 18844 18842 -2
+ Partials 2196 2194 -2
Flags with carried forward coverage won't be shown. Click here to find out more.
|
@@ -20,7 +20,7 @@ | |||
"labware": "labware", | |||
"last_analyzed": "last analyzed", | |||
"last_updated": "last updated", | |||
"left_and_right_mounts": "Left+Right Mounts", | |||
"left_and_right_mounts": "Both Mounts", |
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.
nit
maybe change the name to both_mounts
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.
lgtm
RQA-2048, RQA-2016, RQA-2023
Overview
This PR fixes a few small copy related bugs
Test Plan
Tested all of these out:
Changelog
Review requests
Sanity check code
Risk assessment
Low