#35514: Dictionary based EMAIL_PROVIDERS settings
-----------------------------+--------------------------------------
     Reporter:  Jacob Rief   |                    Owner:  Jacob Rief
         Type:  New feature  |                   Status:  assigned
    Component:  Core (Mail)  |                  Version:  dev
     Severity:  Normal       |               Resolution:
     Keywords:               |             Triage Stage:  Accepted
    Has patch:  1            |      Needs documentation:  0
  Needs tests:  0            |  Patch needs improvement:  1
Easy pickings:  0            |                    UI/UX:  0
-----------------------------+--------------------------------------
Comment (by Mike Edmunds):

 Replying to [comment:28 Jacob Rief]:
 > Apologies for the long delay.

 No worries at all. The time was helpful to figure out some of the other
 design issues around the feature (ones that only became apparent thanks to
 your earlier iterations on the PR!).

 Natalia and I are trying to collect the design decisions in the form of a
 DEP: https://github.com/medmunds/deps/blob/email-providers/draft/email-
 providers.md and discussion at ​https://github.com/django/deps/pull/105.

 I tried to incorporate everything you and I learned from the PR, as well
 as Natalia's request in comment:22 that we replace
 `mail.get_connection(provider="notifications")` with
 `mail.providers["notifications"]`. We're also proposing deprecating
 `get_connection()` and the `connection` argument in general (which I think
 you had suggested in an early PR comment).

 > … If we can agree to just raise an exception, if both the old and new
 configuration options are used together, then this ticket should be
 fixable easily. …

 I think we're all in agreement! Users can pick old configuration, or new
 configuration, but not both at the same time. I've tried to document the
 specific disallowed cases in the DEP: look for the boldface callouts under
 [https://github.com/medmunds/deps/blob/email-providers/draft/email-
 providers.md#deprecated-email-settings Deprecated email settings].

 (Many of my comments in the PR are now outdated, superseded by decisions
 in the DEP.)
-- 
Ticket URL: <https://code.djangoproject.com/ticket/35514#comment:29>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To view this discussion visit 
https://groups.google.com/d/msgid/django-updates/0107019c6daf5d5f-f48085fa-6a74-4fd9-801a-6cac6f725842-000000%40eu-central-1.amazonses.com.

Reply via email to