fix: default FROM email set. closes #2 #19

Closed
Mindiell wants to merge 1 commit from (deleted):fix_2 into main
Contributor

Reprise de l'adresse du serveur, tout simplement.

Reprise de l'adresse du serveur, tout simplement.
Mindiell changed title from fix: default FROM email set to fix: default FROM email set. closes #2 2023-02-18 16:39:45 +01:00
Owner

Je pense que ça ne fonctionnera pas. Enfin si mais qu'avec example.com en dev.

Car l'utilisation des settings c'est de faire un fichier settings_prod.py qui ressemble à :

from ponyconf.settings import *

SERVER_EMAIL = 'ponyconf@afpy.org'

Dans ce cas, DEFAULT_FROM_EMAIL reste inchangé, il pointe toujours sur ponyconf@example.com.

Donc c'est plutôt côté mise en prod qu'il faut le réparer, chacun de son côté, via qqch comme :

from ponyconf.settings import *

DEFAULT_FROM_EMAIL = 'noreply@afpy.org'
SERVER_EMAIL = 'ponyconf@afpy.org'

Ça c'est ce qu'on a en prod depuis le 21 octobre 2022, donc ça pré-date l'issue #2.

Je pense que ça ne fonctionnera pas. Enfin si mais qu'avec `example.com` en dev. Car l'utilisation des settings c'est de faire un fichier `settings_prod.py` qui ressemble à : ```python from ponyconf.settings import * SERVER_EMAIL = 'ponyconf@afpy.org' ``` Dans ce cas, `DEFAULT_FROM_EMAIL` reste inchangé, il pointe toujours sur `ponyconf@example.com`. Donc c'est plutôt côté mise en prod qu'il faut le réparer, chacun de son côté, via qqch comme : ```python from ponyconf.settings import * DEFAULT_FROM_EMAIL = 'noreply@afpy.org' SERVER_EMAIL = 'ponyconf@afpy.org' ``` Ça c'est ce qu'on a en prod depuis le 21 octobre 2022, donc ça pré-date l'issue #2.
Owner

Hum je pense que #2 dit qu'au lieu d'utiliser l'email configuré dans l'interface pour l'instance, accessible via conference.from_email(), le mail serait envoyé avec comme expéditeur l'email dans settings.py.

Hum je pense que #2 dit qu'au lieu d'utiliser l'email configuré dans l'interface pour l'instance, accessible via `conference.from_email()`, le mail serait envoyé avec comme expéditeur l'email dans `settings.py`.
Mindiell closed this pull request 2026-09-04 08:21:45 +02:00

Pull request closed

Sign in to join this conversation.
No description provided.