entrouvert
/
authentic
16
0
Fork 0
Code Issues Pull Requests 35 Releases Activity

Changement de numéro de téléphone sans avoir à retaper son mot de passe (#79891) #111

Merged
pmarillonnet merged 1 commits from wip/79891-phone-change-no-password-form into main 2023-08-16 16:32:44 +02:00
Conversation 1 Commits 1 Files Changed 4 +189 -6
pmarillonnet commented 2023-08-08 15:14:08 +02:00
Owner
Copy Link
No description provided.
pmarillonnet force-pushed wip/79891-phone-change-no-password-form from 1069bbbd08 to b80bb41ea5 2023-08-10 11:56:43 +02:00 Compare
pmarillonnet force-pushed wip/79891-phone-change-no-password-form from e8700b870b to 3975c0dbfa 2023-08-10 13:43:41 +02:00 Compare
pmarillonnet force-pushed wip/79891-phone-change-no-password-form from 3975c0dbfa to e07eba483a 2023-08-10 14:08:23 +02:00 Compare
pmarillonnet changed title from WIP: Changement de numéro de téléphone sans avoir à retaper son mot de passe (#79891) to Changement de numéro de téléphone sans avoir à retaper son mot de passe (#79891) 2023-08-14 10:23:29 +02:00
pmarillonnet commented 2023-08-14 10:24:42 +02:00
Author
Owner
Copy Link

Je propose de déjà passer cela avec des tests qui moquent can_validate_with_password et has_recent_authentication, qu’on pourrait tester à part dans un prochain ticket, en moquant le contenu de la session de l’usager etc. Ça me paraît bien de déjà dans l’immédiat corriger le code tel que ce qui est fait dans cette PR.

Je propose de déjà passer cela avec des tests qui moquent `can_validate_with_password` et `has_recent_authentication`, qu’on pourrait tester à part dans un prochain ticket, en moquant le contenu de la session de l’usager etc. Ça me paraît bien de déjà dans l’immédiat corriger le code tel que ce qui est fait dans cette PR.
pmarillonnet force-pushed wip/79891-phone-change-no-password-form from e07eba483a to d5d41766d0 2023-08-14 10:25:37 +02:00 Compare
vdeniaud approved these changes 2023-08-16 11:54:28 +02:00
vdeniaud left a comment
Owner
Copy Link

Top d'avoir ces nouveaux tests

Top d'avoir ces nouveaux tests
pmarillonnet force-pushed wip/79891-phone-change-no-password-form from d5d41766d0 to 08d3dcf0da 2023-08-16 12:05:57 +02:00 Compare
pmarillonnet merged commit 08d3dcf0da into main 2023-08-16 16:32:44 +02:00
pmarillonnet deleted branch wip/79891-phone-change-no-password-form 2023-08-16 16:32:44 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
vdeniaud
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/authentic#111
No description provided.
Delete Branch "wip/79891-phone-change-no-password-form"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?