Produits Entr'ouvert » Authentic 2

I duplicated #5180, I copy here initial comment by fpeters:

I had some mockups of those management pages, once created they would be linked from the portail-admin.

http://perso.entrouvert.org/~fred/portail-admin/mockups/authentic-utilisateurs-bis.html
http://perso.entrouvert.org/~fred/portail-admin/mockups/authentic-roles.html

Full patchset can be cloned from http://repos.entrouvert.org/authentic.git/ on branch wip/manager. To test it you must go on http://localhost;8000/manager/users/ being logged with an user having permissions user.delete, user.change, user.add, group.add, group.delete, group.change, or being a superuser, which is faster to set.

De loin, sans l'avoir appliqué en local :

Je préférerais l'utilisation directe de gadjo plutôt que d'en recopier la presque intégralité ici.

Pour le javascript minifié, je préférerais qu'il se créer lors du build plutôt que de le distribuer; ou via un système comme django-compressor.

C'est du work in progress, quand gadjo existera sur pypi j'en dépendrai, ou alors peut-être préfère tu que je l'importe comme sous-module git ? Pour la minification, le problème ne me parait pas lié à ce ticket mais on peut bien sûr ouvrir un autre ticket sur l'idée plus générale d'utiliser django-compressor plus systématiquement, mais à mon avis ce travail serait plus utile sur portail-citoyen qui génère plus de vues.

Ok, c'est bien de voir apparaitre cette demande ("quand gadjo existera sur pypi j'en dépendrai") maintenant, j'ai créé #5182 avec.

Je n'avais pas noté que c'était work in progress, je vais en tenir compte.

To test it you must go on http://localhost;8000/manager/users/ [...]

Le fichier authentic2/manager/urls.py dans la branche wip/manager n'a pas de mention d'une partie users/, uniquement roles/, il reste des commits à y pousser ?

Bizarre j'ai ça dans le commit manager: implement users views et je vois la même chose avec cgit:

diff --git a/authentic2/manager/urls.py b/authentic2/manager/urls.py
index 0dcbd04..44fb86b 100644
--- a/authentic2/manager/urls.py
+++ b/authentic2/manager/urls.py
@@ -6,5 +6,9 @@ urlpatterns = patterns('authentic2.views',
         url(r'^roles/$', views.roles, name='a2-manager-roles'),
         url(r'^roles/add/$', views.role_add, name='a2-manager-role-add'),
         url(r'^roles/(?P<role_ref>[^/]*)/$', views.role, name='a2-manager-role'),
+        url(r'^roles/(?P<role_ref>[^/]*)/edit/$', views.role_edit, name='a2-manager-role-edit'),
+        url(r'^users/$', views.users, name='a2-manager-users'),
+        url(r'^users/add/$', views.user_add, name='a2-manager-user-add'),
+        url(r'^users/(?P<pk>[^/]*)/$', views.user_edit, name='a2-manager-user-edit'),
         url(r'^', include('django_select2.urls')),
    )

Je ne sais pourquoi il ne m'avait pas pris tous les commits de la branche, c'est ok maintenant.

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

À l'utilisation, et pour une partie grossièrement présentées dans le diff ci-joint, voici mes notes et des modifications que je suggérerais :

• suppression des liens transversaux "Users management" et "Roles management", ils ont là une position inattendue, et il doit y avoir une page générale /manager/ qui les reprend (c'est en tout cas le cas depuis le portail admin).

Ok, et je pointe le lien accueil dessus en absence d'un setting qui fait pointer celui-ci sur le portail-admin.

• ajout d'un bouton "Rechercher" dans les formulaires de recherche (et suppression du bouton "Effacer"), input type="search" sur le champ d'entrée.

Comment vide-t-on ce champ recherche ? Dans mon idée le bouton Effacer deviendrait une simple croix après passage du webdesign :)

• montée au niveau de la page de l'information sur le rôle sélectionné, avec les liens "add role" (quand pas de rôle sélectionné) et "edit" et "delete" (quand un rôle sélectionné) à la même position que sur la page des utilisateurs (et que sur les pages de passerelle)

Delete, je vois mais edit je ne comprends pas,

• (à voir) position de la recherche dans la page d'un rôle, au-dessus du tableau ça ne me semblait pas aller, je l'ai déplacée dans la barre latérale, mais ça pourrait peut-être aller sous le tableau.

Je vais tenter dessous.

• dans le formulaire derrière "Ajouter un utilisateur à ce rôle :", il ne faudrait pas répéter "utilisateur".

Ok.

• dans les entêtes de colonne, il ne faudrait pas la capitalisation (elle se fait dans tables2, class Column, propriété header, je ne sais pas s'il y a un moyen propre de modifier ça).

Bizarre à lire le code c'est un fallback quand on utilise pas de queryset, sinon c'est sensé prendre l'attribute verbose_name du champ du modèle, à rechercher.

• dans le tableau listant les utilisateurs possédant un rôle, je mettrais en bout de ligne une icône "retirer" (icon-remove-sign, f057 dans font awesome). (plutôt que le context menu).

Ok.

• idéalement il y aurait un message quand on tente d'ajouter un utilisateur disposant déjà du rôle (ou plus idéal encore, qu'il ne soit pas proposé dans le <select>)

Je préfère le message, l'autre solution ne donne pas de raison quand à l'absence de l'utilisateur dans la liste.

• ajout d'un bouton "Rechercher" dans les formulaires de recherche (et suppression du bouton "Effacer"), input type="search" sur le champ d'entrée.

Comment vide-t-on ce champ recherche ? Dans mon idée le bouton Effacer deviendrait une simple croix après passage du webdesign :)

Avec input type="search" il y a une petite croix qui apparait.

http://www.w3schools.com/html/tryit.asp?filename=tryhtml5_input_type_search

• montée au niveau de la page de l'information sur le rôle sélectionné, avec les liens "add role" (quand pas de rôle sélectionné) et "edit" et "delete" (quand un rôle sélectionné) à la même position que sur la page des utilisateurs (et que sur les pages de passerelle)

Delete, je vois mais edit je ne comprends pas,

Tu ne comprends pas quoi ? C'est le lien "Edit" qui existe déjà, il est déplacé vers la barre supérieure.

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

• ajout d'un bouton "Rechercher" dans les formulaires de recherche (et suppression du bouton "Effacer"), input type="search" sur le champ d'entrée.

Comment vide-t-on ce champ recherche ? Dans mon idée le bouton Effacer deviendrait une simple croix après passage du webdesign :)

Avec input type="search" il y a une petite croix qui apparait.

http://www.w3schools.com/html/tryit.asp?filename=tryhtml5_input_type_search

Vu.

• montée au niveau de la page de l'information sur le rôle sélectionné, avec les liens "add role" (quand pas de rôle sélectionné) et "edit" et "delete" (quand un rôle sélectionné) à la même position que sur la page des utilisateurs (et que sur les pages de passerelle)

Delete, je vois mais edit je ne comprends pas,

Tu ne comprends pas quoi ? C'est le lien "Edit" qui existe déjà, il est déplacé vers la barre supérieure.

Ok vu.

Voilà j'ai intégré toutes les remarques, j'ai laissé le champ recherche à gauche finalement.

Non j'ai oublié l'icône retirer...

It would be nice if two latest commits ("integrate fred diff to templates" and "integrate comments from #5180") would be integrated into previous commits; also I would prefer to have gadjo used as a git submodule for the time being (this would be changed once it gets on pypi).

Ces patchs dépendent des patchs actuellement en attente sur gadjo.

Looks good to me.