Projet

Général

Profil

Development #12366

prise en charge de champs de profil "vérifiés"

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

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
Début:
29 juin 2016
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

L'idée c'est qu'on ait des champs de profil dont la valeur ne soit pas éditable par l'usager, qui auraient par exemple été posés en interrogeant un webservice, ou récupérés d'un certificat.

On a derrière envie d'avoir un formulaire avec un champ utilisant cette info mais que celui-ci ne soit pas éditable par l'usager. (mais on a envie du champ parce que le formulaire doit également fonctionner sur des situations où l'usager n'est pas identifié et entre l'info manuellementà.


Fichiers

0001-general-support-prefilling-with-verified-profile-fie.patch (8,4 ko) 0001-general-support-prefilling-with-verified-profile-fie.patch côté wcs, la prise en charge Frédéric Péters, 29 juin 2016 10:32
0001-saml-add-verified-fields-in-user-profile-12366.patch (1,85 ko) 0001-saml-add-verified-fields-in-user-profile-12366.patch côté auquo, pour positionner l'info dans le profil Frédéric Péters, 29 juin 2016 10:33
0001-general-support-prefilling-with-verified-profile-fie.patch (16,7 ko) 0001-general-support-prefilling-with-verified-profile-fie.patch bon patch wcs. Frédéric Péters, 29 juin 2016 10:44

Demandes liées

Dupliqué par w.c.s. - Development #11106: champs de profil certifiésFermé28 mai 2016

Actions

Révisions associées

Révision e5e03418 (diff)
Ajouté par Frédéric Péters il y a presque 8 ans

saml: add verified fields in user profile (#12366)

Révision 318cc4f0 (diff)
Ajouté par Frédéric Péters il y a presque 8 ans

general: support prefilling with "verified" profile fields (#12366)

Historique

#1

Mis à jour par Frédéric Péters il y a presque 8 ans

C'est plutôt moche côté auquotidien, c'est en attendant qu'authentic puisse gérer nom/prénom/email comme les autres attributs.

#2

Mis à jour par Frédéric Péters il y a presque 8 ans

Après ce patch, il faudra également enchainer sur l'enregistrement dans le formdata de l'info comme quoi telle valeur était vérifiée, pour indiquer ça dans le backoffice pour aider les agents dans le traitement.

#4

Mis à jour par Benjamin Dauvergne il y a presque 8 ans

Ack sur le patch SAML.

#5

Mis à jour par Benjamin Dauvergne il y a presque 8 ans

Pourquoi ne pas mettre email aussi dans verified_fields ?

#6

Mis à jour par Frédéric Péters il y a presque 8 ans

Très mauvais raison mais parce que ça gère la situation particulière de Liège, on a nom/prénom certifiés via carte d'identité mais l'email est libre.

#7

Mis à jour par Benjamin Dauvergne il y a presque 8 ans

Ack pour le second patch, je remarque juste que dans cette implémentation il est désormais impossible de faire une demande pour un tiers si les champs "demandeurs" sont pré-remplis avec les données du compte et que ces données sont certifiées; pour moi l'implémentation idéale laisserait l'utilisateur modifier la donnée (ou dire qu'il veut faire une demande pour un tiers) mais ce serait au workflow de dire si champs nom, prénom, numéro national d'identité, etc.. == ceux de form.user qui sont validés alors sauter à l'étape suivante.

#8

Mis à jour par Benjamin Dauvergne il y a presque 8 ans

Ok ack quand même dans la mesure ou le seul utilisateur de ce truc pour l'instant est IMIO.

#9

Mis à jour par Thomas Noël il y a presque 8 ans

Yep Benj, ceci étant pour les formulaires ou le bénéfiaire est différent du demandeur, mais où on doit garder l'info "vérifiée" du demandeur, on peut imaginer des formulaires avec une case à cocher "êtes-vous aussi le bénéficiaire", et si non afficher une page qui demande qui est le bénéficiaire.

(autre astuce, préfill python « =form_user_var_.... » et hop)

Donc ça passe. Je relis le patch rapidos.

#10

Mis à jour par Benjamin Dauvergne il y a presque 8 ans

Yep bien vu, j'aime bien. Reste qu'il faudrait quand même pouvoir réagir au fait qu'un champ est pré-rempli avec une information certifiée et là je ne vois rien, il faudrait une colonne f1_certified ou quelque chose dans ce genre (parce que là à part si le formulaire n'est ouvert qu'aux possesseurs d'une CNIE je ne vois pas trop ce que ça apporte au niveau du traitement de la demande à part d'avoir "readonly" sur le champ.

#11

Mis à jour par Frédéric Péters il y a presque 8 ans

Oui, c'est mon commentaire :

Après ce patch, il faudra également enchainer sur l'enregistrement dans le formdata de l'info comme quoi telle valeur était vérifiée, pour indiquer ça dans le backoffice pour aider les agents dans le traitement.

#12

Mis à jour par Thomas Noël il y a presque 8 ans

Yep, y'a une reflexion a avoir sur ces champs "readonly" qui vont arriver en frontoffice (genre ceux qui pourraient venir d'un calcul =webservice.qf qu'on veut afficher et faire valider à l'usager dans sa demande). A discuter bientôt.

Sinon Ack pour les deux patches.

Juste au cas où, mais je pense que c'est pas important là maintenant : quid du provisionning via hobo ? (genre le cas "extrême" si l'usager est re-provisionné pendant qu'il fait une demande ?).

#13

Mis à jour par Thomas Noël il y a presque 8 ans

Ah, quand même, je verrais bien une docstring ou un commentaire sur « def get_prefill_value(self, user=None): » qui précise que le retour attendu est "value (str), verified (bool)"

#14

Mis à jour par Frédéric Péters il y a presque 8 ans

Thomas Noël a écrit :

Juste au cas où, mais je pense que c'est pas important là maintenant : quid du provisionning via hobo ? (genre le cas "extrême" si l'usager est re-provisionné pendant qu'il fait une demande ?).

La nouvelle valeur sera prise à la volée, silencieusement.

#15

Mis à jour par Frédéric Péters il y a presque 8 ans

  • Statut changé de En cours à Résolu (à déployer)
commit 318cc4f0e40a34c19ae442041d1cb5013aff1780
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Mon Jun 6 23:12:05 2016 +0200

    general: support prefilling with "verified" profile fields (#12366)

et

commit e5e034188d44cc234eb1cab78c16c59d927bcb2a
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Wed Jun 29 10:32:10 2016 +0200

    saml: add verified fields in user profile (#12366)
#16

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

#17

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

  • Version cible mis à v1.48
#18

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

  • Statut changé de Résolu (à déployer) à Fermé

Formats disponibles : Atom PDF