Project

General

Profile

Development #66053

csv_import: stocker une information de source de vérification des attributs de l’usager

Added by Paul Marillonnet 8 months ago. Updated 6 days ago.

Status:
En cours
Priority:
Normal
Category:
-
Target version:
-
Start date:
08 June 2022
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

Description

On a la base nécessaire dans #65612. Dans l’import csv il y a la possibilité de désigner des attributs usagers comme vérifiés.
On peut alors stocker le fait que la vérification provient de l’import csv.


Files


Related issues

Related to Authentic 2 - Development #65612: stocker l’origine (ou les multiples origines) ayant mené à la vérification d’un attribut donné pour un usagerSolution déployée23 May 2022

Actions

History

#1

Updated by Paul Marillonnet 8 months ago

  • Related to Development #65612: stocker l’origine (ou les multiples origines) ayant mené à la vérification d’un attribut donné pour un usager added
#2

Updated by Paul Marillonnet 8 months ago

#3

Updated by Paul Marillonnet 8 months ago

  • Assignee set to Benjamin Dauvergne

(Basé sur #65612, bien sûr.)

#4

Updated by Paul Marillonnet 8 months ago

  • Assignee changed from Benjamin Dauvergne to Paul Marillonnet
#5

Updated by Benjamin Dauvergne 6 months ago

  • Status changed from Solution proposée to Information nécessaire

Reproposer quand #65612 sera terminé. Mais je note déjà que la source est posée que l'attribut soit vérifié ou pas, c'est bizarre.

#6

Updated by Paul Marillonnet about 2 months ago

  • Status changed from Information nécessaire to En cours

Benjamin Dauvergne a écrit :

Reproposer quand #65612 sera terminé.

Ok, #65612 en solution proposée.

Mais je note déjà que la source est posée que l'attribut soit vérifié ou pas, c'est bizarre.

Ah, une erreur de ma part, je regarde.

#7

Updated by Paul Marillonnet about 2 months ago

Ok, version corrigée.

Dans 0002 un peu plus de tuyauterie car il faut gérer le cas où la valeur de l’attribut déjà existant est la même mais il n’est pas encore vérifié.

Et avant ça 0001 trivial qui facilite un peu l’affaire, en stockant directement dans l’entête l’objet attribut lié (au lieu d’un simple flag indiquant que la valeur est à aller écrire dans un attribut).

#8

Updated by Benjamin Dauvergne 15 days ago

J'ai rebasé (3 lignes de modifications dans les tests au niveau de CsvField "phone" qui est devenu un attribut).

#9

Updated by Benjamin Dauvergne 13 days ago

  • Status changed from Solution proposée to En cours

J'ai corrigé les tests et UserCsvImporter pour que .run() retourne False pour n'importe quelle erreur, ensuite il y a un souci maintenant que phone est à la fois un attribut et une propriété de la classe CustomUser qui fait que le statut vérifié n'apparaît pas dans l'objet AttributeValue (parce que header.field is True et donc header.attribute n'est jamais ni défini ni utilisé).

Des choses à revoir donc.

#10

Updated by Paul Marillonnet 7 days ago

Benjamin Dauvergne a écrit :

[…] il y a un souci maintenant que phone est à la fois un attribut et une propriété de la classe CustomUser qui fait que le statut vérifié n'apparaît pas dans l'objet AttributeValue (parce que header.field is True et donc header.attribute n'est jamais ni défini ni utilisé).

Ok, je vais regarder ce que tu as ajouté dans cette nouvelle branche et je corrige ce souci sur l’état de vérification du téléphone du à l’ambiguïté entre attribut de profil étendu et attribut immédiat de la classe User custom.

#11

Updated by Paul Marillonnet 6 days ago

Et j’ai poussé les correctifs nécessaires dans une troisième branche au dessus de la tienne, je vais ranger proprement en commits distincts et proposer à nouveau les patches ici.

Also available in: Atom PDF