Changer le type de valeur du champ LAST_DPT_DIAGNOSTIC de entier à string

MTES-MCT / sparte

MIT License

6 stars 1 forks source link

Changer le type de valeur du champ LAST_DPT_DIAGNOSTIC de entier à string #626

Open alexisig opened 1 week ago

alexisig commented 1 week ago

Le champs est utilisé pour conserver l'identifiant du département du dernier diagnostic téléchargé par un utilisateur sur brevo.

Problème : la valeur du code département est un string car elle peut contenir des lettres (cas de la corse), et pour les départements dont le code est inférieur à 10, pour lesquels un 0 est toujours présent au début. Dans le cas où le diagnostic est fait sur plusieurs départements, un string de ce format est retourné : "35,56"

Solution : contacter le support brevo pour migrer le type de ce champs d'entier à string.

alexisig commented 1 week ago

lié à #589

JeremieSeguin commented 1 week ago

@alexisig : il y a deux attributs de contact qu'il faudrait que je transforme en STRING

LAST_DPT_DIAGNOSTIC (int) et A_TELECHARGE_RAPPORT_TRIENNAL (booleen)

J'imaginais supprimer les deux attributs, et les re créer tous les deux en type string. Ensuite, j'imaginais mettre à jour les valeurs pour chaque contact à partir de METABASE

Penses-tu que ça soit ok de procéder comme ça avec le code de la plateforme ?

alexisig commented 5 days ago

@JeremieSeguin je pense que c'est pas quelque chose qui peut être fait facilement en autonomie de votre côté (les requêtes à faire sur metabase seraient complexes), et on va pas avoir le temps côté dev pour le faire.

est-ce que tu aurais une solution plus low-tech ?

JeremieSeguin commented 5 days ago

@JeremieSeguin je pense que c'est pas quelque chose qui peut être fait facilement en autonomie de votre côté (les requêtes à faire sur metabase seraient complexes), et on va pas avoir le temps côté dev pour le faire.

est-ce que tu aurais une solution plus low-tech ?

Peut-être via cette US ?
Autrement hormis via METABASE je n'ai pas d'idée pour récupérer les données en base et les déposer sur Brevo.

Pour info :

J'ai migré le type de LAST_DPT_DIAGNOSTIC en string avec les valeurs connues de l'ancien attribut LAST_DPT_DIAGNOSTIC (qui était un nombre). => Le champs est toujours vide pour les utilisateurs de Corse et ceux dont l'utilisation de l'app date d'avant la mise en place de cet attribut.
J'ai migré le type de A_TELECHARGE_RAPPORT_TRIENNAL en texte. Il est vide pour tous les utilisateurs.

Pourquoi cette US ? Aujourd'hui l'OCS GE a été intégrée pour la Corse du Sud. Je ne peux pas tenir informé nos utilisateurs/utilisatrices.

alexisig commented 5 days ago

@JeremieSeguin Top que tu aies pu corriger LAST_DPT_DIAGNOSTIC directement depuis brevo !

Pour le dump, c'est un effort important pour une seule occasion.

Comme on a maintenant livré tous les départements pour lesquels le bug existait, peut-être c'est ok de se dire qu'on met en place cette notification qu'à partir des prochains départements ?

JeremieSeguin commented 5 days ago

Yes, pourquoi pas. On peut fermer cette US en tout cas. Mais laisser celle-ci en priorité basse.