Merge pull request #1457 from basecamp/assignee-update

A couple of fixes for updating cards:

app/controllers/cards/assignments_controller.rb

@@ -6,7 +6,6 @@ class Cards::AssignmentsController < ApplicationController
 
   def create
     @card.toggle_assignment @collection.users.active.find(params[:assignee_id])
-
-    render turbo_stream: turbo_stream.replace([ @card, :assignees ], partial: "cards/display/perma/assignees", locals: { card: @card.reload })
+    render turbo_stream: turbo_stream.replace([ @card, :meta ], partial: "/cards/display/perma/meta", method: "morph", locals: { card: @card.reload })
   end
 end

app/controllers/cards/triages_controller.rb

@@ -5,11 +5,11 @@ class Cards::TriagesController < ApplicationController
     column = @card.collection.columns.find(params[:column_id])
     @card.triage_into(column)
 
-    render_card_replacement
+    redirect_to @card
   end
 
   def destroy
     @card.send_back_to_triage
-    render_card_replacement
+    redirect_to @card
   end
 end

test/controllers/cards/assignments_controller_test.rb

@@ -13,17 +13,17 @@ class Cards::AssignmentsControllerTest < ActionDispatch::IntegrationTest
   test "create" do
     assert_changes "cards(:logo).assigned_to?(users(:david))", from: false, to: true do
       post card_assignments_path(cards(:logo)), params: { assignee_id: users(:david).id }, as: :turbo_stream
-      assert_assignees_replaced(cards(:logo))
+      assert_meta_replaced(cards(:logo))
     end
 
     assert_changes "cards(:logo).assigned_to?(users(:david))", from: true, to: false do
       post card_assignments_path(cards(:logo)), params: { assignee_id: users(:david).id }, as: :turbo_stream
-      assert_assignees_replaced(cards(:logo))
+      assert_meta_replaced(cards(:logo))
     end
   end
 
   private
-    def assert_assignees_replaced(card)
-      assert_turbo_stream action: :replace, target: dom_id(card, :assignees)
+    def assert_meta_replaced(card)
+      assert_turbo_stream action: :replace, target: dom_id(card, :meta)
     end
 end

test/controllers/cards/triages_controller_test.rb

@@ -12,7 +12,7 @@ class Cards::TriagesControllerTest < ActionDispatch::IntegrationTest
 
     assert_changes -> { card.reload.column }, from: original_column, to: column do
       post card_triage_path(card, column_id: column.id)
-      assert_card_container_rerendered(card)
+      assert_redirected_to card
     end
   end
 
@@ -20,8 +20,8 @@ class Cards::TriagesControllerTest < ActionDispatch::IntegrationTest
     card = cards(:shipping)
 
     assert_changes -> { card.reload.column }, to: nil do
-      delete card_triage_path(card)
-      assert_card_container_rerendered(card)
+      delete card_triage_path(card), as: :turbo_stream
+      assert_redirected_to card
     end
   end
 end