Projet

Général

Profil

Development #25633

cellule "fiche utilisateur" (?)

Ajouté par Frédéric Péters il y a plus de 5 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
10 août 2018
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
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).


Fichiers


Demandes liées

Lié à Hobo - Development #25634: exposer la définition du profil dans les settingsFermé10 août 2018

Actions
Lié à Publik - Development #19756: Personnalisation accrue du portail agent pour en faire aussi la page d'entrée des agents d'accueilFermé29 octobre 201730 juin 2018

Actions

Révisions associées

Révision 90a11b13 (diff)
Ajouté par Frédéric Péters il y a plus de 5 ans

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

Historique

#1

Mis à jour par Frédéric Péters il y a plus de 5 ans

#2

Mis à jour par Frédéric Péters il y a plus de 5 ans

#3

Mis à jour par Frédéric Péters il y a plus de 5 ans

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

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

Ç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

Mis à jour par Thomas Noël il y a plus de 5 ans

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

Mis à jour par Thomas Noël il y a plus de 5 ans

  • Statut changé de Solution proposée à 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

Mis à jour par Frédéric Péters il y a plus de 5 ans

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

Mis à jour par Thomas Noël il y a plus de 5 ans

  • Statut changé de Solution proposée à Solution validée

Impec.

#10

Mis à jour par Frédéric Péters il y a plus de 5 ans

  • Statut changé de Solution validée à 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

Mis à jour par Frédéric Péters il y a plus de 5 ans

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

Formats disponibles : Atom PDF