From 57dc1f258f33b9ed418883b252a4b9611b56e0a3 Mon Sep 17 00:00:00 2001 From: dbxnr Date: Wed, 3 Mar 2021 10:07:45 +0000 Subject: [PATCH] Make it clear reminder email does not relate to new issue (#16661) * Make it clear rejection emailis a reminder * black formatting --- .../send_pending_rejection_last_warning_notifications.py | 4 +++- src/olympia/reviewers/tests/test_commands.py | 2 +- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/src/olympia/reviewers/management/commands/send_pending_rejection_last_warning_notifications.py b/src/olympia/reviewers/management/commands/send_pending_rejection_last_warning_notifications.py index 2e6c2052ab..843b4a4885 100644 --- a/src/olympia/reviewers/management/commands/send_pending_rejection_last_warning_notifications.py +++ b/src/olympia/reviewers/management/commands/send_pending_rejection_last_warning_notifications.py @@ -95,7 +95,9 @@ class Command(BaseCommand): 'delayed_rejection_days': self.EXPIRING_PERIOD_DAYS, } template = 'reject_multiple_versions_with_delay' - subject = 'Mozilla Add-ons: %s%s will be disabled on addons.mozilla.org' + subject = ( + 'Reminder - Mozilla Add-ons: %s%s will be disabled on addons.mozilla.org' + ) # This re-sends the notification sent when the versions were scheduled # for rejection, but with the new delay in the body of the email now # that the notification is about to expire. diff --git a/src/olympia/reviewers/tests/test_commands.py b/src/olympia/reviewers/tests/test_commands.py index 9c4256aac1..0dc6ee8f30 100644 --- a/src/olympia/reviewers/tests/test_commands.py +++ b/src/olympia/reviewers/tests/test_commands.py @@ -861,7 +861,7 @@ class TestSendPendingRejectionLastWarningNotification(TestCase): assert addon.reviewerflags.notified_about_expiring_delayed_rejections message = mail.outbox[0] assert message.subject == ( - 'Mozilla Add-ons: %s will be disabled on addons.mozilla.org' + 'Reminder - Mozilla Add-ons: %s will be disabled on addons.mozilla.org' % str(addon.name) ) assert message.to == [author.email]