entrouvert/wcs
entrouvert
/
wcs
14
1
Fork 0
Code Issues Pull Requests 35 Releases Activity

misc: update default validation error messages (#76350) #693

Merged
fpeters merged 2 commits from wip/76350-new-validation-error-messages into main 2023-09-19 13:24:21 +02:00
Conversation 0 Commits 2 Files Changed 9 +161 -87
fpeters commented 2023-09-18 18:25:32 +02:00
Owner
Copy Link
No description provided.
fpeters force-pushed wip/76350-new-validation-error-messages from 95c31045ad to 8f8bbb5f2c 2023-09-18 18:34:23 +02:00 Compare
fpeters force-pushed wip/76350-new-validation-error-messages from 8f8bbb5f2c to 090f47f315 2023-09-18 18:42:56 +02:00 Compare
fpeters force-pushed wip/76350-new-validation-error-messages from 090f47f315 to a36472175c 2023-09-18 18:48:19 +02:00 Compare
fpeters force-pushed wip/76350-new-validation-error-messages from a36472175c to 27dc0c928e 2023-09-18 18:51:27 +02:00 Compare
fpeters force-pushed wip/76350-new-validation-error-messages from 27dc0c928e to 8bbc141dc2 2023-09-18 19:05:04 +02:00 Compare
fpeters changed title from WIP: misc: update default validation error messages (#76350) to misc: update default validation error messages (#76350) 2023-09-18 19:22:03 +02:00
fpeters force-pushed wip/76350-new-validation-error-messages from 8bbc141dc2 to 78b9f6b80a 2023-09-18 19:35:48 +02:00 Compare
tnoel approved these changes 2023-09-19 11:32:33 +02:00
tnoel left a comment
Owner
Copy Link

J'ai posé quelques remarques sans vraie importance, je valide quand même...

J'ai posé quelques remarques sans vraie importance, je valide quand même...
wcs/locale/fr/LC_MESSAGES/django.po Outdated
@ -6617,0 +6638,4 @@
"characters, for example FR7600001000010000000000101."
msgstr ""
"Veuillez saisir un IBAN composé d'au moins 14 et de maximum 34 caractères "
"alphanumériques, par exemple FR7600001000010000000000101."
tnoel commented 2023-09-19 10:49:58 +02:00
Owner
Copy Link

J'aurais pas ajouté «alphanumériques» parce que personne ne sait ce que ça veut dire.

J'aurais pas ajouté «alphanumériques» parce que personne ne sait ce que ça veut dire.
fpeters commented 2023-09-19 12:20:26 +02:00
Author
Owner
Copy Link

J'ai repris le message d'Anaïs.

J'ai repris le message d'Anaïs.
wcs/locale/fr/LC_MESSAGES/django.po Outdated
@ -6657,1 +6681,3 @@
msgstr "date invalide : la date doit être postérieure ou égale à %s"
msgid "You should enter a valid date. It must be on or after %s."
msgstr ""
"Veuillez saisir une date valide. Elle doit être postérieure ou égale à %s"
tnoel commented 2023-09-19 10:55:23 +02:00
Owner
Copy Link

Ca va donner un truc comme « Elle doit être postérieure ou égale à 31/12/2023 » ?

Je trouve que ça serait plus compréhensible d'avoir « Elle doit être avant le 31/12/2023 » (parce que "postérieure/antérieure" moi ça me demande 10 secondes de réflexion à chaque lecture). Bon, après, je ne sais pas comment dire le "ou égale", « Elle doit être égale à, ou avant le 31/12/2023 » c'est affreux.

En tout cas sans doute remplacer le « à %s » par « le %s » ?

Ca va donner un truc comme « Elle doit être postérieure ou égale à 31/12/2023 » ? Je trouve que ça serait plus compréhensible d'avoir « Elle doit être avant le 31/12/2023 » (parce que "postérieure/antérieure" moi ça me demande 10 secondes de réflexion à chaque lecture). Bon, après, je ne sais pas comment dire le "ou égale", « Elle doit être égale à, ou avant le 31/12/2023 » c'est affreux. En tout cas sans doute remplacer le « à %s » par « le %s » ?
fpeters commented 2023-09-19 12:21:56 +02:00
Author
Owner
Copy Link

Le ticket ne change pas cette partie des messages; j'en verrais bien un autre.

Le ticket ne change pas cette partie des messages; j'en verrais bien un autre.
fpeters commented 2023-09-19 12:33:57 +02:00
Author
Owner
Copy Link

ou peut-être vite fait, ces idées :

  • Elle doit être après le DATE (inclus) ?
  • Elle doit être supérieure ou égale à DATE ?
ou peut-être vite fait, ces idées : * Elle doit être après le DATE (inclus) ? * Elle doit être supérieure ou égale à DATE ?
wcs/qommon/form.py
@ -1289,2 +1295,3 @@
'title': _('NIR (France)'),
'error_message': _('Invalid NIR'),
'error_message': _(
'You should enter a valid 15-digits social security number, '
tnoel commented 2023-09-19 11:30:52 +02:00
Owner
Copy Link

Subtilité, c'est n'est pas un nombre (15-digits) car le département est 2A ou 2B pour la région Corse... Mais bon, sans doute que les personnes nées en corse le savent, partout on demande les "15 chiffres"... Laissons ainsi.

Subtilité, c'est n'est pas un nombre (15-digits) car le département est 2A ou 2B pour la région Corse... Mais bon, sans doute que les personnes nées en corse le savent, partout on demande les "15 chiffres"... Laissons ainsi.
fpeters commented 2023-09-19 12:22:12 +02:00
Author
Owner
Copy Link

J'ai repris le message proposé par Anaïs.

J'ai repris le message proposé par Anaïs.
fpeters force-pushed wip/76350-new-validation-error-messages from 78b9f6b80a to f3f7d0f273 2023-09-19 12:35:03 +02:00 Compare
fpeters merged commit 4224369610 into main 2023-09-19 13:24:21 +02:00
fpeters deleted branch wip/76350-new-validation-error-messages 2023-09-19 13:24:21 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
tnoel
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/wcs#693
No description provided.
Delete Branch "wip/76350-new-validation-error-messages"

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?