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

manager: prevent phone id duplicates at user edition time (#85276) #227

Closed
pmarillonnet wants to merge 4 commits from wip/85276-bo-phone-identifier-duplicate-prevention into wip/82737-authn-tel-post-registration-account-selection-buggy-form
pull from: wip/85276-bo-phone-identifier-duplicate-prevention
merge into: entrouvert:wip/82737-authn-tel-post-registration-account-selection-buggy-form
Conversation 2 Commits 4 Files Changed 10 +368 -12
pmarillonnet commented 2024-01-04 16:37:38 +01:00
Owner
Copy Link
No description provided.
pmarillonnet force-pushed wip/85276-bo-phone-identifier-duplicate-prevention from d631037c9d to e5a3ddb18e 2024-01-11 11:09:24 +01:00 Compare
pmarillonnet force-pushed wip/85276-bo-phone-identifier-duplicate-prevention from e5a3ddb18e to c9e1b93e38 2024-01-11 11:14:33 +01:00 Compare
pmarillonnet force-pushed wip/85276-bo-phone-identifier-duplicate-prevention from c9e1b93e38 to f91ea63636 2024-01-11 11:21:20 +01:00 Compare
pmarillonnet force-pushed wip/85276-bo-phone-identifier-duplicate-prevention from f91ea63636 to b64ed171e1 2024-01-11 11:27:36 +01:00 Compare
pmarillonnet force-pushed wip/85276-bo-phone-identifier-duplicate-prevention from b64ed171e1 to 2ee143c362 2024-01-11 12:23:05 +01:00 Compare
pmarillonnet force-pushed wip/85276-bo-phone-identifier-duplicate-prevention from 2ee143c362 to 92350ed639 2024-01-11 14:18:07 +01:00 Compare
pmarillonnet changed title from WIP: manager: prevent phone id duplicates at user edition time (#85276) to manager: prevent phone id duplicates at user edition time (#85276) 2024-01-11 14:22:32 +01:00
bdauvergne approved these changes 2024-01-11 17:08:54 +01:00
bdauvergne left a comment
Owner
Copy Link

Ok mais simplifier le style.

Ok mais simplifier le style.
src/authentic2/manager/forms.py
@ -175,0 +193,4 @@
raise ValidationError(_('This phone number identifier is already used.'))
elif getattr(self.instance.ou, 'phone_is_unique', False):
if (
other_owners := AttributeValue.objects.filter(
bdauvergne commented 2024-01-11 17:08:36 +01:00
Owner
Copy Link

T'abuses un peu de l'opérateur walrus là, en plus c'est moins lisible comme ça.

T'abuses un peu de l'opérateur walrus là, en plus c'est moins lisible comme ça.
bdauvergne commented 2024-01-11 17:09:18 +01:00
Owner
Copy Link

T'abuses un peu de l'opérateur walrus là, en plus c'est moins lisible comme ça (trop d'indentation).

> T'abuses un peu de l'opérateur walrus là, en plus c'est moins lisible comme ça (trop d'indentation).
pmarillonnet commented 2024-01-11 17:17:04 +01:00
Author
Owner
Copy Link

T'abuses un peu de l'opérateur walrus là, en plus c'est moins lisible comme ça.

Ok pas de souci, je vais revoir cette partie :)

> T'abuses un peu de l'opérateur walrus là, en plus c'est moins lisible comme ça. Ok pas de souci, je vais revoir cette partie :)
nroche approved these changes 2024-01-11 17:14:35 +01:00
nroche commented 2024-01-11 17:18:43 +01:00
Owner
Copy Link

(Oups Benjamin est passé avant moi. Je ne l'ai pas formulée mais je me suis fait la même remarque que lui sur l'opérateur Warlus)

Et aussi, tu as laissé "[wip]" dans le message du commit.

(Oups Benjamin est passé avant moi. Je ne l'ai pas formulée mais je me suis fait la même remarque que lui sur l'opérateur Warlus) Et aussi, tu as laissé "[wip]" dans le message du commit.
pmarillonnet closed this pull request 2024-01-15 08:38:19 +01:00
pmarillonnet commented 2024-01-15 08:56:05 +01:00
Author
Owner
Copy Link

Oups, imbrication de PR, celle-ci a été fermée par mégarde, je vais en git-push le contenu, à l’ancienne.

Oups, imbrication de PR, celle-ci a été fermée par mégarde, je vais en git-push le contenu, à l’ancienne.
All checks were successful
gitea/authentic/pipeline/head This commit looks good
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
bdauvergne
nroche
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
3 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#227
No description provided.
Delete Branch "wip/85276-bo-phone-identifier-duplicate-prevention"

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?