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

Mindiell · 2023-02-18T15:39:14Z

Mindiell commented

2023-02-18 15:39:14 +00:00

Reprise de l'adresse du serveur, tout simplement.

Mindiell added 1 commit 2023-02-18 15:39:15 +00:00

136919747c fix: default FROM email set

Mindiell changed title from ~~fix: default FROM email set~~ to fix: default FROM email set. closes #2

2023-02-18 15:39:45 +00:00

mdk commented

2023-03-16 09:53:49 +00:00

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.

mdk commented

2023-03-18 14:25:14 +00:00

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`.

This pull request can be merged automatically.

This branch is out-of-date with the base branch

You are not authorized to merge this pull request.

You can also view command line instructions.