From 0799ad050b5112e9c5c3aa03313f580c62091fae Mon Sep 17 00:00:00 2001
From: Mike Dalessio <mike@37signals.com>
Date: Wed, 2 Jul 2025 22:08:09 -0400
Subject: [PATCH] Collection workflow edits render a turbo frame

so we don't disturb the rest of the edit page.
---
 .../collections/workflows_controller.rb       |  2 +-
 .../collections/edit/_workflows.html.erb      | 44 ++++++++++---------
 .../collections/workflows_controller_test.rb  |  2 +-
 3 files changed, 25 insertions(+), 23 deletions(-)

diff --git a/app/controllers/collections/workflows_controller.rb b/app/controllers/collections/workflows_controller.rb
index a24e646ee..fd503473c 100644
--- a/app/controllers/collections/workflows_controller.rb
+++ b/app/controllers/collections/workflows_controller.rb
@@ -5,7 +5,7 @@ class Collections::WorkflowsController < ApplicationController
 
   def update
     @collection.update! workflow: @workflow
-    redirect_to edit_collection_path(@collection), notice: "Collection updated"
+    render turbo_stream: turbo_stream.replace([ @collection, :workflows ], partial: "collections/edit/workflows", locals: { collection: @collection })
   end
 
   private
diff --git a/app/views/collections/edit/_workflows.html.erb b/app/views/collections/edit/_workflows.html.erb
index 1b6154415..73d36de66 100644
--- a/app/views/collections/edit/_workflows.html.erb
+++ b/app/views/collections/edit/_workflows.html.erb
@@ -1,21 +1,23 @@
-<strong class="txt-large">Workflows</strong>
-<div class="flex justify-center margin-block-end">Use a Workflow to track progress in this Collection.</div>
-<div class="flex flex-wrap gap justify-center">
-  <% Workflow.all.each do |workflow| %>
-    <div class="workflow-preview txt-align-start border border-radius <%= 'workflow-preview--selected' if workflow == collection.workflow %>">
-      <%= button_to collection_workflow_path(collection), method: :patch do %>
-        <%= hidden_field_tag "collection[workflow_id]", workflow.id %>
-        <strong class="txt-small overflow-ellipsis flex-item-no-shrink"><%= workflow.name %></strong>
-        <ul class="list-style-none txt-xx-small margin-none flex flex-column gap-half">
-          <% workflow.stages.each do |stage| %>
-            <li class="overflow-ellipsis flex align-center gap-half min-width">
-              <span class="workflow-preview__swatch btn btn--circle" style="--btn-background: <%= stage.color.present? ? stage.color : Card::Colored::DEFAULT_COLOR %>;"></span>
-              <span class="overflow-ellipsis"><%= stage.name %></span>
-            </li>
-          <% end %>
-        </ul>
-      <% end %>
-    </div>
-  <% end %>
-</div>
-<p class="margin-block-end-double"><%= link_to "Manage workflows", workflows_path, class: "btn btn--plain txt-link" %></p>
+<%= turbo_frame_tag @collection, :workflows do %>
+  <strong class="txt-large">Workflows</strong>
+  <div class="flex justify-center margin-block-end">Use a Workflow to track progress in this Collection.</div>
+  <div class="flex flex-wrap gap justify-center">
+    <% Workflow.all.each do |workflow| %>
+      <div class="workflow-preview txt-align-start border border-radius <%= 'workflow-preview--selected' if workflow == collection.workflow %>">
+        <%= button_to collection_workflow_path(collection), method: :patch do %>
+          <%= hidden_field_tag "collection[workflow_id]", workflow.id %>
+          <strong class="txt-small overflow-ellipsis flex-item-no-shrink"><%= workflow.name %></strong>
+          <ul class="list-style-none txt-xx-small margin-none flex flex-column gap-half">
+            <% workflow.stages.each do |stage| %>
+              <li class="overflow-ellipsis flex align-center gap-half min-width">
+                <span class="workflow-preview__swatch btn btn--circle" style="--btn-background: <%= stage.color.present? ? stage.color : Card::Colored::DEFAULT_COLOR %>;"></span>
+                <span class="overflow-ellipsis"><%= stage.name %></span>
+              </li>
+            <% end %>
+          </ul>
+        <% end %>
+      </div>
+    <% end %>
+  </div>
+  <p class="margin-block-end-double"><%= link_to "Manage workflows", workflows_path, class: "btn btn--plain txt-link" %></p>
+<% end %>
diff --git a/test/controllers/collections/workflows_controller_test.rb b/test/controllers/collections/workflows_controller_test.rb
index 68a85b956..e4e739720 100644
--- a/test/controllers/collections/workflows_controller_test.rb
+++ b/test/controllers/collections/workflows_controller_test.rb
@@ -10,7 +10,7 @@ class Collections::WorkflowsControllerTest < ActionDispatch::IntegrationTest
 
     patch collection_workflow_path(collection), params: { collection: { workflow_id: workflows(:on_call).id } }
 
-    assert_redirected_to edit_collection_path(collection)
+    assert_turbo_stream action: :replace, target: dom_id(collection, :workflows)
     assert_equal workflows(:on_call), collection.reload.workflow
   end
 end