Missing Turbo-Frame header when submitting a form about turbo HOT 16 CLOSED

@seanpdoyle using beta 4. Turbo header is not sent with POST request, only GET.

from turbo.

Sorry if it wasn't clear, I was advocating to set the Turbo-Frame header only when a form within a frame is being submitted

from turbo.

In the following example, only the first GET button will include a request header with Turbo-Frame, the other 2 do not include it in the header.

<%= turbo_frame_tag 'user_frame' do %>
  <%= button_to('Edit User', '/users/1', method: "get") %>    <!-- ADDS header['Turbo-Frame'] --> 
  <%= button_to('Save',      '/users/1', method: "post") %>   <!-- NO header['Turbo-Frame'] --> 
  <%= button_to('Delete',    '/users/1', method: "delete") %> <!-- NO header['Turbo-Frame'] --> 
<% end %>

Is this intended or a bug?

using: @hotwired/turbo 7.0.0-beta.3

from turbo.

I am still experiencing the issue in beta-4.

GET (links) send the header.
POST (form submissions) do not.

from turbo.

@tleish that's intentional: see #52 and #127.

Ah. I did not realize this was intentional. I also didn't realize I could not use turbo-stream with a GET method.

from turbo.

Adding the Turbo-Frame header on non-frame requests would blur the concept of frames.

For invalid forms, we recommend responding with the 422 HTTP status and the form contents will be replaced. See #39. You can also respond with turbo streams.

from turbo.

Ah! Yeah, that should be set, and if not, it's a bug.

from turbo.

Hi!

If this is a bug, then I can also confirm it :). Inside the a frame, when I click a link, I can see the Turbo-Frame header. When I submit a form right below that link (inside the same frame), the Turbo-Frame header appears to be missing.

Cheers!

from turbo.

I can confirm this issue (using Django):

1. Request inside turbo frame using a link: Turbo-Frame header present.
2. Request inside turbo frame using a form submission: Turbo-Frame header not present.

from turbo.

If you prefer an alternative approach than PR #110 I'm happy to work on it.

from turbo.

Run into the same issue today (using hotwire with Golang backend). Looks like the suggested fix is still hanging :-\

from turbo.

As part of #142, we changed how the headers are constructed from a merge operation to a direct mutation. Combined with #127, all form submissions (GET or POST, inside a frame or referencing one from the outside) should result in a requests with the Turbo-Frame header.

That behavior was shipped as part of https://github.com/hotwired/turbo/releases/tag/v7.0.0-beta.4.

@Intrepidd @danjac @oleksandr are you still experiencing this issue?

from turbo.

@seanpdoyle — created this JSFiddle to show the headers being sent. Click any of the buttons and see the headers sent through fetch below the buttons. Turbo only adds turbo-frame to header on a GET.

from turbo.

One other item to note, should the GET request also include text/vnd.turbo-stream.html in the accept header (which it currently does not)?

from turbo.

@tleish that's intentional: see #52, #127, and hotwired/turbo-site#40 (comment).

from turbo.

I confirm as well, header is still not sent for my POST use-case

from turbo.