-
Notifications
You must be signed in to change notification settings - Fork 4
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
feat: Dimensions flat #36
Conversation
on devrait pas mettre jusqu'a 10 dimensions dans les inserts ? (vu qu'il y a 10 colonnes de prêtes) |
See: #38 |
test: 10 dimensions
Hello, |
je propose qu'on garde le comportement actuel avec le champ |
je vais tenter un script de migration et si ça se complique, ça me va qu'on laisse la colonne |
Ok donc @lionelB tu reviens dans ce ticket pour nous dire si on peut la merge ? Ou alors on merge direct (cc @revolunet) et on effacera la colonne ensuite dans une deuxième PR si tu arrive à faire le script ? EDIT: En fait oui, je pense qu'on peut faire en deux temps comme ça on avance sur le sujet :
Ça vous va ? @lionelB Si tu n'as pas le temps je peux essayer de faire le script de migration (je découvre ce projet, mais j'imagine que ça devrait aller !) et vous faire une PR ? |
est-ce qu'on peut pas garder l'insert dans comme ca on a juste une feature en plus : les dimensions flat. |
je suis d'accord avec @revolunet et on pourra faire une pr pour le script de migration avec la suppression des ancienne colonnes |
Parfait, merci faisons comme ça. @revolunet j'ai fait la modif (via une PR comme j'ai pas les droits) pour garder les deux fonctionnement et @goffle vient de la merge. Si c'est bon tu peux merge @revolunet. Side question : @lionelB tu veux faire le script ou je le fais alors ? |
🎉 This PR is included in version 1.3.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
No description provided.