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

Settings: Permettre de désactiver la connexion ProConnect en local #5735

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tonial
Copy link
Contributor

@tonial tonial commented Mar 8, 2025

🤔 Pourquoi ?

C'est un peu dommage car les settings de dev importent ceux de test.
Du coup il n'est pas possible de ficher une valeur dans test en ignorant les variables d'environnement sans devoir remettre la valeur de bas.py dans dev.py...

Qui est pour inverser la logique et avoir dev qui import base, et test qui import dev ?

🍰 Comment ?

Décrivez en quelques mots la solution retenue et mise en oeuvre, les difficultés ou problèmes rencontrés. Attirez l'attention sur les décisions d'architecture ou de conception importantes.

🚨 À vérifier

  • Mettre à jour le CHANGELOG_breaking_changes.md ?
  • Ajouter l'étiquette « Bug » ?

🏝️ Comment tester ?

Les instructions pour reproduire le problème, les profils de test, le parcours spécifique à utiliser, etc. Si vous disposez d'une recette jetable, mettre l'URL pour tester dans cette partie.

💻 Captures d'écran

@tonial tonial added the no-changelog Ne doit pas figurer dans le journal des changements. label Mar 8, 2025
@tonial tonial self-assigned this Mar 8, 2025
@francoisfreitag
Copy link
Contributor

Qui est pour inverser la logique et avoir dev qui import base, et test qui import dev ?

Plutôt contre, l’environnement de test est plus léger que l’environnement de dev, et devoir configurer par ex djdt en test, alors qu’on ne veut pas du tout avoir djdt en test (pour rester au plus proche de la prod).

Après, on pourrait imaginer avoir test qui importe base, et dev qui importe base, sans que dev ne passe par test. Ça fait un peu de répétition, mais ça évite de faire et défaire et refaire.

@tonial
Copy link
Contributor Author

tonial commented Mar 10, 2025

J'ai regardé et il n'y a que 2 settings qu'on a besoin de surcharger dans test puis dans dev, donc ce n'est pas très pertinent de tout dupliquer.
Je vais juste merger cette PR et laisser le reste.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-changelog Ne doit pas figurer dans le journal des changements.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants