Project

General

Profile

Development #13612

gérer les champs booléens

Added by Benjamin Dauvergne over 2 years ago. Updated 5 months ago.

Status:
Fermé
Priority:
Normal
Start date:
14 Oct 2016
Due date:
% Done:

100%

Patch proposed:
Yes
Planning:
No

Description

-

0001-feeder-import-boolean-form-s-fields-fixes-13612.patch View (4.97 KB) Benjamin Dauvergne, 15 May 2017 12:07 PM

0001-feeder-import-boolean-form-s-fields-fixes-13612.patch View (5.14 KB) Benjamin Dauvergne, 14 Jan 2019 10:57 AM

0002-feeder-import-boolean-form-s-fields-fixes-13612.patch View (8.57 KB) Benjamin Dauvergne, 15 Jan 2019 07:23 PM

0001-add-tests-13612.patch View (10.3 KB) Benjamin Dauvergne, 15 Jan 2019 07:23 PM

0002-feeder-import-boolean-form-s-fields-fixes-13612.patch View (9.1 KB) Benjamin Dauvergne, 16 Jan 2019 05:19 PM

0001-add-tests-13612.patch View (11.1 KB) Benjamin Dauvergne, 16 Jan 2019 05:19 PM

0001-feeder-import-boolean-form-s-fields-fixes-13612.patch View (9.09 KB) Benjamin Dauvergne, 17 Jan 2019 11:28 PM

0001-feeder-import-boolean-form-s-fields-fixes-13612.patch View (9.77 KB) Benjamin Dauvergne, 18 Jan 2019 09:19 AM

0001-feeder-import-boolean-form-s-fields-fixes-13612.patch View (9.62 KB) Benjamin Dauvergne, 18 Jan 2019 10:54 AM

Associated revisions

Revision bd7bc694 (diff)
Added by Benjamin Dauvergne 5 months ago

feeder: import boolean form's fields (fixes #13612)

History

#1 Updated by Benjamin Dauvergne about 2 years ago

#2 Updated by Benjamin Dauvergne 8 months ago

  • Status changed from Nouveau to 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.

#3 Updated by Benjamin Dauvergne 5 months ago

Toujours le souci de pas de tests sur wcs-olap, mais c'est rebasé.

#4 Updated by Benjamin Dauvergne 5 months ago

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).

#5 Updated by Emmanuel Cazenave 5 months ago

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 ?

#6 Updated by Benjamin Dauvergne 5 months ago

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.

#7 Updated by Frédéric Péters 5 months ago

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.

#8 Updated by Emmanuel Cazenave 5 months ago

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.

#9 Updated by Benjamin Dauvergne 5 months ago

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.

#10 Updated by Benjamin Dauvergne 5 months ago

Voilà le code définitif, je vais en rester là niveau infra de test.

#11 Updated by Benjamin Dauvergne 5 months ago

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.

#12 Updated by Benjamin Dauvergne 5 months ago

Ajout du check sur le schéma JSON.

#14 Updated by Emmanuel Cazenave 5 months ago

  • Status changed from Solution proposée to Solution validée

Sans sourciller.

#15 Updated by Benjamin Dauvergne 5 months ago

  • % Done changed from 0 to 100
  • Status changed from Solution validée to Résolu (à déployer)

#16 Updated by Benjamin Dauvergne 5 months ago

  • Status changed from Résolu (à déployer) to Fermé

Also available in: Atom PDF