Error: Form responses must redirect to another location

[hotwired]

Error: Form responses must redirect to another location

[dhh]

Fixed in 0.5.3. Render your form error response with unprocessable entity (422) and Turbo will display it.

[pbougie]

How do I do a complete form with validation, populated form fields on validation failure, redirect on success with Turbo? I can't find a single complete example that works. What about a mechanism to turn it off so my current forms can continue working until I find a workable solution/have time to migrate?

[nyrf]

It works if use turbo_frame_tag to wrap the form, but normal form still got the error.

<%= turbo_frame_tag "post" do %>
  <%= render 'form', post: @post %>
<% end %>

[jakemumu]

Sorry to bring this up from the dead, but when I use this trick of status: :unprocessable_entity it kills javascript running in the application. -- does anyone else have this issue?

[dylanjha]

@pbougie if you found an easy way to have forms opt-out of Turbo behavior that would be 💯 . The "default" rails way to do things that I usually see is:

• submit a form
• redirect on success
• render html on error

As such, since Turbo currently requires redirects on all forms, it's a major blocker to prevent folks switching over.

Thanks all. Other than that small snag, this Turbo thing is pretty neat, I was able to drop it into an existing rails app pretty easily! 🙌

[mhfs]

Yeah, Turbo currently requires a redirect. Not compatible with responding with HTML to a POST for error handling. We're looking into ways to support this.

Curious on how you're dealing with forms and validations in Hey/Basecamp. Client side validations and being certain the submit will succeed and result in a redirect?

EDIT: I think I got the hang of it for a more traditional rails form validation flow. Please let me know if this is considered good practice.

In your form:

   <%= form_with model: @user, id: 'registration-form' %>

In controller:

  def create
    @user = User.new(user_params)

    respond_to do |format|
      if @user.save
        format.turbo_stream { redirect_to root_path }
      else
        format.turbo_stream { render :form }
      end
    end
  end

And form.turbo_stream.erb

<%= turbo_stream.replace 'registration-form' do %>
  <%= render partial: 'form', locals: { user: @user }, formats: [:html] %>
<% end %

[charlietag]

Hi @nyrf ,

I'm trying Rails 6.1 + gem hotwire-rails (latest version).

And I think it works this way
No turbo_stream.erb files needed.
It works like traditional form submission usage

  # POST /posts
  # POST /posts.json
  def create
    @post = Post.new(post_params)

    respond_to do |format|
      if @post.save
        format.html { redirect_to @post, notice: 'Post was successfully created.' }
-        format.json { render :show, status: :created, location: @post }
      else
-        format.html { render :new }
+        format.html { render :new, status: :unprocessable_entity}
-        format.json { render json: @post.errors, status: :unprocessable_entity }
      end
    end
  end

[charlietag]

I dont think the scafold generators know to add the status: :unprocessable_entity in rails 7.01

I think it does.

[dhh]

Yeah, Turbo currently requires a redirect. Not compatible with responding with HTML to a POST for error handling. We're looking into ways to support this.

[adammiribyan]

Upgrading to rails 6.1 fixed this.

[adammiribyan]

Also, make sure you're using at least ruby 2.6.

[nyrf]

@adammiribyan I'm using ruby 2.7.2 and Rails 6.1.0.

[adammiribyan]

@nyrf what's your controller code for that form?

[nyrf]

@adammiribyan I used default scaffold command, rails g scaffold post title body:text

[asecondwill]

I dont think the scafold generators know to add the status: :unprocessable_entity in rails 7.01

[lapser]

Yeah, Turbo currently requires a redirect. Not compatible with responding with HTML to a POST for error handling. We're looking into ways to support this.

@dhh Any news on supporting TURBO_STREAM responses like render(html: "Support HTML", ...)?

[marcoroth]

dhh Any news on supporting TURBO_STREAM responses like render(html: "Support HTML", ...)?

@lapser you can already do this today, just add the turbo_stream format to your respond_to block and add the status: :unprocessable_entity:

format.turbo_stream { render turbo_stream: turbo_stream.append_all("body", "Form submission failed"), status: :unprocessable_entity }

[topherfangio]

Sorry to bring this up from the dead, but when I use this trick of status: :unprocessable_entity it kills javascript running in the application. -- does anyone else have this issue?

@jakemumu I just ran into this today and realized it wasn't a Turbo error, but rather the 3rd party library I was using.

It only initializes on page load, but we're replacing parts of the dom. So I had to wrap it with a Stimulus controller and manually run the code (a Datepicker in this instance).

I imagine you are likely hitting the same issue.