Use Turbo.fetch to prevent page refreshes bounces

[thiagoyoussef]

Extracted from fizzy as requested by @jorgemanrubia at basecamp/fizzy@ebf8490

[jorgemanrubia]

Thanks @thiagoyoussef 👏.

[jorgemanrubia]

@dhh @jeremy could we get this one merged please?

[marcelolx]

Merged @jorgemanrubia!

[marcelolx]

We will have to fix this in https://github.com/rails/request.js, though

[marcelolx]

@jorgemanrubia just so I fully understand this, essentially, it would be ideal for request.js to always use Turbo's fetch when available, not just for a specific response kind?

[marcelolx]

Ok, changing it on the request.js repo, here is the PR rails/request.js#90

[marcelolx]

Ok, released v13 of the js package https://github.com/rails/request.js/releases/tag/v0.0.13, and v14 of the gem https://github.com/rails/requestjs-rails/releases/tag/v0.0.14

[thiagoyoussef]

Will send the follow-up PR to Fizzy, thanks guys!

[jorgemanrubia]

it would be ideal for request.js to always use Turbo's fetch when available, not just for a specific response kind?

Yes, exactly. The response kind filter didn't really make sense here: we want that whenever you use this library to make fetch requests, it triggers the Turbo mechanism to prevent self-triggered page refreshes, regardless the response kind (which can't be really anticipated). Thanks for merging it 🙏