Development #51120
Indiquer dans la lise des rôles lesquels sont synchronisés via LDAP
0%
Description
On a déjà cette information sur la page d'un rôle, mais on ne l'a pas directement sur la page qui liste tous les rôles.
Fichiers
Révisions associées
Historique
Mis à jour par Nicolas Roche il y a environ 3 ans
- Fichier Screenshot_2021-02-16 Connexion - Rôles.png Screenshot_2021-02-16 Connexion - Rôles.png ajouté
- Fichier 0001-manager-add-a-column-to-display-role-is-synced-from-.patch 0001-manager-add-a-column-to-display-role-is-synced-from-.patch ajouté
- Tracker changé de Support à Development
- Statut changé de Nouveau à Solution proposée
- Assigné à mis à Nicolas Roche
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a environ 3 ans
Pour moi il ne faut pas que cette colonne apparaisse sur les installations où aucun groupe n'est synchronisé.
Aussi en terme de présentation je serais à limiter les colonnes et plutôt trouver comment passer l'information dans ce qui existe, peut-être par exemple dans la colonne avec le nombre de membres, indiqués entre parenthèses derrière (via LDAP) quand l'info est tirée du LDAP; ou étendre la colonne libellé avec l'info (ex: "Blah (rôle synchronisé LDAP)"); ou autre.
(étendre la colonne "nombre de membres" étant ma première idée parce que le très long libellé de celle-ci amène aujourd'hui pas mal d'espace qui pourrait être opportunément utilisé, sans serrer le reste).
Mis à jour par Nicolas Roche il y a environ 3 ans
- Fichier 0001-manager-add-a-column-to-display-role-is-synced-from-.patch 0001-manager-add-a-column-to-display-role-is-synced-from-.patch ajouté
Remarque prise en compte : '(via LDAP)' ajouté dans la colonne "Nombre de membres directs".
Mis à jour par Benjamin Dauvergne il y a environ 3 ans
Nicolas Roche a écrit :
Remarque prise en compte : '(via LDAP)' ajouté dans la colonne "Nombre de membres directs".
Je préférerai un simple '(LDAP)' en dehors du noeud <a/>
derrière le libellé, là on ne sait pas trop ce que ça vient faire dans la colonne du nombre de membres.
Mis à jour par Nicolas Roche il y a environ 3 ans
- Fichier Screenshot_2021-02-18 Connexion(1).png Screenshot_2021-02-18 Connexion(1).png ajouté
- Fichier 0001-manager-add-a-column-to-display-role-is-synced-from-.patch 0001-manager-add-a-column-to-display-role-is-synced-from-.patch ajouté
- Statut changé de Solution proposée à Information nécessaire
Je préférerai un simple '(LDAP)' en dehors du noeud <a/> derrière le libellé,
Donc ne plus utiliser de tables.LinkColumn pour le libellé ?
(j'ai l'impression que ce n'est pas possible autrement, s'il s'agit de l'insérer dans la même colonne).
Je réalise que ce travaille avait déjà été fait d'une autre façon sur la table OuUserRolesTable.
Je n'ai pas réussit à le montrer dans la capture d'écran mais quand on passe la souris sur le la croix grisée à droite, le texte suivant s'affiche :
'This role is synchronised from LDAP, changing members is not allowed.'
La remarque ci-dessus est mise en évidence dans le test qui affiche l'attribut 'disabled'.
J'ai néanmoins affiché '(LDAP)' dans le libéllé des rôles sur les 3 écrans suivants :- https://authentic.dev.publik.love/manage/roles/
- https://authentic.dev.publik.love/manage/users/<uuid>/roles/ (une seule OU)
- https://authentic.dev.publik.love/manage/users/<uuid>/roles/ (plusieurs OU)
J'imagine qu'il va falloir harmoniser ces 2 comportements, et donc je viens aux nouvelles.
Mis à jour par Benjamin Dauvergne il y a environ 3 ans
Nicolas Roche a écrit :
Je préférerai un simple '(LDAP)' en dehors du noeud <a/> derrière le libellé,
Donc ne plus utiliser de tables.LinkColumn pour le libellé ?
(j'ai l'impression que ce n'est pas possible autrement, s'il s'agit de l'insérer dans la même colonne).
Je n'ai pas le code en tête mais oui il faut probablement une colonne utilisant un template maintenant plutôt qu'une LinkColumn.
J'imagine qu'il va falloir harmoniser ces 2 comportements, et donc je viens aux nouvelles.
Le nouveau comportement est plus explicite, il a ma préférence (après on peut toujours utiliser un propriété title="..." sur le (LDAP)
pour donner des détails, mais là tout de suite je n'en vois pas).
Mis à jour par Nicolas Roche il y a environ 3 ans
- Fichier 0001-manager-add-a-column-to-display-role-is-synced-from-.patch 0001-manager-add-a-column-to-display-role-is-synced-from-.patch ajouté
- Statut changé de Information nécessaire à Solution proposée
(
j'ai l'impression que ce n'est pas possible autrement, s'il s'agit de l'insérer dans la même colonne).
Vraiment désolé, j'ai parlé trop vite.
edit:
Mais je serais d'accord au final tu préfères un TemplateColumn.
Il me reste encore à choisir entre 2 possibilités si je veux harmoniser le même rendu partout :Le nouveau comportement est plus explicite
- est-ce que je retire ce code du template de la la table OuUserRolesTable :
member = tables.TemplateColumn( ... '{%% if not record.can_manage_members %%}disabled ' 'title="{%% trans "%s" %%}"{%% endif %%}/>' % (ugettext_noop('You are not authorized to manage this role'), ugettext_noop('This role is synchronised from LDAP, changing members is not allowed.')),
- ou est-ce que je rajoute ce code aux tables
UserRolesTable
(dans le champvia
qui contient déjà un template) etRoleTable
(en lui ajoutant un champ template, ou en transformant son champmember_count
en champ template) ?
Sinon, il reste encore la possibilité de ne pas toucher à l'existant : ce patch.
Mis à jour par Benjamin Dauvergne il y a environ 3 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Nicolas Roche il y a environ 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 2bdd088d430c0376f52f2c889ede962bb8904981 Author: Nicolas ROCHE <nroche@entrouvert.com> Date: Tue Feb 16 17:52:40 2021 +0100 manager: add a column to display role is synced from ldap (#51120)
Mis à jour par Frédéric Péters il y a environ 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
manager: add a column to display role is synced from ldap (#51120)