Set "Turbolinks-Location" header when redirecting from routes

[artemave]

So that turbolinks in the browser updates the url.

turbolinks/turbolinks#169

[Fudoshiki]

merge?

[kaspergrubbe]

I think I am still seeing this issue in 5.2.0.

[artemave]

@rafaelfranca merge?

[mensfeld]

Can confirm in 6.0.0

[emilkarl]

Is this supposed to be merged or is there other ways to solve this now?

[artemave]

@sstephenson merge?

[hgeorgilas]

Is this going to be merged at some point?

[tomclose]

I've just run into this too. The current behaviour isn't compatible with stimulus reflex: if you use redirect(...) to a page using stimulus reflex, you'll run into errors Reflex SearchReflex#perform failed: No route matches ....

It would be really handy if turbolinks changed the urls when using redirect from the routes file. @rafaelfranca is there any reason not to merge this? (Not clear to me that this solution works anymore. As a workaround, I'm avoiding using redirect in the routes file and instead rewriting to use redirect_to in a controller action.)