Fix #2716: Inversion possible du signe des indicateurs d'états de gestion
Changements apportés
- Style UX
-
Suit le guide de style écrit par Sophie.
-
- Review
-
La MR est assignée à une personne pertinente pour en faire une review
-
- Git
-
Rebasé sur master -
Historique des commits nettoyé (rebase interactif) -
Messages de commit mentionnant l'issue (Ref #XXX : ou Fix #XXX :)
-
- Tests
-
Tests "verts" en local (pytest) -
Validé par la CI
-
- Code propre
-
PEP8 flake8 -
Conventions de nommage des variables, des fonctions et des classes respectées -
Typo dans les variables, des fonctions et des classes -
Typo dans les chaînes de caractères -
Code commenté -
Commentaires morts (inutile, tests) nettoyés
-
- Base de données
-
Appliqué toutes les migrations présentes amont -
Pas d'import de modèles dans les migrations -
Créer une migration de merge si nécessaire -
endi_anonymize.py mis à jour si besoin (ajout/modif de modèle). -
j'évite d'utiliser backref=
au profit deback_populates=
dans lesrelationship
-
Points de vigilance / Retours souhaités
- Cohérence des données après la mise à jour (celery en 6.4.0)
- Les signes des états de trésorerie et états de gestion sont les bons après la mise à jour
- Les totaux intègrent bien la valeur de la mesure suivant le changement de convention
- L'idéal serait de tester avec un "vrai" jeu de donnée sur un "vrai" serveur, ou à défaut de bien vérifier que tout s'est bien passé lors du déploiement chez les béta-testeurs
Navigateurs testés
Edited by Nicolas Ménégale