-
Notifications
You must be signed in to change notification settings - Fork 179
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): fix various app copy #13557
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## chore_release-7.0.0 #13557 +/- ##
=======================================================
- Coverage 71.30% 71.30% -0.01%
=======================================================
Files 2419 2419
Lines 68082 68083 +1
Branches 7908 7909 +1
=======================================================
Hits 48547 48547
- Misses 17683 17684 +1
Partials 1852 1852
Flags with carried forward coverage won't be shown. Click here to find out more.
|
8cf612f
to
b01bec2
Compare
b01bec2
to
062fa01
Compare
062fa01
to
1f1da03
Compare
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, just left a question/suggestion for combining the i18n keys that have the same value
"pipette_left": "pipette", | ||
"pipette_right": "pipette", |
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.
can these be updated into 1 i18n key now?
Overview
This PR makes two copy changes:
Test Plan
Changelog
See description
Review requests
Make sure copy is updated properly
Risk assessment
Low