Projet

Général

Profil

Development #23210

UI pour import/export des roles et des OU

Ajouté par Emmanuel Cazenave il y a presque 6 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Catégorie:
-
Version cible:
-
Début:
16 avril 2018
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

La partie UI de #16514.

Comment, je ne sais, comme dans combo ?


Fichiers


Demandes liées

Lié à Authentic 2 - Development #11450: add a view for imports of roles and users Fermé20 juin 2016

Actions

Révisions associées

Révision 623c64a0 (diff)
Ajouté par Emmanuel Cazenave il y a presque 6 ans

add UI for import/export site (#23210)

Révision 184cb711 (diff)
Ajouté par Emmanuel Cazenave il y a presque 6 ans

add UI messages (#23210)

Historique

#1

Mis à jour par Frédéric Péters il y a presque 6 ans

#2

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.

#3

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.

#4

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.

#5

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.

#6

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.

#7

Mis à jour par Benjamin Dauvergne il y a presque 6 ans

Je dirai de faire comme dans Combo si c'est possible.

#8

Mis à jour par Emmanuel Cazenave il y a presque 6 ans

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

#9

Mis à jour par Benjamin Dauvergne il y a presque 6 ans

Tu peux mettre un screenshot pour voir ce que ça donne ?

#10

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.

#11

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

#12

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.

#13

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.

#14

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 $}.

#16

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.

#17

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)
#18

Mis à jour par Emmanuel Cazenave il y a presque 6 ans

  • Statut changé de Résolu (à déployer) à Solution déployée
#19

Mis à jour par Benjamin Dauvergne il y a plus de 5 ans

  • Statut changé de Solution déployée à Fermé

Formats disponibles : Atom PDF