Skip to content

🚀 Fix error display at step 2 of rdv wizard

Vincent Agnano requested to merge fix-step2-error into production

Created by: n-b

fixes #1880 (closed)

L’erreur Ă©tait liĂ©e au fait que l’on se basait sur le path de la requĂȘte courante (request.params.deep_dup). Le partial step2 est affichĂ©, selon le cas, soit par Admin::RdvWizardStepsController#new, soit par Admin::RdvWizardStepsController#create. Dans les deux cas, la façon de passer les paramĂštres est lĂ©gĂšrement diffĂ©rente.

Il y a quelque chose de bizarre avec ce formulaire: L’action #create est utilisĂ©e sur des GET, et pas sur des POST, ce qui est relativement cohĂ©rent dans la mesure oĂč rien n’est rĂ©ellement crĂ©Ă© en base; cependant, #create redirige bien vers #new (au step suivant) en cas de succĂšs, mais render directement l’étape actuelle en cas d’échec. Dans la mesure oĂč dans ce wizard, tout est stateless, on pourrait n’avoir qu’une action d’un step Ă  l’autre avant le “submit” final.

Checklist avant review:

  • reparcourir le code rapidement pour voir les problĂšmes Ă©vidents (fichiers touchĂ©s inutilement, debug logs qui trainent
).
  • Tester la fonctionnalitĂ© sur la review app

Merge request reports