-
Notifications
You must be signed in to change notification settings - Fork 32
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
feat: mark strings for i18n on lpr page #1221
Conversation
da29e53
to
30849db
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #1221 +/- ##
==========================================
+ Coverage 85.34% 85.37% +0.03%
==========================================
Files 537 537
Lines 11632 11672 +40
Branches 2413 2464 +51
==========================================
+ Hits 9927 9965 +38
Misses 1656 1656
- Partials 49 51 +2 ☔ View full report in Codecov by Sentry. |
5367fa8
to
6f6176d
Compare
if (actionSlug) { | ||
downloadButtonLabel = intl.formatMessage({ | ||
id: 'admin.portal.lpr.current.report.csv.download', | ||
defaultMessage: 'Download current report (CSV)', |
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.
where we are handling Download full report (CSV)
?
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.
Download full report (CSV)
is handled here
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 👍
[Verify] please do verify that npm run i18n_extract
is working fine.
executed |
…l report section on lpr page
6f6176d
to
951ea91
Compare
For all changes
Only if submitting a visual change