Skip to content
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

refactor: isoler les settings de test #864

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

vincentporte
Copy link
Contributor

@vincentporte vincentporte commented Dec 19, 2024

Description

🎸 Isoler les settings de test, pour nettoyer les valeurs par défaut des settings de base et de dev.
🎸 Utiliser ces settings de test pour rendre les tests executables dès le tirage du dépôt.
🎸 Utiliser ces settings dans la CI pour harmoniser l'execution des tests en local et dans les actions github

Type de changement

🚧 technique

Points d'attention

🦺 refactor de l'appel aux settings liés aux envois de mails
🦺 simplification de la gestion des routes brevo
🦺 harmonisation des settings utilisés pour envoyer le lien magique par mail ou dans les message django
🦺 prise en compte des noms de bucket issus des settings dans les fichiers de migration

📦 prerequis à la PR #857

@vincentporte vincentporte added python Pull requests that update Python code technical debt labels Dec 19, 2024
@vincentporte vincentporte force-pushed the vp/isolate_test_conf branch 3 times, most recently from 990b218 to a04191f Compare December 19, 2024 15:03
@vincentporte vincentporte marked this pull request as ready for review December 19, 2024 15:10
@vincentporte vincentporte marked this pull request as draft December 19, 2024 16:03
@vincentporte vincentporte marked this pull request as ready for review December 19, 2024 16:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
python Pull requests that update Python code technical debt
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant