Follow render_card_replacement pattern

And clean up the rest of the implementation a tad.
This commit is contained in:
David Heinemeier Hansson
2025-04-18 15:20:38 +02:00
parent 7395d4e034
commit f3000a02c5
3 changed files with 16 additions and 10 deletions
+16 -8
View File
@@ -5,21 +5,29 @@ class Cards::PinsController < ApplicationController
end
def create
pin = @card.pin_by Current.user
broadcast_add_to_tray pin
@pin = @card.pin_by Current.user
broadcast_add_pin_to_tray
render_pin_button_replacement
end
def destroy
pin = @card.unpin_by Current.user
broadcast_remove_from_tray pin
@pin = @card.unpin_by Current.user
broadcast_remove_pin_from_tray
render_pin_button_replacement
end
private
def broadcast_add_to_tray(pin)
pin.broadcast_prepend_to [ Current.user, :pins_tray ], target: "pins", partial: "my/pins/pin"
def broadcast_add_pin_to_tray
@pin.broadcast_prepend_to [ Current.user, :pins_tray ], target: "pins", partial: "my/pins/pin"
end
def broadcast_remove_from_tray(pin)
pin.broadcast_remove_to [ Current.user, :pins_tray ]
def broadcast_remove_pin_from_tray
@pin.broadcast_remove_to [ Current.user, :pins_tray ]
end
def render_pin_button_replacement
turbo_stream.replace [ @card, :pin_button ], partial: "cards/pins/pin_button", locals: { card: @card }
end
end
@@ -1 +0,0 @@
<%= turbo_stream.replace [ @card, :pin_button ], partial: "cards/pins/pin_button", locals: { card: @card } %>
@@ -1 +0,0 @@
<%= render template: "cards/pins/create" %>