It's been a while since we’ve last seen you, but we wanted to thank you for contributing to localization efforts at - {{ full_url('pontoon.homepage') }}. + {homepage_url}.
@@ -15,18 +13,16 @@
As part of our efforts to keep our localization community active and engaged, we periodically review account activity. We’ve noticed you haven’t signed in - to your Pontoon account in the past {{ settings.INACTIVE_CONTRIBUTOR_PERIOD }} - months. While we completely understand that circumstances change, we wanted to - inform you that we may deactivate dormant accounts in the future to ensure the + to your Pontoon account in the past {INACTIVE_CONTRIBUTOR_PERIOD} months. + While we completely understand that circumstances change, we wanted to inform + you that we may deactivate dormant accounts in the future to ensure the security and efficiency of our platform.
If you’d like to keep your Pontoon account active (we hope you will!) please sign in to Pontoon by visiting - {{ full_url('pontoon.homepage') }} + {homepage_url} and click the “Sign in” button in the top right corner.
diff --git a/pontoon/messaging/templates/messaging/emails/content/inactive_manager.html b/pontoon/messaging/templates/messaging/emails/content/inactive_manager.html index b0b4767fac..9f4950cf69 100644 --- a/pontoon/messaging/templates/messaging/emails/content/inactive_manager.html +++ b/pontoon/messaging/templates/messaging/emails/content/inactive_manager.html @@ -10,10 +10,10 @@As part of our efforts to keep our localization community active and engaged, we periodically review account activity. We’ve noticed that in the past - {{ settings.INACTIVE_MANAGER_PERIOD }} months you have not submitted any - translations or reviewed any translation submissions for your locale. We - understand that life can get busy, and finding time to contribute to projects - isn't always easy. We get it. + {INACTIVE_MANAGER_PERIOD} months you have not submitted any translations or + reviewed any translation submissions for your locale. We understand that life + can get busy, and finding time to contribute to projects isn't always easy. We + get it.
@@ -40,18 +40,14 @@
In addition, another Team Manager responsibility is to evaluate candidates for advanced roles within their community. Please review the - contributors page + contributors page for your team and see if there are any motivated and active community members that may be interested in taking on additional responsibilities within your team.
- {{ full_url('pontoon.teams.team', locale.code) }} + {team_url}
diff --git a/pontoon/messaging/templates/messaging/emails/content/inactive_translator.html b/pontoon/messaging/templates/messaging/emails/content/inactive_translator.html index 2e5849a4f5..f8f38c9bb4 100644 --- a/pontoon/messaging/templates/messaging/emails/content/inactive_translator.html +++ b/pontoon/messaging/templates/messaging/emails/content/inactive_translator.html @@ -9,10 +9,10 @@
As part of our efforts to keep our localization community active and engaged, we periodically review account activity. We’ve noticed that in the past - {{ settings.INACTIVE_TRANSLATOR_PERIOD }} months you have not submitted any - translations or reviewed any translation submissions for your locale. We - understand that life can get busy, and finding time to contribute to projects - isn't always easy. We get it. + {INACTIVE_TRANSLATOR_PERIOD} months you have not submitted any translations or + reviewed any translation submissions for your locale. We understand that life + can get busy, and finding time to contribute to projects isn't always easy. We + get it.
@@ -37,9 +37,7 @@
- {{ full_url('pontoon.teams.team', locale.code) }} + {team_url}
diff --git a/pontoon/messaging/templates/messaging/emails/content/onboarding_1.html b/pontoon/messaging/templates/messaging/emails/content/onboarding_1.html index 957848c47d..f27cc168b4 100644 --- a/pontoon/messaging/templates/messaging/emails/content/onboarding_1.html +++ b/pontoon/messaging/templates/messaging/emails/content/onboarding_1.html @@ -10,22 +10,19 @@
Thank you again for joining translation community on Pontoon. As always, you can continue contributing translations at - {{ full_url('pontoon.homepage') }}. + {homepage_url}.
@@ -39,15 +37,14 @@
Each localization community has its own specific workflows, so do reach out to - your Team Managers to learn - more. + your Team Managers to learn more.
For in-depth documentation on Pontoon, see our - Documentation for localizers. + Documentation for localizers.
Finally, if you’re interested in the latest updates, announcements about new Pontoon features and more, be sure to enable “News and updates“ in your - settings so you - can be the first to get notified. + settings so you can be the first to get notified.
diff --git a/pontoon/messaging/tests/test_emails.py b/pontoon/messaging/tests/test_emails.py index 28828355da..3d150da502 100644 --- a/pontoon/messaging/tests/test_emails.py +++ b/pontoon/messaging/tests/test_emails.py @@ -1,15 +1,27 @@ +from collections import defaultdict from datetime import date, datetime, timezone from unittest.mock import patch import pytest +from django.core import mail +from django.template import TemplateSyntaxError from django.test.client import RequestFactory +from django.urls import NoReverseMatch +from pontoon.base.models import User from pontoon.insights.models import LocaleInsightsSnapshot from pontoon.messaging.emails import ( _get_monthly_locale_stats, + send_inactive_contributor_emails, + send_inactive_manager_emails, + send_inactive_translator_emails, + send_onboarding_email_1, + send_onboarding_emails_2, + send_onboarding_emails_3, send_verification_email, ) +from pontoon.messaging.models import EmailContent from pontoon.test.factories import LocaleFactory @@ -60,3 +72,123 @@ def test_get_monthly_locale_stats_uses_end_of_month_snapshot(): assert result[locale.pk].pk == snapshot_nov_1.pk assert result[locale.pk].approved_strings == 100 assert result[locale.pk].completion == 100.0 + + +@pytest.mark.django_db +def test_send_onboarding_email_1(user_a): + try: + send_onboarding_email_1(user_a) + except EmailContent.DoesNotExist: + pytest.fail("EmailContent for 'onboarding_1' is missing from the DB.") + except NoReverseMatch as e: + pytest.fail(f"URL resolution failed: check URL config: {e}") + except TemplateSyntaxError as e: + pytest.fail(f"Template is broken: {e}") + except Exception as e: + pytest.fail(f"An unexpected error occurred: {e}") + + assert len(mail.outbox) == 2 + assert mail.outbox[0].to == [user_a.contact_email] + + +@pytest.mark.django_db +def test_send_onboarding_emails_2(user_a): + + users = User.objects.filter(pk=user_a.pk) + + try: + send_onboarding_emails_2(users) + except EmailContent.DoesNotExist: + pytest.fail("EmailContent for 'onboarding_2' is missing from the DB.") + except NoReverseMatch as e: + pytest.fail(f"URL resolution failed: check URL config: {e}") + except TemplateSyntaxError as e: + pytest.fail(f"Template is broken: {e}") + except Exception as e: + pytest.fail(f"An unexpected error occurred: {e}") + + assert len(mail.outbox) == 2 + assert mail.outbox[0].to == [user_a.contact_email] + + +@pytest.mark.django_db +def test_send_onboarding_emails_3(user_a): + + users = User.objects.filter(pk=user_a.pk) + + try: + send_onboarding_emails_3(users) + except EmailContent.DoesNotExist: + pytest.fail("EmailContent for 'onboarding_3' is missing from the DB.") + except NoReverseMatch as e: + pytest.fail(f"URL resolution failed: check URL config: {e}") + except TemplateSyntaxError as e: + pytest.fail(f"Template is broken: {e}") + except Exception as e: + pytest.fail(f"An unexpected error occurred: {e}") + + assert len(mail.outbox) == 2 + assert mail.outbox[0].to == [user_a.contact_email] + + +@pytest.mark.django_db +def test_send_inactive_contributor_emails(user_a): + + users = User.objects.filter(pk=user_a.pk) + + try: + send_inactive_contributor_emails(users) + except EmailContent.DoesNotExist: + pytest.fail("EmailContent for 'inactive_contributor' is missing from the DB.") + except NoReverseMatch as e: + pytest.fail(f"URL resolution failed: check URL config: {e}") + except TemplateSyntaxError as e: + pytest.fail(f"Template is broken: {e}") + except Exception as e: + pytest.fail(f"An unexpected error occurred: {e}") + + assert len(mail.outbox) == 2 + assert mail.outbox[0].to == [user_a.contact_email] + + +@pytest.mark.django_db +def test_send_inactive_translator_emails(user_a, locale_a): + translators = defaultdict(set) + + users = User.objects.filter(pk=user_a.pk) + translators[user_a.pk].add(locale_a) + try: + send_inactive_translator_emails(users, translators) + except EmailContent.DoesNotExist: + pytest.fail("EmailContent for 'inactive_translator' is missing from the DB.") + except NoReverseMatch as e: + pytest.fail(f"URL resolution failed: check URL config: {e}") + except TemplateSyntaxError as e: + pytest.fail(f"Template is broken: {e}") + except Exception as e: + pytest.fail(f"An unexpected error occurred: {e}") + + assert len(mail.outbox) == 2 + assert mail.outbox[0].to == [user_a.contact_email] + + +@pytest.mark.django_db +def test_send_inactive_manager_emails(user_a, locale_a): + managers = defaultdict(set) + + users = User.objects.filter(pk=user_a.pk) + managers[user_a.pk].add(locale_a) + + try: + send_inactive_manager_emails(users, managers) + except EmailContent.DoesNotExist: + pytest.fail("EmailContent for 'inactive_manager' is missing from the DB.") + except NoReverseMatch as e: + pytest.fail(f"URL resolution failed: check URL config: {e}") + except TemplateSyntaxError as e: + pytest.fail(f"Template is broken: {e}") + except Exception as e: + pytest.fail(f"An unexpected error occurred: {e}") + + assert len(mail.outbox) == 2 + assert mail.outbox[0].to == [user_a.contact_email]