-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Emails : nouveau modèle TemplateTransactional pour stocker nos configurations #1018
Emails : nouveau modèle TemplateTransactional pour stocker nos configurations #1018
Conversation
875273d
to
610a42f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tu peux merge les deux migration mais sinon LGTM 🎉
from django.db import migrations, models | ||
|
||
|
||
class Migration(migrations.Migration): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tu peux la supprimer et faire une seul migration ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bien vu ! 58fd1d4
72c24b6
to
a371b84
Compare
Quoi ?
Dans l'application
conversations
(réutilisée au lieu de créer une nouvelle app), nouveau modèleTemplateTransactional
L'idée est d'y mettre notre config d'e-mails, et permettre une migration en douceur de Mailjet vers Brevo (voir possibilité de revert, ou d'utiliser les 2 en parallèle si besoin !)
Cette PR contient seulement :