Fix bug multiple request when drag and drop position stage (#589)

Author: YukioArie

app/controllers/accounts/pipelines_controller.rb

@@ -177,16 +177,13 @@ def create
 
   # PATCH/PUT /pipelines/1 or /pipelines/1.json
   def update
-    respond_to do |format|
-      if @pipeline.update(pipeline_params)
+    if @pipeline.update(pipeline_params)
+      respond_to do |format|
         format.html do
           redirect_to account_pipeline_path(Current.account, @pipeline),
                       notice: t('flash_messages.updated', model: Pipeline.model_name.human)
         end
-        format.json { render :show, status: :ok, location: @pipeline }
-      else
-        format.html { render :edit, status: :unprocessable_entity }
-        format.json { render json: @pipeline.errors, status: :unprocessable_entity }
+        format.turbo_stream
       end
     end
   end

app/javascript/controllers/drag_stages_position_controller.js

@@ -32,6 +32,16 @@ export default class extends Controller {
       url: this.data.get('url'),
       type: 'PATCH',
       data: data,
+      beforeSend: (xhr) => {
+        xhr.setRequestHeader("Accept", "text/vnd.turbo-stream.html");
+        return true;
+      },
+      success: (response) => {
+        Turbo.renderStreamMessage(response);
+      },
+      error: (response) => {
+        Turbo.renderStreamMessage(response);
+      },
     });
   }
 }

app/views/accounts/pipelines/_form.html.erb

@@ -1,4 +1,4 @@
-<%= form_with(model: pipeline, url: pipeline.persisted? ? account_pipeline_path(Current.account, pipeline) : account_pipelines_path(Current.account), class: "flex flex-col gap-5") do |form| %>
+<%= form_with(model: pipeline, url: pipeline.persisted? ? account_pipeline_path(Current.account, pipeline) : account_pipelines_path(Current.account), class: "flex flex-col gap-5", data: { turbo: false }) do |form| %>
   <% if pipeline.errors.any? %>
     <div>
       <ul>