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

Open

Mindiell wants to merge 1 commit from Mindiell/PonyConf:fix_2 into main

Mindiell commented

2023-02-18 16:39:14 +01:00

Contributor

Reprise de l'adresse du serveur, tout simplement.

Mindiell added 1 commit

2023-02-18 16:39:15 +01:00

fix: default FROM email set 136919747c

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

mdk commented

2023-03-16 10:53:49 +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.

mdk commented

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

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

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.

View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u fix_2:Mindiell-fix_2

git switch Mindiell-fix_2

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch main

git merge --no-ff Mindiell-fix_2

git switch Mindiell-fix_2

git rebase main

git switch main

git merge --ff-only Mindiell-fix_2

git switch Mindiell-fix_2

git rebase main

git switch main

git merge --no-ff Mindiell-fix_2

git switch main

git merge --squash Mindiell-fix_2