Project

General

Profile

Development #55836

auth_fc: ignorer les attributs requis mais non éditable lors de la vérification de remplissage des attributs

Added by Benjamin Dauvergne 2 months ago. Updated 2 months ago.

Status:
Solution déployée
Priority:
Normal
Category:
-
Target version:
-
Start date:
26 Jul 2021
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

Description

Voir #55835.


Files

Associated revisions

Revision 0091342a (diff)
Added by Benjamin Dauvergne 2 months ago

auth_fc: refactor missing required attributes check (#55836)

Revision 61fa179c (diff)
Added by Benjamin Dauvergne 2 months ago

auth_fc: ignore non user_editable attributes when checking missing required attributes (#55836)

Revision 1ed6387c (diff)
Added by Benjamin Dauvergne 2 months ago

auth_fc: only check required attributes on user registration (#55836)

History

#2

Updated by Benjamin Dauvergne 2 months ago

  • Assignee set to Benjamin Dauvergne
#3

Updated by Benjamin Dauvergne 2 months ago

J'ai ajouté une condition supplémentaire: on ne fait cette vérification que sur une création de compte; pas de raison d'embêter les gens enregistré par mail qui lient leur compte existant avec FC.

#4

Updated by Valentin Deniaud 2 months ago

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

Je trouve qu'il manque un bout de test pour montrer que les attributs avec user_editable=False sont bien masqués, et même un autre pour distinguer cette histoire de création/liaision, mais soit.

#5

Updated by Benjamin Dauvergne 2 months ago

  • Status changed from Solution validée to Résolu (à déployer)
commit 1ed6387c6234eb5a5844a8168bb613cd19feeb63
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Mon Jul 26 16:38:32 2021 +0200

    auth_fc: only check required attributes on user registration (#55836)

commit 61fa179c7719c806dab1924af0ff8f6bdf544936
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Mon Jul 26 16:36:56 2021 +0200

    auth_fc: ignore non user_editable attributes when checking missing required attributes (#55836)

commit 0091342a6b5cc296d1c3d95b1626dd70b3e268ad
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Mon Jul 26 16:36:30 2021 +0200

    auth_fc: refactor missing required attributes check (#55836)
#6

Updated by Frédéric Péters 2 months ago

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

Also available in: Atom PDF