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

validation js revue et à revoir #244

Merged
fpeters merged 1 commits from wip/75724-live-validation-js into main 2023-04-18 14:29:20 +02:00
Conversation 2 Commits 1 Files Changed 2 +166 -1
fpeters commented 2023-04-17 12:25:32 +02:00
Owner
Copy Link

Le javascript de #240 avec en plus une série de commits, adaptations et/ou corrections.

Et un dernier commit XXX avec mon blocage sur ce qu'il y aurait à reprendre (et modifier plus profondément le code proposé).

Le javascript de https://git.entrouvert.org/entrouvert/wcs/pulls/240 avec en plus une série de commits, adaptations et/ou corrections. Et un dernier commit XXX avec mon blocage sur ce qu'il y aurait à reprendre (et modifier plus profondément le code proposé).
fpeters force-pushed wip/75724-live-validation-js from b7ad8fdb4f to 07f81d27b7 2023-04-17 15:08:15 +02:00 Compare
csechet added 6 commits 2023-04-17 18:14:46 +02:00
gitea/wcs/pipeline/head This commit looks good Details
2cae6d3bf7 fix formatting
csechet commented 2023-04-17 18:24:36 +02:00
Owner
Copy Link

Voilà mes propositions.

L'appel live était fait même si on avait une erreur de validation détectée en JS en amont, j'ai fixé ça.

Les modifications que j'ai faites supposent que la liste des templates pour chaque code d'erreur est exhaustive pendant la génération du HTML : on met à jour le message d'erreur uniquement.

Un peu plus généralement :

Utiliser plusieurs templates, et gérer une partie des erreurs coté JS, l'autre coté serveur me parait compliquer les choses pour sauver une requête de temps en temps. Je ne sais également pas à quel point ça sera utile d'avoir des templates différents selon les types d'erreur. Mais si ça n'est pas utilisé, mon avis serait de faire une requête pour la validation systématiquement, supprimer le support de l'API JS constraint, et n'avoir qu'un template pour afficher le message d'erreur, qui serait retourné en JSON. Ça permettrait aussi d’être sur de ne pas avoir de comportements différents entre les appels live et à la soumission du formulaire. Après je rate peut-être des choses, je n'ai pas suivi les discussions en amont.

Maintenant, ça fonctionne comme ça, ça dépend du planning, on peut en discuter plus tard et faire un autre ticket, ou pas.

Voilà mes propositions. L'appel live était fait même si on avait une erreur de validation détectée en JS en amont, j'ai fixé ça. Les modifications que j'ai faites supposent que la liste des templates pour chaque code d'erreur est exhaustive pendant la génération du HTML : on met à jour le message d'erreur uniquement. Un peu plus généralement : Utiliser plusieurs templates, et gérer une partie des erreurs coté JS, l'autre coté serveur me parait compliquer les choses pour sauver une requête de temps en temps. Je ne sais également pas à quel point ça sera utile d'avoir des templates différents selon les types d'erreur. Mais si ça n'est pas utilisé, mon avis serait de faire une requête pour la validation systématiquement, supprimer le support de l'API JS constraint, et n'avoir qu'un template pour afficher le message d'erreur, qui serait retourné en JSON. Ça permettrait aussi d’être sur de ne pas avoir de comportements différents entre les appels live et à la soumission du formulaire. Après je rate peut-être des choses, je n'ai pas suivi les discussions en amont. Maintenant, ça fonctionne comme ça, ça dépend du planning, on peut en discuter plus tard et faire un autre ticket, ou pas.
fpeters force-pushed wip/75724-live-validation-js from 2cae6d3bf7 to cd952c176f 2023-04-18 11:02:17 +02:00 Compare
fpeters force-pushed wip/75724-live-validation-js from cd952c176f to cb90ebc84e 2023-04-18 11:32:07 +02:00 Compare
fpeters commented 2023-04-18 14:13:06 +02:00
Author
Owner
Copy Link

C'est repris en réunissant tous les commits js et en rebasant sur la branche de #241 pour moi ça peut être relu, et on pourrait assez bien considérer que Corentin et moi avons relu.

C'est repris en réunissant tous les commits js et en rebasant sur la branche de https://git.entrouvert.org/entrouvert/wcs/pulls/241 pour moi ça peut être relu, et on pourrait assez bien considérer que Corentin et moi avons relu.
fpeters changed title from WIP: validation js revue et à revoir to validation js revue et à revoir 2023-04-18 14:13:10 +02:00
fpeters force-pushed wip/75724-live-validation-js from cb90ebc84e to 2d784e000a 2023-04-18 14:23:16 +02:00 Compare
lguerin approved these changes 2023-04-18 14:23:57 +02:00
fpeters merged commit 2d784e000a into main 2023-04-18 14:29:20 +02:00
fpeters deleted branch wip/75724-live-validation-js 2023-04-18 14:29:20 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
lguerin
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/wcs#244
No description provided.
Delete Branch "wip/75724-live-validation-js"

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?