Skip to content

Commit

Permalink
Rename Monthly Monitor report to *Plus report
Browse files Browse the repository at this point in the history
Also, since that setting only applies to Plus users, moved the
string to the non-localised strings. (The previous string might
come back in a future feature, but as I understand it, Pontoon will
remember it for easy re-translation.)
  • Loading branch information
Vinnl committed Aug 27, 2024
1 parent 1c9440f commit d8f6ff4
Show file tree
Hide file tree
Showing 4 changed files with 9 additions and 6 deletions.
5 changes: 5 additions & 0 deletions locales-pending/settings-premium.ftl
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,11 @@
# License, v. 2.0. If a copy of the MPL was not distributed with this
# file, You can obtain one at http://mozilla.org/MPL/2.0/.

## Email preferences

settings-alert-preferences-allow-monthly-monitor-plus-report-title = Monthly { -brand-monitor-plus } report
settings-alert-preferences-allow-monthly-monitor-plus-report-subtitle = A monthly update of new exposures, what’s been fixed, and what needs your attention.
## Cancel Plus subscription

settings-cancel-plus-title = Cancel { -brand-monitor-plus } subscription
Expand Down
2 changes: 0 additions & 2 deletions locales/en/settings.ftl
Original file line number Diff line number Diff line change
Expand Up @@ -14,8 +14,6 @@ settings-alert-preferences-allow-breach-alerts-title = Instant breach alerts
settings-alert-preferences-allow-breach-alerts-subtitle = These alerts are sent immediately once a data breach is detected
settings-alert-preferences-option-one = Send breach alerts to the affected email address
settings-alert-preferences-option-two = Send all breach alerts to the primary email address
settings-alert-preferences-allow-monthly-monitor-report-title = Monthly { -brand-monitor } report
settings-alert-preferences-allow-monthly-monitor-report-subtitle = A monthly update of new exposures, what’s been fixed, and what needs your attention.
## Monitored email addresses

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -185,12 +185,12 @@ export const AlertAddressForm = (props: Props) => {
<div>
<b>
{l10n.getString(
"settings-alert-preferences-allow-monthly-monitor-report-title",
"settings-alert-preferences-allow-monthly-monitor-plus-report-title",
)}
</b>
<p>
{l10n.getString(
"settings-alert-preferences-allow-monthly-monitor-report-subtitle",
"settings-alert-preferences-allow-monthly-monitor-plus-report-subtitle",
)}
</p>
</div>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -703,7 +703,7 @@ it("checks that monthly monitor report is enabled", () => {
);

const monthlyMonitorReportBtn = screen.getByLabelText(
"Monthly ⁨Monitor⁩ report",
"Monthly ⁨Monitor Plus⁩ report",
{ exact: false },
);
expect(monthlyMonitorReportBtn).toHaveAttribute("aria-checked", "true");
Expand Down Expand Up @@ -749,7 +749,7 @@ it("sends an API call to disable monthly monitor reports", async () => {
</TestComponentWrapper>,
);
const monthlyMonitorReportBtn = screen.getByLabelText(
"Monthly ⁨Monitor⁩ report",
"Monthly ⁨Monitor Plus⁩ report",
{ exact: false },
);

Expand Down

0 comments on commit d8f6ff4

Please sign in to comment.