Skip to content

Fix requirements prod - #54

Open
lvillanne-recia wants to merge 4 commits into
Noethys:mainfrom
GIP-RECIA:fix_requirements_prod
Open

lvillanne-recia wants to merge 4 commits into
Noethys:mainfrom
GIP-RECIA:fix_requirements_prod

Conversation

@lvillanne-recia

Copy link
Copy Markdown
Contributor

Bonjour,

Suite à la PR 46 fix: correction du système d'installation des dépendances cette nouvelle PR vient ajouter un système conditionnelle dans le code afin de ne charger le module debug_toolbar seulement quand c'est nécessaire, tout en le chargeant au bon endroit dans le code pour ne pas casser le fonctionnement en debug.

Actuellement l'installation par le requirement de prod ne fonctionne pas car le module n'est pas installé mais il est encore utilisé dans le code.
Cette nouvelle PR vient donc réparer cet oubli afin de rendre l'installation du module debug_toolbar inutile en production.

@Noethys

Noethys commented May 8, 2026

Copy link
Copy Markdown
Owner

Bonjour,

Autant ce code est judicieux, autant il me pose un petit souci : Il ne tient compte que du DEBUG du SETTINGS. Mais il ne tient pas compte du DEBUG du SETTINGS_PRODUCTION. Alors que c'est celui-ci que j'utilise en développement pour ne pas avoir à modifier les SETTINGS. Y-a-t'il moyen de modifier cela ?

@lvillanne-recia

Copy link
Copy Markdown
Contributor Author

Bonjour,

Effectivement j'avais loupé le fait que le DEBUG utilisé était celui de settings.py dans mon code.
J'ai réordonné le code, c'est maintenant bien le DEBUG de settings_production.py qui est utilisé, s'il existe, pour voir si la debug_toolbar doit être incluse ou non. Le if sur la variable DEBUG se fait maintenant après l'inclusion du settings_production.py.
J'ai aussi respecté l'ordre d'inclusion des différents middleware et je ne note pas de régression de mon côté.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants