Development #70276
rsa13: intégrer la configuration des colonnes CSV au modèle
0%
Description
Ce sera plus simple que de jouer aves variables hobo.
Fichiers
Révisions associées
rsa13: add fields to manage CSV columns (#70276)
Historique
Mis à jour par Benjamin Dauvergne il y a plus d'un an
- Fichier 0002-rsa13-add-fields-to-manage-CSV-columns-70276.patch 0002-rsa13-add-fields-to-manage-CSV-columns-70276.patch ajouté
- Fichier 0001-rsa13-adapt-verify_cert-field-migration-70276.patch 0001-rsa13-adapt-verify_cert-field-migration-70276.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Noël il y a plus d'un an
Je n'ai pas compris le 0001 alors qu'il y a aussi un AlterField dans le 0002 sur verify_cert. Tu voulais sans doute éviter le second ?
Et pourquoi dans la migration ne pas rapatrier les valeurs de settings.FOOBAR ou self.FOOBAR, une fois pour toute, histoire de supprimer toute magie ?
Mis à jour par Benjamin Dauvergne il y a plus d'un an
- Fichier 0002-rsa13-add-fields-to-manage-CSV-columns-70276.patch 0002-rsa13-add-fields-to-manage-CSV-columns-70276.patch ajouté
- Fichier 0001-rsa13-adapt-verify_cert-field-migration-70276.patch 0001-rsa13-adapt-verify_cert-field-migration-70276.patch ajouté
Ok.
Mis à jour par Benjamin Dauvergne il y a plus d'un an
Mis à jour par Thomas Noël il y a plus d'un an
L'ami pylint n'est pas content, avec ce bizarre message :
passerelle/contrib/rsa13/models.py:67: [E0101(return-in-init), RSA13Form.__init__] Explicit return in __init__
Mis à jour par Benjamin Dauvergne il y a plus d'un an
- Fichier 0002-rsa13-add-fields-to-manage-CSV-columns-70276.patch 0002-rsa13-add-fields-to-manage-CSV-columns-70276.patch ajouté
- Fichier 0001-rsa13-adapt-verify_cert-field-migration-70276.patch 0001-rsa13-adapt-verify_cert-field-migration-70276.patch ajouté
My bad.
Mis à jour par Thomas Noël il y a plus d'un an
- Statut changé de Solution proposée à Solution validée
Et je me demande tout à coup : imagine-t-on à un moment que « DEFAULTS » puisse changer, auquel cas un connecteur déjà instancié ne verra pas ce changement et ne pourra pas en profiter magiquement ?
Si oui, on pourrait dire que le DEFAULT[...] est à utiliser quand un paramétrage est vide ; et afficher le paramétrage DEFAULT[...] juste en help_text, genre « if empty, default is .... » ?
Si ce niveau de détail est inutile, alors tu peux pousser ainsi.
Mis à jour par Benjamin Dauvergne il y a plus d'un an
Thomas Noël a écrit :
Si ce niveau de détail est inutile, alors tu peux pousser ainsi.
Ouais, je suis déjà allé trop loin pour un connecteur à usage unique, je voulais juste tuer les tickets "ajout d'un colonne" qui arrivent une fois par trimestre.
Mis à jour par Benjamin Dauvergne il y a plus d'un an
- Statut changé de Solution validée à Résolu (à déployer)
commit c0b972e7acd8f84676512727e0e78a066dcc545f
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date: Fri Oct 14 12:43:42 2022 +0200
rsa13: add fields to manage CSV columns (#70276)
commit 4322b1a819847a3f6f4fb8ecfbfbce16f70f5bab
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date: Fri Oct 14 12:42:34 2022 +0200
rsa13: adapt verify_cert field migration (#70276)Mis à jour par Transition automatique il y a plus d'un an
- Statut changé de Résolu (à déployer) à Solution déployée
rsa13: adapt verify_cert field migration (#70276)