feat: add number (not integer) management#599
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #599 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 1 1
Lines 4 4
=========================================
Hits 4 4
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
| "boolean": Boolean, | ||
| "integer": Integer, | ||
| "integer[]": ARRAY(Integer), | ||
| "number": Float(precision=24), |
There was a problem hiding this comment.
Salut @edelclaux, en essayant de tester tes évolutions je me suis posé une question :
de la même manière qu'il y a les types integer et integer[], pour number il ne faut pas également ajouter number[]: ARRAY(Float(precision=24)) ?
There was a problem hiding this comment.
effectivement, il doit bien y avoir des cas de valeurs multiples de float. Donc oui, surement.
|
J'ai testé ta PR, le rapport d'import m'affiche toujours les mêmes erreurs que je mentionne ici PnX-SI/GeoNature#4069 |
|
Ne fonctionne pas car la table transitoire n'est pas impactée par le changement. |
| if type_widget == "number": | ||
| return "integer" | ||
| return "number" | ||
|
|
There was a problem hiding this comment.
manque la conversion du type de widget integer ce qui ne met pas à jour la table transitoire
if type_widget == "integer":
return "integer"Ceci dit cela pose la question de l'éparpillement de la définition type de widget => type de donnée
|
Par contre cela pose un autre soucis, le type_widget integer n'existe pas dans les GenericForm. C'est un champ qui ne peut s'afficher sur le formulaire |
4 participants
Closes #598