-
Notifications
You must be signed in to change notification settings - Fork 155
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Custom redirect_status only available when a flash YAML file exist #239
Comments
Uh, sorry. Seems even with the |
@taq thanks for your report. The I was just testing Which responders version are you using? The |
Hey @carlosantoniodasilva , thanks for your answer. Using responders (3.0.1) |
@taq gotcha, so (You can also try out my branch if you want the |
Thanks 👍🏻🙂 |
I was having some issues with redirection on Firefox, getting a "Error: Form responses must redirect to another location" from turbo, and noticed that seems I needed a 302 redirect different status. As said on the README:
This would solve my problem, but I was not getting the custom redirect status working. I checked on console via
And got
:see_other
, ok, but it was still not working (on Chrome everything was fine). Then I saw that on the responders.rb fileActionController::Responder.redirect_status
is located inside theinitializer "responders.flash_responder"
block, setting thecattr_accessor :redirect_status
. As I didn't have a flash YAML file, the default one installed with the generator is foren
and I'm usingpt-BR
as my app language, I suppose theredirect_status
attribute was never set.When I created the
pt-BR
flash file, everything worked. So, I'm not sure if is just code on the wrong place, if the flash file is mandatory or I am missing something. Maybe running the app always withen
set with the file created by the generator can miss this point on other languages.Thanks!
The text was updated successfully, but these errors were encountered: