Project

General

Profile

Project management #7773

Changements interfaces du /manage d'Authentic

Added by Pierre Cros about 4 years ago. Updated almost 4 years ago.

Status:
Fermé
Priority:
Normal
Category:
-
Target version:
-
Start date:
06 Jul 2015
Due date:
18 Sep 2015
% Done:

0%

Patch proposed:
Planning:
No
Demande du club utilisateur:
No

Description

Mockups joint. Ils ont déjà été discutés avec Benjamin et Fred (même si la discussion doit se poursuivre).

Lié à #7767 et #7768.

Ces changements me sont apparus nécessaires quand j'ai du présenter la gestion des rôles à Naïma. Et ils doivent être faits avant la fin août pour qu'elle puisse documenter l'ensemble correctement.

screenshot-cresson entrouvert org 2015-07-07 09-41-05.png View (42.5 KB) Benjamin Dauvergne, 07 Jul 2015 11:43 AM

permissions.ep - rôles mis à jour (223 KB) Pierre Cros, 03 Aug 2015 04:32 PM

rôles.pdf - rôles mis à jour (52 KB) Pierre Cros, 03 Aug 2015 04:33 PM

add-remove-roles.png View (10.1 KB) Benjamin Dauvergne, 22 Sep 2015 09:54 AM

7277
7836

Related issues

Related to Publik - Bug #7767: Cacher "Applications" dans Authentic Fermé 03 Jul 2015
Related to Publik - Bug #7768: Cacher "Entités" dans Authentic quand il n'y en a qu'une Fermé 03 Jul 2015
Related to Authentic 2 - Development #7788: Improve experience when there is only one organizational unit Fermé 06 Jul 2015
Related to Authentic 2 - Development #7793: a2_rbac: do not create ou management roles when there is less than 2 OUs Fermé 06 Jul 2015
Related to Authentic 2 - Development #7800: manager: improve stylesheet for dialog boxes Fermé 07 Jul 2015
Related to Authentic 2 - Bug #7787: Allow limiting the content types associated to manager roles, add a global manager role like ou scoped manager roles Fermé 06 Jul 2015
Related to Authentic 2 - Development #7761: Add application name (and entity?) to the role names in selectors Fermé 03 Jul 2015
Related to Authentic 2 - Development #8280: Improve UI for managing roles and users Fermé 16 Sep 2015
Related to Authentic 2 - Development #8141: hide link between role and services Fermé 31 Aug 2015 31 Aug 2015
Related to Authentic 2 - Development #8129: Improve display of user in user select2 field when adding users to role Fermé 28 Aug 2015
Related to Authentic 2 - Development #8130: Hide role permissions management for now Fermé 28 Aug 2015
Related to Authentic 2 - Development #7986: Expose more fields of the OrganizationalUnit model in admin Fermé 31 Jul 2015

History

#1 Updated by Pierre Cros about 4 years ago

  • Subject changed from Changements interfaces du /manage to Changements interfaces du /manage d'Authentic

#2 Updated by Frédéric Péters about 4 years ago

  • Related to Bug #7767: Cacher "Applications" dans Authentic added

#3 Updated by Frédéric Péters about 4 years ago

  • Related to Bug #7768: Cacher "Entités" dans Authentic quand il n'y en a qu'une added

#4 Updated by Pierre Cros about 4 years ago

Un point qui était marqué comme résolu mais qui ne l'est pas pour moi :
  • pour les rôles toutes OU on fait "Administrateur des utilisateurs/rôles/services" Et pour les rôles par OU, "Rôles - {{ou.name}}", "Utilisateurs - {{ou.name}}"

En fait les rôles "superuser" à montpel sont tous identiques d'une commune à l'autre, on peut pas les distinguer. Pour les rôles métiers, je pense aussi intéressant d'avoir le nom de la commune (même si on peut en débattre).

#5 Updated by Benjamin Dauvergne about 4 years ago

Sur cresson j'ai ce screenshot, ça correspnd à la description non ? Je viens de regarde c'est pareil sur https://idp-test-entrouvert.montpellier3m.fr/.

Pour le rôle superuser il vient d'ailleurs, j'ai un ticket ouvert (et résolu dans mon git) pour afficher l'entité à coté du nom dans le sélecteur, mais en fait pour les rôles "Utilisateurs - <entité>" ça va être redondant. Mais on pourrait ne pas faire ça et plutôt le renommer autrement lorsqu'il est créé, genre "Administrateur Clapiers" mais comme l'entité Clapiers et le w.c.s. de Clapiers portent le même nom il va y avoir collision entre l'administrateur des rôles/utilisateurs de Clapiers et l'administrateur w.c.s de Clapiers (en plus avoir "Adminstrateur" d'un coté et Superuser de l'autre c'est nul).

Pour moi c'est le noeud du problème, on a deux objets différents qui ont le même nom. Il faudrait peut-être aller vers un modèle à la cdg59, les administrateurs d'une entité, sont "administrateur" des utilisateurs, des rôles pour l'entité et aussi de tous les services, i.e. le rôle "Administrateur Clapier" hérite de "Utilisateurs - Clapiers", "Rôles - Clapiers" et aussi de "Superuser" sur tous les services de Clapiers (ici juste le w.c.s). Idéalement pour éviter les erreurs on essayerait de cacher le rôle "Superuser" pour ne laisser visible que "Administrateur Clapiers".

#7 Updated by Benjamin Dauvergne about 4 years ago

  • Related to Development #7788: Improve experience when there is only one organizational unit added

#8 Updated by Benjamin Dauvergne about 4 years ago

  • Related to Development #7793: a2_rbac: do not create ou management roles when there is less than 2 OUs added

#9 Updated by Benjamin Dauvergne about 4 years ago

#10 Updated by Benjamin Dauvergne about 4 years ago

  • Related to Bug #7787: Allow limiting the content types associated to manager roles, add a global manager role like ou scoped manager roles added

#11 Updated by Benjamin Dauvergne about 4 years ago

  • Related to Development #7761: Add application name (and entity?) to the role names in selectors added

#12 Updated by Benjamin Dauvergne about 4 years ago

Je ne comprends pas une demande du pdf: écran de création des rôles "sans entité" ? Si il y en a plusieurs il faut bien montrer dans quelle entité est le rôle il me semble.

#13 Updated by Frédéric Péters about 4 years ago

Je ne vois pas bien dans le PDF à quoi ça correspond (page 3 il y a un champ "entité"), mais de mon côté, je serais, quand il n'y a pas d'entité définie (ou une seule), de ne pas afficher les champs et colonne. (mais je pense l'avoir déjà écrit dans un autre ticket).

#14 Updated by Benjamin Dauvergne about 4 years ago

Frédéric Péters a écrit :

Je ne vois pas bien dans le PDF à quoi ça correspond (page 3 il y a un champ "entité"), mais de mon côté, je serais, quand il n'y a pas d'entité définie (ou une seule), de ne pas afficher les champs et colonne. (mais je pense l'avoir déjà écrit dans un autre ticket).

Ok je garde cette interprétation.

#15 Updated by Benjamin Dauvergne about 4 years ago

Il reste la plupart des modifications IHM dans le /manage et le fait de cacher les entité dans les différents formulaires.

#16 Updated by Pierre Cros about 4 years ago

  • File deleted (permissions.pdf)

#17 Updated by Pierre Cros about 4 years ago

  • File deleted (permissions.ep)

#18 Updated by Pierre Cros about 4 years ago

Après discussion avec Benjamin, les mockups mis à jour.

On s'est mis d'accord sur une phase 1 : rentre le truc utilisable aujourd'hui à Montpel sans parler de création des rôles w.c.s. dans Authentic (qui fera l'objet d'un autre ticket). Ça veut dire implémenter et installer les 3 premières pages du mockup.

#19 Updated by Pierre Cros about 4 years ago

Voir aussi #8007 #8009 #8010 #8011

#20 Updated by Benjamin Dauvergne almost 4 years ago

  • Due date changed from 28 Aug 2015 to 31 Aug 2015

#21 Updated by Benjamin Dauvergne almost 4 years ago

  • Due date changed from 31 Aug 2015 to 18 Sep 2015

#22 Updated by Benjamin Dauvergne almost 4 years ago

#23 Updated by Benjamin Dauvergne almost 4 years ago

#24 Updated by Benjamin Dauvergne almost 4 years ago

  • Related to Development #8129: Improve display of user in user select2 field when adding users to role added

#25 Updated by Benjamin Dauvergne almost 4 years ago

#26 Updated by Benjamin Dauvergne almost 4 years ago

  • Related to Development #7986: Expose more fields of the OrganizationalUnit model in admin added

#27 Updated by Benjamin Dauvergne almost 4 years ago

  • Status changed from Nouveau to Solution déployée

#28 Updated by Pierre Cros almost 4 years ago

Je me base sur https://cresson.entrouvert.org/manage/users/
et https://cresson.entrouvert.org/manage/roles/ pour faire mes remarques.

J'ai pas fait de tests de non régression ( = j'ai pas cliqué partout pour
voir si des trucs étaient pétés). Tout ça m'a l'air trés bien cela étant.

  • https://cresson.entrouvert.org/manage/users/ : "libellé" pas top, je
    préfère "Utilisateur".
  • "Ce rôle est interne vou**e** ne pouvez pas en changer le gestionnaire".
  • L'effet d'ombre derrière les liens grisés (Supprimer, Modifier la
    description, Gestionnaires) : même si j'apprécie l'effort, ça nuit à la
    lisibilité un peu et à l'esthétique surtout (parce que c'est vraiment
    différent du reste du thème qui est très "à plat" pour le reste).
  • "Contient les membres des rôles:" et "Contient les permissions des rôles:"
    => espaces manquantes avant le ":" (mais les libellés sont excellents, bien
    vu).
  • "This role is technical..." => pas traduit
  • https://cresson.entrouvert.org/manage/roles/19/ Les boutons - et + orange
    sont trop collés au texte. Pareil ici
    https://cresson.entrouvert.org/manage/roles/18/ il faut avoir le même espace
    avant et après le "-"

#29 Updated by Benjamin Dauvergne almost 4 years ago

7836

Pierre Cros a écrit :

Je me base sur https://cresson.entrouvert.org/manage/users/
et https://cresson.entrouvert.org/manage/roles/ pour faire mes remarques.

J'ai pas fait de tests de non régression ( = j'ai pas cliqué partout pour
voir si des trucs étaient pétés). Tout ça m'a l'air trés bien cela étant.

Ça me va, j'aimais pas libellé mais pas plus Identifiant qui était dans le pdf.

  • "Ce rôle est interne vou**e** ne pouvez pas en changer le gestionnaire".

Ok.

  • L'effet d'ombre derrière les liens grisés (Supprimer, Modifier la
    description, Gestionnaires) : même si j'apprécie l'effort, ça nuit à la
    lisibilité un peu et à l'esthétique surtout (parce que c'est vraiment
    différent du reste du thème qui est très "à plat" pour le reste).

Voir avec Fred, c'est le thème Gadjo qui fournit ça. J'ai ouvert #8307.

  • "Contient les membres des rôles:" et "Contient les permissions des rôles:"
    => espaces manquantes avant le ":" (mais les libellés sont excellents, bien
    vu).

Ok.

  • "This role is technical..." => pas traduit

Ok.

Ok changé en ça:

.role-add {
    margin-left: 1ex;
}
.role-remove {
    margin-left: 0.5ex;
    margin-right: 1ex;
}

ce qui donne la capture.

#30 Updated by Pierre Cros almost 4 years ago

Ok pour la capture.

#31 Updated by Pierre Cros almost 4 years ago

  • Status changed from Solution déployée to Fermé

Also available in: Atom PDF