Outils pour améliorer CVT multiétape #152

Closed
maieul wants to merge 4 commits from maieul:cvt_multi_avance_rapide into 3.2
maieul commented 2 weeks ago

Cette PR vise à permettre l'avance rapide sur du CVT multi étapes. Par exemple passer de l'étape 1 à l'étape 3 sans l'étape 2.

La vérification de l'étape 2 est toujours appellée dans ce cas, mais c'est au formulaire qui propose l'avance rapide de ne pas vérifier l'étape 2 si pertinent.

Le but au départ est de permettre de "sauter" des étapes dans un formulaire formidable, via afficher_si (j'ouvrirai la PR sur saisies bientôt).

Concrètement cette PR indique la branche 3.2, car j'ai du reporter un commit de @Cerdic avant de faire mon boulot. Si elle est acceptée, il faudra reporter mes 3 commits dans master.

Ces 3 commits :

  1. Permettre à une fonction verifier de faire un `set_request('etape_dans_le_futur') pour sauter directement à l'étape dans le futur
  2. Fournir plus d'infos au pipeline saisies_verifier_etapes pour faciliter le travail (bien qu'on pourrait récuperer toutes ces infos via _request()
  3. Permet lorsqu'on fait une avance rapide à l'étape N de ne pas tester cette étape N, mais uniquement les étapes antérieurs à N.

Testé avec des formulaires formidables complexes + en tentant d'imiter un·e fraudeur·euse qui enverrait depuis un navigateur un 'aller_a_etape' pour faire une avance rapide. Le système teste bien les étapes intermédiaires. Ainsi, la contrainte "c'est bien au formulaire de décider le cas échéant de ne pas tester certaines étapes" est correctement prise en compte.

Mais d'autres avis/regards sur le code sont les bienvenus, car j'ai pas mal galéré pour trouver le bon équilibre, et j'ai peut être raté un truc.

Cette PR vise à permettre l'avance rapide sur du CVT multi étapes. Par exemple passer de l'étape 1 à l'étape 3 sans l'étape 2. La vérification de l'étape 2 est toujours appellée dans ce cas, mais c'est au formulaire qui propose l'avance rapide de ne pas vérifier l'étape 2 si pertinent. Le but au départ est de permettre de "sauter" des étapes dans un formulaire formidable, via afficher_si (j'ouvrirai la PR sur saisies bientôt). Concrètement cette PR indique la branche 3.2, car j'ai du reporter un commit de @Cerdic avant de faire mon boulot. Si elle est acceptée, il faudra reporter mes 3 commits dans master. Ces 3 commits : 1. Permettre à une fonction verifier de faire un `set_request('etape_dans_le_futur') pour sauter directement à l'étape dans le futur 2. Fournir plus d'infos au pipeline `saisies_verifier_etapes` pour faciliter le travail (bien qu'on pourrait récuperer toutes ces infos via _request() 3. Permet lorsqu'on fait une avance rapide à l'étape N de ne pas tester cette étape N, mais uniquement les étapes antérieurs à N. Testé avec des formulaires formidables complexes + en tentant d'imiter un·e fraudeur·euse qui enverrait depuis un navigateur un 'aller_a_etape' pour faire une avance rapide. Le système teste bien les étapes intermédiaires. Ainsi, la contrainte "c'est bien au formulaire de décider le cas échéant de ne pas tester certaines étapes" est correctement prise en compte. Mais d'autres avis/regards sur le code sont les bienvenus, car j'ai pas mal galéré pour trouver le bon équilibre, et j'ai peut être raté un truc.
maieul added 4 commits 2 weeks ago
maieul force-pushed cvt_multi_avance_rapide from 5e9764a666 to f894819818 2 weeks ago
b_b commented 2 weeks ago
Owner

Pas d'avis sur le code (pour l'instant) que je n'utilise pas, mais sur le principe ça me chagrine d'envoyer dans la 3.2 pour reporter ensuite sur master :\

Pas d'avis sur le code (pour l'instant) que je n'utilise pas, mais sur le principe ça me chagrine d'envoyer dans la 3.2 pour reporter ensuite sur master :\
Poster

bah c'est surtout que cela aurait été compliqué de faire dans l'autres sens, ou alors fallait penser à prendre aussi le commit de cedric. Là l'avantage c'est que la PR est plus explicite

bah c'est surtout que cela aurait été compliqué de faire dans l'autres sens, ou alors fallait penser à prendre aussi le commit de cedric. Là l'avantage c'est que la PR est plus explicite
Owner

cherry-pické en 3.3 via ca8cff7c7f 82992b9219 2ef287aa50
et reporté en 3.2 via 3789759973 240ca1577d bcc3c36067 82cf0ba8d7

cherry-pické en 3.3 via ca8cff7c7f 82992b9219 2ef287aa50 et reporté en 3.2 via 3789759973 240ca1577d bcc3c36067 82cf0ba8d7
Poster

mille merci !

mille merci !
cerdic closed this pull request 1 week ago
maieul deleted branch cvt_multi_avance_rapide 1 week ago
This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
3 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.