Development #23210
UI pour import/export des roles et des OU
0%
Fichiers
Demandes liées
Historique
Mis à jour par Frédéric Péters il y a presque 6 ans
- Lié à Development #11450: add a view for imports of roles and users ajouté
Mis à jour par Frédéric Péters il y a presque 6 ans
Dans la page /manage/roles/, à droite de "Ajouter un rôle", des liens "Importer" et "Exporter", l'export produisant le json, l'import ouvrant une boite de dialogue où on peut choisir un fichier.
Mis à jour par Benjamin Dauvergne il y a presque 6 ans
À voir si on reprend le filtrage ou pas (permettant ainsi d'exporter les rôles d'une seule OU ou d'importer les rôles dans une seule OU, et dans ce cas est-ce que le sélecteur d'OU influe sur l'OU par défaut). Ça m'irait qu'on donne quelques objectifs fonctionnels ici, sinon on vise juste export/import total.
Mis à jour par Emmanuel Cazenave il y a presque 6 ans
Avec ce qui a été fait dans #16514 on ne peut faire que du total pour l'instant.
Mis à jour par Benjamin Dauvergne il y a presque 6 ans
Oui oui, je note juste qu'avec l'interface actuelle pour que ce soit cohérent il faudrait respecter le filtrage, maintenant on peut vouloir immédiatement cette UI et se permettre cette incohérence pendant un certain temps.
Mis à jour par Emmanuel Cazenave il y a presque 6 ans
Frédéric Péters a écrit :
à droite de "Ajouter un rôle"
Dans la zone grise, au dessus de "Recherche" ?
Ou dans la zone blanche et du coup "Ajouter un rôle" se retrouve un peu décalé à vers la gauche ?
Aussi on est sur la page /manage/roles mais on va exporter/importer des OU et des roles.
Mis à jour par Benjamin Dauvergne il y a presque 6 ans
Je dirai de faire comme dans Combo si c'est possible.
Mis à jour par Emmanuel Cazenave il y a presque 6 ans
- Fichier 0001-add-UI-for-import-export-site-23210.patch 0001-add-UI-for-import-export-site-23210.patch ajouté
- Fichier 0002-add-UI-messages-23210.patch 0002-add-UI-messages-23210.patch ajouté
- Patch proposed changé de Non à Oui
Comme dans combo, c'est à dire à travers l'icone kébab sur /manage
.
Ce qui pose peut-être un problème puisque la page /manage
, on y accède pas à coup de clics, mais ça me parait plus logique que depuis la page
rôles parce que on importe/exporte des rôles et des OU (et plus tard des utilisateurs ...).
Pas de dry-run (ça m'a l'air bien compliqué de désactiver le provisionning).
Rien fait sur des éventuelles des histoires permissions du type "je peux faire un import/export que si j'ai le rôle machin".
Mis à jour par Benjamin Dauvergne il y a presque 6 ans
Tu peux mettre un screenshot pour voir ce que ça donne ?
Mis à jour par Emmanuel Cazenave il y a presque 6 ans
Avec le kébab déplié parce que j'ai cliqué dessus.
Les deux vues site-export
et site-import
sont exactement pareil que dans combo.
Mis à jour par Pierre Cros il y a presque 6 ans
Qui est la cible de cette évolution ? Si c'est nous, en interne, pas de pb.
Si ce sont les clients, c'est un écran sur lequel ils ne vont pas comme tu le pointais et du coup, il faudrait quelque chose sur un écran sur lequel ils vont.
Pierre
Mis à jour par Benjamin Dauvergne il y a presque 6 ans
Emmanuel Cazenave a écrit :
Comme dans combo, c'est à dire à travers l'icone kébab sur
/manage
.
Ce qui pose peut-être un problème puisque la page/manage
, on y accède pas à coup de clics, mais ça me parait plus logique que depuis la page
rôles parce que on importe/exporte des rôles et des OU (et plus tard des utilisateurs ...).
Pas de dry-run (ça m'a l'air bien compliqué de désactiver le provisionning).
Oui, je pense qu'il va falloir attendre Django 1.11 et le hook transaction.on_commit qui permet de n'exécuter une action que si la transaction en cours termine.
Rien fait sur des éventuelles des histoires permissions du type "je peux faire un import/export que si j'ai le rôle machin".
Pour l'instant je n'ouvrirai ça qu'au superadmin, c'est à dire nous en général.
Mis à jour par Emmanuel Cazenave il y a presque 6 ans
Benjamin Dauvergne a écrit :
Rien fait sur des éventuelles des histoires permissions du type "je peux faire un import/export que si j'ai le rôle machin".
Pour l'instant je n'ouvrirai ça qu'au superadmin, c'est à dire nous en général.
Tu peux me pointer un bout de code ou quelque chose, j'ai aucune idée de comment faire ça.
Pierre Cros a écrit :
Qui est la cible de cette évolution ? Si c'est nous, en interne, pas de pb.
J'imagine que c'est plus pour nous, mais pas sûr, il faudrait que quelqu'un d'autre dise.
Mis à jour par Benjamin Dauvergne il y a presque 6 ans
Emmanuel Cazenave a écrit :
Benjamin Dauvergne a écrit :
Rien fait sur des éventuelles des histoires permissions du type "je peux faire un import/export que si j'ai le rôle machin".
Pour l'instant je n'ouvrirai ça qu'au superadmin, c'est à dire nous en général.
Tu peux me pointer un bout de code ou quelque chose, j'ai aucune idée de comment faire ça.
Pour l'instant tu peux simplement faire dans tes vues:
if not request.user.is_superuser: raise PermissionDenied # from django.core.exceptions
et idem dans ton template tu caches le menu avec {% if user.is_superuser $}
.
Mis à jour par Emmanuel Cazenave il y a presque 6 ans
- Fichier 0001-add-UI-for-import-export-site-23210.patch 0001-add-UI-for-import-export-site-23210.patch ajouté
- Fichier 0002-add-UI-messages-23210.patch 0002-add-UI-messages-23210.patch ajouté
Merci.
Voilà, avec la restriction sur superuser
.
Mis à jour par Benjamin Dauvergne il y a presque 6 ans
J'aime bien quand on indente les structures de contrôle comme les tags HTML:
{% if user.is_superuser %} <a class="extra-actions-menu-opener"></a>(ça vaut pour les
{% block %}
aussi)
Ack.
Mis à jour par Emmanuel Cazenave il y a presque 6 ans
- Statut changé de Nouveau à Résolu (à déployer)
Donc l'indentation était bonne dans mon patch c'est ça ?
commit 623c64a064c5f3f4942275094bb1d3763ef923d1 Author: Emmanuel Cazenave <ecazenave@entrouvert.com> Date: Thu May 3 11:40:25 2018 +0200 add UI for import/export site (#23210)
commit 184cb7114853f9a07c08dd96f3ae7964c2161538 Author: Emmanuel Cazenave <ecazenave@entrouvert.com> Date: Thu May 3 11:41:07 2018 +0200 add UI messages (#23210)
Mis à jour par Emmanuel Cazenave il y a presque 6 ans
- Statut changé de Résolu (à déployer) à Solution déployée
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
- Statut changé de Solution déployée à Fermé
add UI for import/export site (#23210)