-
Notifications
You must be signed in to change notification settings - Fork 51
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
[FEATURE] Supprimer la colonne "grainId" de la table "element-answers" (PIX-12459) #8935
[FEATURE] Supprimer la colonne "grainId" de la table "element-answers" (PIX-12459) #8935
Conversation
Une fois les applications déployées, elles seront accessibles via les liens suivants :
Les variables d'environnement seront accessibles via les liens suivants : |
2df6e31
to
2795683
Compare
ça ne marchera pas 👉 Dans le fichier de migration > ici <, lors du scénario de test proposé, on fini avec une ou plusieurs lignes dans la table, et le fait que la colonne ne soit pas nullable génère une erreur SQL lors du rollback.
Réponse 1 heure plus tard: Une autre stratégie aurait été de gérer la rétro-compatibilité en deux PR distinctes:
Puis une version plus tard:
|
2795683
to
cf6bf54
Compare
Je suis vraiment pas convaincu de supprimer cette colonne, comme je n'étais pas là lors des points qui ont fait émergé ce ticket, je ne pense pas avoir tout le contexte. |
Pour le contexte : le métier ne souhaite pas réutiliser les éléments, ni les grains c'est donc OK de supprimer la colonne |
cf6bf54
to
d1f0ce6
Compare
Co-authored-by: Rébecca Kaci <82950611+reibecca@users.noreply.github.com>
don't save grainId in element-answers table anymore Co-authored-by: miranda.lin-guignard <miranda.w.lin@gmail.com>
d1f0ce6
to
bd8c4de
Compare
🦄 Problème
Cette information n'est pas utilisée par l'équipe métier, nous n'avons donc pas le besoin de l'enregistrer.
🤖 Proposition
Faire une migration pour supprimer la colonne
grainId
de la tableelement-anwers
de la BDD.🌈 Remarques
On a choisi de mettre la colonne à
nullable
dans le down de la migration, sinon le rollback ne fonctionne pas dans le cas où il y a déjà des données dans la BDD.💯 Pour tester
En local :
api
lancer la commandenpm run db:migrate
grainId
de la tableelement-anwers
n'existe plusdidacticiel-modulix
Vérifiez
il n'y a pas d'erreurnpm run db:rollback:latest
grainId
de la tableelement-anwers
est à nouveau présenteSur la RA :
didacticiel-modulix
Vérifiez
il n'y a pas d'erreur