Development #13612
gérer les champs booléens
100%
Description
-
Fichiers
Révisions associées
Historique
Mis à jour par Benjamin Dauvergne il y a presque 7 ans
- Fichier 0001-feeder-import-boolean-form-s-fields-fixes-13612.patch 0001-feeder-import-boolean-form-s-fields-fixes-13612.patch ajouté
- Assigné à mis à Benjamin Dauvergne
- Patch proposed changé de Non à Oui
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
- Statut changé de Nouveau à Solution proposée
Patch oublié dans les limbes, je n'ai pas vérifié que ça se rebasait bien, ça attendra d'avoir de vrais tests dans wcs-olap.
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
- Fichier 0001-feeder-import-boolean-form-s-fields-fixes-13612.patch 0001-feeder-import-boolean-form-s-fields-fixes-13612.patch ajouté
Toujours le souci de pas de tests sur wcs-olap, mais c'est rebasé.
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
- Fichier 0002-feeder-import-boolean-form-s-fields-fixes-13612.patch 0002-feeder-import-boolean-form-s-fields-fixes-13612.patch ajouté
- Fichier 0001-add-tests-13612.patch 0001-add-tests-13612.patch ajouté
Commencé des tests en me basant sur ce que fait combo au niveau lancement de w.c.s. (j'en ai fait une jolie fixture, ce serait encore plus pratique si je pouvais me contenter d'un pip install wcs).
Mis à jour par Emmanuel Cazenave il y a environ 5 ans
Benjamin Dauvergne a écrit :
Commencé des tests en me basant sur ce que fait combo au niveau lancement de w.c.s. (j'en ai fait une jolie fixture, ce serait encore plus pratique si je pouvais me contenter d'un pip install wcs).
Tu peux pas faire comme dans le tox.ini de hobo : http://git.entrouvert.org/wcs.git/snapshot/wcs-master.tar.gz ?
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
Emmanuel Cazenave a écrit :
Benjamin Dauvergne a écrit :
Commencé des tests en me basant sur ce que fait combo au niveau lancement de w.c.s. (j'en ai fait une jolie fixture, ce serait encore plus pratique si je pouvais me contenter d'un pip install wcs).
Tu peux pas faire comme dans le tox.ini de hobo :
http://git.entrouvert.org/wcs.git/snapshot/wcs-master.tar.gz?
Je ne sais pas si wcs s'installe via pip, j'attendais confirmation par Fred que c'est possible maintenant, j'en profiterai pour faire un objet Wcs à la Slapd dans ldaptools, ça pourra finir en module pytest-wcs qu'on pourrait ajouter à w.c.s.
Mis à jour par Frédéric Péters il y a environ 5 ans
Je ne sais pas si wcs s'installe via pip, j'attendais confirmation par Fred que c'est possible maintenant
Je n'en sais rien.
Mis à jour par Emmanuel Cazenave il y a environ 5 ans
Aucune dépendance déclarée dans le setup.py donc probablement pas.
Mais du coup je ne comprends pas ta fixture peut marcher Benj.
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
Emmanuel Cazenave a écrit :
Aucune dépendance déclarée dans le setup.py donc probablement pas.
Mais du coup je ne comprends pas ta fixture peut marcher Benj.
Ben je fais comme dans hobo: les dépendances de w.c.s. sont déclarées dans le tox.ini (django, vobject, psycopg2-binary et quixote), j'aurai aimé remplacé ça par une simple référence au git-master de w.c.s.
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
- Fichier 0002-feeder-import-boolean-form-s-fields-fixes-13612.patch 0002-feeder-import-boolean-form-s-fields-fixes-13612.patch ajouté
- Fichier 0001-add-tests-13612.patch 0001-add-tests-13612.patch ajouté
Voilà le code définitif, je vais en rester là niveau infra de test.
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
- Fichier 0001-feeder-import-boolean-form-s-fields-fixes-13612.patch 0001-feeder-import-boolean-form-s-fields-fixes-13612.patch ajouté
Les tests sont passés avec l'ajout d'un Jenkinsfile et de job propres sur jenkins2 (et au passage j'ai renommé le slug du projet en wcs-olap pour avoir la remontée des statuts du job wcs-olap-wip, j'ai aussi ajouté un Redirect dans la conf apache du vieux nom vers le nouveau).
Voilà à mon humble avis c'est validable sans sourciller.
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
- Fichier 0001-feeder-import-boolean-form-s-fields-fixes-13612.patch 0001-feeder-import-boolean-form-s-fields-fixes-13612.patch ajouté
Ajout du check sur le schéma JSON.
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
- Fichier 0001-feeder-import-boolean-form-s-fields-fixes-13612.patch 0001-feeder-import-boolean-form-s-fields-fixes-13612.patch ajouté
sujet sans points médians.
Mis à jour par Emmanuel Cazenave il y a environ 5 ans
- Statut changé de Solution proposée à Solution validée
Sans sourciller.
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
- % réalisé changé de 0 à 100
Appliqué par commit bd7bc69468e4dfbc74dfebf87984b0128e87d433.
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
- Statut changé de Résolu (à déployer) à Fermé
feeder: import boolean form's fields (fixes #13612)