Update canteen.py > class Ministries > Labels Open data #4812
Conversation
J'ai mis à jour les labels de l'open data avec les changement de ministre Cela correspond à ce ticket Picardie https://www.notion.so/fabe4c2eba6248129d7780b10c8c0df6?v=7292cd681d954696a18da80b73156671&p=1454a08edabc80c5b85ce9b781b538f1&pm=s
|
Hello @raphodn @qloridant @Charline-L j'ai mis à jour les labels des ministère pour l'open data. Pouvez-vous regarder ma PR. Merci à vous 🎄 |
| AGRICULTURE = "agriculture", "Agriculture, Alimentation et Forêts" | ||
| ARMEE = "armee", "Armées" | ||
| TERRITOIRES = "territoires", "Cohésion des territoires - Relations avec les collectivités territoriales" | ||
| AGRICULTURE = "agriculture_alimentation", "Agriculture, Alimentation et Forêts" |
There was a problem hiding this comment.
là ca ne change pas les labels, mais les clés, celles utilisées/stockées dans la base de données
du coup ca va demander quelques changements supplémentaires
There was a problem hiding this comment.
est-ce que c'est amené à changer souvent ?
est-ce que ca concerne seulement l'open data ?
une option est de laisser tel quel (sans changement à la BDD), mais venir modifier/remplacer les les valeurs au moment des exports.
There was a problem hiding this comment.
mais ca ne me semble pas judicieux de faire ces changements au moment de l'export, car je préférerais que des données exportées puissent être réimportées sans modification. donc à moins de changer aussi l'import...
|
Maintenant que nous avons mis au carré les ministères de manière générique,
ça ne devrait pas changer.
Ces ajustements sont surtout pour les utilisateurs non tech de l'open data
qui nous aide au déploiement sur le territoire afin qu'ils puissent être
autonome sur les filtres.
Pas besoin de se compliqué la vie sur ce sujet à mon avis :)
Le ven. 20 déc. 2024 à 17:19, Raphaël Odini ***@***.***> a
écrit :
… ***@***.**** commented on this pull request.
------------------------------
In data/models/canteen.py
<#4812 (comment)>:
> @@ -123,29 +123,29 @@ class Actions(models.TextChoices):
class Ministries(models.TextChoices):
AFFAIRES_ETRANGERES = "affaires_etrangeres", "Affaires étrangères"
- AGRICULTURE = "agriculture", "Agriculture, Alimentation et Forêts"
- ARMEE = "armee", "Armées"
- TERRITOIRES = "territoires", "Cohésion des territoires - Relations avec les collectivités territoriales"
+ AGRICULTURE = "agriculture_alimentation", "Agriculture, Alimentation et Forêts"
mais ca ne me semble pas judicieux de faire ces changements au moment de
l'export, car je préférerais que des données exportées puissent être
réimportées sans modification. donc à moins de changer aussi l'import...
—
Reply to this email directly, view it on GitHub
<#4812 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BCVAVCG6KUWCTGOLS3I57O32GQ7RRAVCNFSM6AAAAABT7DLRCKVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDKMJXHA4DMMBWGY>
.
You are receiving this because you were assigned.Message ID:
***@***.***>
|
|
@Tdauvet89 |
|
Hello @qloridant , oui je suis d'accord de l'appliquer à tous. C'est pour permettre de filtrer correctement tant sur l'open data, que sur metabase, ou encore sur un fichier CSV que l'on aurait téléchargé depuis l'open data |
J'ai mis à jour les labels de l'open data avec les changement de ministre
Cela correspond à ce ticket Picardie
https://www.notion.so/fabe4c2eba6248129d7780b10c8c0df6?v=7292cd681d954696a18da80b73156671&p=1454a08edabc80c5b85ce9b781b538f1&pm=s