- Issue created by @chlandin
- 🇧🇪Belgium dieterholvoet Brussels
Maybe name the new option 'Confirmation message' instead of 'Redirect response', because the latter is an existing technical term and might be confusing. You can see this in the controller, the line below your change contains
RedirectResponse
:)Apart from that, looks good! If you can just change that I'll test and merge.
- 🇮🇳India prem suthar Ahemdabad- Gujrat , Jodhpur - Rajsthan
@dieterholvoet i have reviewed the MR!9 Its working perfectly as per issue requirement .
i have attached the Screenshots also. I Move this issue to RTBC. - 🇧🇪Belgium dieterholvoet Brussels
@prem suthar interesting, you changed the completely wrong setting in
issue_review-1.png
, but it still resulted in the correct confirmation message inissue_review-2.png
? That's some magic! - 🇧🇪Belgium dieterholvoet Brussels
I actually tested the MR, and all seems to work as expected. Thanks for your contribution @chlandin!
-
dieterholvoet →
committed 0dc3566f on 1.0.x authored by
chlandin →
Issue #3500343 by chlandin, dieterholvoet: Add the possibility to set...
-
dieterholvoet →
committed 0dc3566f on 1.0.x authored by
chlandin →