Project

General

Profile

Development #25633

cellule "fiche utilisateur" (?)

Added by Frédéric Péters over 4 years ago. Updated about 4 years ago.

Status:
Fermé
Priority:
Normal
Assignee:
-
Target version:
-
Start date:
10 August 2018
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:

Description

Pour présenter les attributs sur le portail agent/guichet; c'est quelque chose qu'on peut "manuellement" faire avec une cellule JSON (et côté front c'est déjà mis en pratique à Strasbourg), mais avoir ça sous forme d'une véritable cellule pourrait être plus clair et offrir davantage (genre un accès à la définition des attributs, via l'hobo.json, pour en obtenir les libellés).


Files


Related issues

Related to Hobo - Development #25634: exposer la définition du profil dans les settingsFermé10 August 2018

Actions
Related to Publik - Development #19756: Personnalisation accrue du portail agent pour en faire aussi la page d'entrée des agents d'accueilFermé29 October 201730 June 2018

Actions

Associated revisions

Revision 90a11b13 (diff)
Added by Frédéric Péters over 4 years ago

profile: add a new "user profile" cell (#25633)

History

#1

Updated by Frédéric Péters over 4 years ago

#2

Updated by Frédéric Péters over 4 years ago

#3

Updated by Frédéric Péters over 4 years ago

  • Related to Development #19756: Personnalisation accrue du portail agent pour en faire aussi la page d'entrée des agents d'accueil added
#4

Updated by Benjamin Dauvergne over 4 years ago

Ça limite un peu l'indentation (et puis c'est un poil moins obscure) d'utiliser la fixture settings plutôt que:

    with override_settings(
                KNOWN_SERVICES={'authentic': {'idp': {
                    'title': 'IdP', 'url': 'http://example.org/'}}}), \

idem pour mock comme décorateur (enfin je trouve).

#5

Updated by Thomas Noël over 4 years ago

Je ne suis pas sûr de l'utilité de context['concerned_user'] = self.get_concerned_user(context)

Je pense qu'on pourrait éviter la création de cette nouvelle variable, soit en utilisant juste {{ selected_user|name_id }} dans l'URL, soit {% firstof selected_user|name_id request.user|name_id %} si on veut que ça affiche l'utilisateur connecté quand il n'y a pas de selected_user. Il faut alors que name_id commence par un if not user: return None (ce qui serait bien de toute façon).

#7

Updated by Thomas Noël over 4 years ago

  • Status changed from Solution proposée to Solution validée

J'aurai bien vu un if not user: stop au début du filtre name_id pour éviter de faire une requête SQL pour rien sur le selected_user inexistant (affichage de la cellule en frontoffice) ... mais c'est peanuts en fait.

#8

Updated by Frédéric Péters over 4 years ago

J'y ai plutôt ajouté une méthode is_visible(), que le rendu soit totalement zappé si jamais la cellule était visible sans être connecté.

#9

Updated by Thomas Noël over 4 years ago

  • Status changed from Solution proposée to Solution validée

Impec.

#10

Updated by Frédéric Péters over 4 years ago

  • Status changed from Solution validée to Résolu (à déployer)
commit 90a11b1328f4e73c0a63ccf194e9ebaac4038c5b
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Fri Aug 10 17:11:26 2018 +0200

    profile: add a new "user profile" cell (#25633)
#11

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

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF