Development #45082
ui d'export/import des rôles
0%
Description
C'est une partie de ce qui était dans #11450 mais je prends prétexte d'un besoin récent (gestion recette/prod toodego) pour en faire un ticket limité aux rôles.
Fichiers
Demandes liées
Révisions associées
manager: clearer semantics and interface for export (#45082)
CSV export is not called "export" anymore, and button is not in the
same place as real JSON export.
manager: enable title overriding in import template (#45082)
manager: open import site form in popup (#45082)
manager: add ou and role imports (#45082)
Historique
Mis à jour par Frédéric Péters il y a presque 4 ans
- Lié à Development #11450: add a view for imports of roles and users ajouté
Mis à jour par Valentin Deniaud il y a plus de 3 ans
Quel est le besoin/cas d'usage, par rapport à l'import/export du site, qui exporte rôles et ou ?
Mis à jour par Frédéric Péters il y a plus de 3 ans
Dans le ticket #40315 lié c'est un déploiement multi-collectivités et le besoin était d'exporter les rôles d'une seule collectivité, sur la recette, puis l'importer sur la prod.
Mis à jour par Valentin Deniaud il y a plus de 3 ans
- Fichier 0003-manager-add-ou-and-role-imports-45082.patch 0003-manager-add-ou-and-role-imports-45082.patch ajouté
- Fichier 0002-manager-clearer-semantics-and-interface-for-export-4.patch 0002-manager-clearer-semantics-and-interface-for-export-4.patch ajouté
- Fichier 0001-ou_views-fix-typo-45082.patch 0001-ou_views-fix-typo-45082.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
L'export était déjà là, c'est #29162. Il fallait bien 0002 pour que ce soit intelligible, et 0003 ajoute l'import correspondant.
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
- Statut changé de Solution proposée à En cours
1. C'est vraiment qu'une question de style Django mais à la longue c'est fatiguant à lire car on se demande à quoi sert exactement le Form, la validation c'est normalement le boulot des formulaires, sauf si vraiment ça rend les choses complexes il faut le laisser là :
try:
json_site = json.loads(self.request.FILES['site_json'].read())
except ValueError:
form.add_error('site_json', _('File is not in the expected JSON format.'))
return self.form_invalid(form)
Dans form_valid() on doit avoir des données déjà validées.
2. Il y a des modifications à l'UI actuelle qui n'ont pas direct avec le ticket il me semble, ça devrait au moins aller dans un commit à part
<li><a href="{% url 'a2-manager-site-import' %}" rel="popup">{% trans 'Import Site' %}</a></li>
Le souci des popups c'est qu'avec des formulaires qui retournent fréquemment des erreurs ça marche mal (peut-être pas dans a2 je ne sais plus, me semble que je gérer ça mais...)
2.1 Idem pour title/TitleMixin ça a un lien indirect, ça peut faire un commit à part (manager: enable title overriding in import views...).
3. Dans les tests d'export OU / rôle je vérifierai qu'on n'y trouve pas autre chose (des rôles ou des OUs), sait-on jamais, et qu'à l'import idem on importe pas ce qui dépasse.
Mis à jour par Valentin Deniaud il y a plus de 3 ans
- Fichier 0004-manager-open-import-site-form-in-popup-45082.patch 0004-manager-open-import-site-form-in-popup-45082.patch ajouté
- Fichier 0002-manager-clearer-semantics-and-interface-for-export-4.patch 0002-manager-clearer-semantics-and-interface-for-export-4.patch ajouté
- Fichier 0005-manager-add-ou-and-role-imports-45082.patch 0005-manager-add-ou-and-role-imports-45082.patch ajouté
- Fichier 0003-manager-enable-title-overriding-in-import-template-4.patch 0003-manager-enable-title-overriding-in-import-template-4.patch ajouté
- Fichier 0001-ou_views-fix-typo-45082.patch 0001-ou_views-fix-typo-45082.patch ajouté
- Statut changé de En cours à Solution proposée
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
- Statut changé de Solution proposée à Solution validée
Tout bon pour moi.
Mis à jour par Valentin Deniaud il y a plus de 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 4f3c6a47d6f970c7f659567e945076f597136265
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date: Mon Aug 24 14:44:38 2020 +0200
manager: add ou and role imports (#45082)
commit 7c1e2e1d2b896315c28ed652789fbbfb3058a916
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date: Mon Aug 24 14:43:41 2020 +0200
manager: open import site form in popup (#45082)
commit bd4b8b8eef9b13c96f742d986e37e6406d7f1d84
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date: Mon Aug 24 14:41:42 2020 +0200
manager: enable title overriding in import template (#45082)
commit 621a5e978f24481be73b90cc314376f6b083f118
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date: Thu Aug 6 17:34:21 2020 +0200
manager: clearer semantics and interface for export (#45082)
CSV export is not called "export" anymore, and button is not in the
same place as real JSON export.
commit 057084200373b8bc149eabf0e72a3acac63d6fad
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date: Thu Aug 6 15:14:22 2020 +0200
ou_views: fix typo (#45082)Mis à jour par Frédéric Péters il y a plus de 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
ou_views: fix typo (#45082)