Skip to content

Fixed #37261 -- Documented error reporting migration to MAILERS. - #21934

Merged
nessita merged 1 commit into
django:mainfrom
medmunds:37261-docs-email-migration-clarification
Sep 15, 2026
Merged

nessita merged 1 commit into
django:mainfrom
medmunds:37261-docs-email-migration-clarification

Conversation

@medmunds

@medmunds medmunds commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Trac ticket number

ticket-37261

Branch description

Documented specifics for migrating AdminEmailHandler and BrokenLinkEmailsMiddleware to MAILERS when relying on fail_silently with an unreliable email backend connection.

AI Assistance Disclosure (REQUIRED)

  • No AI tools were used in preparing this PR.
  • If AI tools were used, I have disclosed which ones, and fully reviewed and verified their output.

Checklist

  • This PR follows the contribution guidelines.
  • This PR does not disclose a security vulnerability (see vulnerability reporting).
  • This PR targets the main branch.
  • The commit message is written in past tense, mentions the ticket number (if applicable), and ends with a period (see guidelines).
  • I have not requested, and will not request, an automated AI review for this PR.
  • I have checked the "Has patch" ticket flag in the Trac system.
  • I have added or updated relevant tests.
  • I have added or updated relevant docs, including release notes if applicable.
  • I have attached screenshots in both light and dark modes for any UI changes.

@medmunds

medmunds commented Sep 8, 2026

Copy link
Copy Markdown
Contributor Author

I couldn't come up with anything useful to add (retroactively) to the 6.1 release notes, but could be convinced otherwise if someone has a suggestion.

@medmunds
medmunds force-pushed the 37261-docs-email-migration-clarification branch from 737dfd7 to 09c4427 Compare September 8, 2026 19:15
@nessita
nessita self-requested a review September 15, 2026 01:39

@nessita nessita left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks great, I added two optional editorial suggestions and one must-fix. Thanks! 🌟

Comment thread docs/howto/mailers-migration.txt Outdated
Comment thread docs/howto/mailers-migration.txt Outdated
Comment thread docs/howto/mailers-migration.txt
@medmunds
medmunds force-pushed the 37261-docs-email-migration-clarification branch from 09c4427 to 2af2230 Compare September 15, 2026 16:37
Documented specifics for migrating AdminEmailHandler and
BrokenLinkEmailsMiddleware to MAILERS when relying on fail_silently with
an unreliable email backend connection.
@medmunds
medmunds force-pushed the 37261-docs-email-migration-clarification branch from 2af2230 to f9d444a Compare September 15, 2026 16:46
@medmunds

Copy link
Copy Markdown
Contributor Author

@nessita thanks, adopted all your suggestions

@nessita nessita left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Amazing, thank you!

@nessita
nessita merged commit bbf5a99 into django:main Sep 15, 2026
27 of 29 checks passed
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.

2 participants