auth tél : à la création de compte, sur un numéro de téléphone déclaré dans un format erroné, afficher un exemple de numéro attendu (#88045) #273

Merged
pmarillonnet merged 1 commits from wip/88045-phone-authn-account-creation-example-of-expected-input-value into main 2024-04-02 11:45:27 +02:00
Owner
No description provided.
pmarillonnet added 1 commit 2024-03-12 11:05:53 +01:00
pmarillonnet added 1 commit 2024-03-12 11:44:59 +01:00
gitea/authentic/pipeline/head Something is wrong with the build of this commit Details
bb6063286d
settings: provide local example formats for phone numbers (#88045)
pmarillonnet force-pushed wip/88045-phone-authn-account-creation-example-of-expected-input-value from bb6063286d to 266d1f39d0 2024-03-12 11:50:12 +01:00 Compare
pmarillonnet added 1 commit 2024-03-12 12:07:19 +01:00
pmarillonnet force-pushed wip/88045-phone-authn-account-creation-example-of-expected-input-value from c1722811b8 to 0673a7b984 2024-03-12 12:08:36 +01:00 Compare
pmarillonnet force-pushed wip/88045-phone-authn-account-creation-example-of-expected-input-value from 0673a7b984 to 56b67016d4 2024-03-12 12:10:30 +01:00 Compare
pmarillonnet force-pushed wip/88045-phone-authn-account-creation-example-of-expected-input-value from 56b67016d4 to 518db3a36d 2024-03-12 14:06:38 +01:00 Compare
pmarillonnet force-pushed wip/88045-phone-authn-account-creation-example-of-expected-input-value from 518db3a36d to 1c86747fc3 2024-03-12 15:51:27 +01:00 Compare
pmarillonnet force-pushed wip/88045-phone-authn-account-creation-example-of-expected-input-value from 1c86747fc3 to d55f704d08 2024-03-12 16:14:15 +01:00 Compare
pmarillonnet force-pushed wip/88045-phone-authn-account-creation-example-of-expected-input-value from d55f704d08 to a0d632f387 2024-03-12 17:05:45 +01:00 Compare
pmarillonnet changed title from WIP: auth tél : à la création de compte, sur un numéro de téléphone déclaré dans un format erroné, afficher un exemple de numéro attendu (#88045) to auth tél : à la création de compte, sur un numéro de téléphone déclaré dans un format erroné, afficher un exemple de numéro attendu (#88045) 2024-03-12 17:05:50 +01:00
tnoel requested changes 2024-03-12 17:51:16 +01:00
Dismissed
tnoel left a comment
Owner

Un peu comme avec example.com, pour les exemples de numéro il faut plutôt se baser sur les numéros non attribuables, cf https://fr.wikipedia.org/wiki/Plan_de_num%C3%A9rotation_en_France section « Les numéros réservés pour les œuvres audiovisuelles ». Je ne sais pas si un équivalent existe en Belgique.

Un peu comme avec example.com, pour les exemples de numéro il faut plutôt se baser sur les numéros non attribuables, cf https://fr.wikipedia.org/wiki/Plan_de_num%C3%A9rotation_en_France section « Les numéros réservés pour les œuvres audiovisuelles ». Je ne sais pas si un équivalent existe en Belgique.
pmarillonnet force-pushed wip/88045-phone-authn-account-creation-example-of-expected-input-value from a0d632f387 to d19826b922 2024-03-19 16:38:13 +01:00 Compare
pmarillonnet force-pushed wip/88045-phone-authn-account-creation-example-of-expected-input-value from d19826b922 to ca5db9315e 2024-03-20 09:44:30 +01:00 Compare
pmarillonnet added 1 commit 2024-03-20 09:50:37 +01:00
Author
Owner

Un peu comme avec example.com, pour les exemples de numéro il faut plutôt se baser sur les numéros non attribuables, cf https://fr.wikipedia.org/wiki/Plan_de_num%C3%A9rotation_en_France section « Les numéros réservés pour les œuvres audiovisuelles ».

Ok très bien, je connaissais l’équivalent états-unien sans savoir que ça existait aussi en France, nickel.

Je ne sais pas si un équivalent existe en Belgique.

Rapide recherche pour trouver quelque chose de similaire là-bas, rien qui ressort, je vais laisser l’exemple tel quel.

> Un peu comme avec example.com, pour les exemples de numéro il faut plutôt se baser sur les numéros non attribuables, cf https://fr.wikipedia.org/wiki/Plan_de_num%C3%A9rotation_en_France section « Les numéros réservés pour les œuvres audiovisuelles ». Ok très bien, je connaissais l’équivalent états-unien sans savoir que ça existait aussi en France, nickel. > Je ne sais pas si un équivalent existe en Belgique. Rapide recherche pour trouver quelque chose de similaire là-bas, rien qui ressort, je vais laisser l’exemple tel quel.
pmarillonnet force-pushed wip/88045-phone-authn-account-creation-example-of-expected-input-value from e8fb93a0ec to 6bc73621fa 2024-03-20 09:56:22 +01:00 Compare
Author
Owner

Un peu comme avec example.com, pour les exemples de numéro il faut plutôt se baser sur les numéros non attribuables, cf https://fr.wikipedia.org/wiki/Plan_de_num%C3%A9rotation_en_France section « Les numéros réservés pour les œuvres audiovisuelles ».

Ok très bien, je connaissais l’équivalent états-unien sans savoir que ça existait aussi en France, nickel.

Je viens de corriger l’exemple de numéro français pour y placer un numéro de cette plage réservée. Merci pour le tuyau.

> > Un peu comme avec example.com, pour les exemples de numéro il faut plutôt se baser sur les numéros non attribuables, cf https://fr.wikipedia.org/wiki/Plan_de_num%C3%A9rotation_en_France section « Les numéros réservés pour les œuvres audiovisuelles ». > > Ok très bien, je connaissais l’équivalent états-unien sans savoir que ça existait aussi en France, nickel. Je viens de corriger l’exemple de numéro français pour y placer un numéro de cette plage réservée. Merci pour le tuyau.
pmarillonnet requested review from tnoel 2024-03-20 10:04:42 +01:00
tnoel requested changes 2024-03-21 11:39:27 +01:00
Dismissed
tnoel left a comment
Owner

J'aurais dû mieux relire la première fois. Dis moi si tu trouves ma remarque déplacée, je comprendrais (on est sur du détail, mais bon, comme c'est affiché aux gens, les détails comptent).

J'aurais dû mieux relire la première fois. Dis moi si tu trouves ma remarque déplacée, je comprendrais (on est sur du détail, mais bon, comme c'est affiché aux gens, les détails comptent).
@ -164,2 +170,2 @@
' {code} country code ({country}).'
).format(code=settings.DEFAULT_COUNTRY_CODE, country=default_country)
f'Phone number must be dialable from {conf[1]} '
f'(e.g. format {conf[2]}, for instance a number like {conf[3]}).'
Owner

En mode tatasse je me dis que ça fait beaucoup de mots dans la parenthèse, qu'on pourrait aller à l'essentiel et juste

   f'Phone number must be dialable from {conf[1]} (e.g. {conf[3]})'.

et de fait supprimer la notion de example_format

En mode tatasse je me dis que ça fait beaucoup de mots dans la parenthèse, qu'on pourrait aller à l'essentiel et juste ``` f'Phone number must be dialable from {conf[1]} (e.g. {conf[3]})'. ``` et de fait supprimer la notion de example_format
Author
Owner

Ok, ça ma va, j’ai poussé un commit à squasher qui retire cette notion d’example_format.

J’avais toutefois cru comprendre que les bonnes pratiques d’accessibilité imposaient de préciser à la fois le format attendu et un exemple de valeur conforme à ce format. Je vais demander là confirmation à un dev front que la PR peut passer dans cet état, je la repasse en “wip” en attendant.

Ok, ça ma va, j’ai poussé un commit à squasher qui retire cette notion d’`example_format`. J’avais toutefois cru comprendre que les bonnes pratiques d’accessibilité imposaient de préciser à la fois le format attendu et un exemple de valeur conforme à ce format. Je vais demander là confirmation à un dev front que la PR peut passer dans cet état, je la repasse en “wip” en attendant.
Author
Owner

@tjund me dit que c’est ok, je squash le tout.

@tjund me dit que c’est ok, je squash le tout.
pmarillonnet force-pushed wip/88045-phone-authn-account-creation-example-of-expected-input-value from 6bc73621fa to 4c28d43e0f 2024-03-21 11:44:46 +01:00 Compare
pmarillonnet added 1 commit 2024-03-21 11:52:12 +01:00
gitea/authentic/pipeline/head This commit looks good Details
e9df84022e
[wip/tosquash] remove example_format
pmarillonnet changed title from auth tél : à la création de compte, sur un numéro de téléphone déclaré dans un format erroné, afficher un exemple de numéro attendu (#88045) to WIP: auth tél : à la création de compte, sur un numéro de téléphone déclaré dans un format erroné, afficher un exemple de numéro attendu (#88045) 2024-03-21 11:54:44 +01:00
pmarillonnet force-pushed wip/88045-phone-authn-account-creation-example-of-expected-input-value from e9df84022e to 36bdbb081f 2024-03-28 15:54:01 +01:00 Compare
pmarillonnet changed title from WIP: auth tél : à la création de compte, sur un numéro de téléphone déclaré dans un format erroné, afficher un exemple de numéro attendu (#88045) to auth tél : à la création de compte, sur un numéro de téléphone déclaré dans un format erroné, afficher un exemple de numéro attendu (#88045) 2024-03-28 15:58:08 +01:00
pmarillonnet requested review from tnoel 2024-03-28 15:58:15 +01:00
tnoel approved these changes 2024-03-29 16:11:25 +01:00
pmarillonnet force-pushed wip/88045-phone-authn-account-creation-example-of-expected-input-value from 36bdbb081f to 6a7a4814a9 2024-04-02 11:37:26 +02:00 Compare
pmarillonnet merged commit 6a7a4814a9 into main 2024-04-02 11:45:27 +02:00
pmarillonnet deleted branch wip/88045-phone-authn-account-creation-example-of-expected-input-value 2024-04-02 11:45:27 +02:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
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#273
No description provided.