Skip to content

Conversation

@tudoramariei
Copy link
Member

No description provided.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds Django SES admin integration so SES sending statistics can be viewed from the Django admin UI (and ensures the runtime has the required permissions/config).

Changes:

  • Extend ECS task IAM policy to allow reading SES account/stats info.
  • Conditionally enable django_ses (app + URLs) when SES email backend is active.
  • Update Unfold admin sidebar and i18n catalogs to surface the SES stats entry.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
terraform/iam.tf Grants ECS task role SES read permissions needed for stats/verified addresses.
backend/redirectioneaza/urls.py Conditionally mounts django_ses URLs under an admin path.
backend/redirectioneaza/settings/base.py Conditionally adds django_ses to INSTALLED_APPS when SES backend is used.
backend/redirectioneaza/settings/unfold.py Adds an “Advanced” sidebar section and conditionally inserts “Django SES Stats”.
backend/locale/ro/LC_MESSAGES/django.po Adds/adjusts translations for new admin/sidebar strings.
backend/locale/en/LC_MESSAGES/django.po Updates catalog references and adds new msgids for admin/sidebar strings.
backend/locale/hy/LC_MESSAGES/django.po Updates catalog references and adds new msgids for admin/sidebar strings.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@tudoramariei tudoramariei requested a review from andreiio January 20, 2026 11:56
@tudoramariei tudoramariei merged commit b4cf916 into main Jan 20, 2026
7 checks passed
@tudoramariei tudoramariei deleted the feat/admin-ses branch January 20, 2026 12:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants