From 8e64fa5daa71c888a81d3bf3e4fbb253270cbadb Mon Sep 17 00:00:00 2001 From: Gaetan Craig-Riou Date: Mon, 25 Sep 2023 13:42:35 +0200 Subject: [PATCH] Revert to using `morph` instead of `cable_ready` The issue is with with the stimilus tooltip controller, it add some element to the DOM which create issue when it's modified by StimulusReflex. See here for a more detailed explanation: https://github.com/stimulusreflex/stimulus_reflex/issues/314#issuecomment-702479357 --- app/reflexes/admin/orders_reflex.rb | 9 ++------- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/app/reflexes/admin/orders_reflex.rb b/app/reflexes/admin/orders_reflex.rb index a518b54386c2..0a9640147d60 100644 --- a/app/reflexes/admin/orders_reflex.rb +++ b/app/reflexes/admin/orders_reflex.rb @@ -8,13 +8,8 @@ def capture payment_capture = OrderCaptureService.new(@order) if payment_capture.call - cable_ready.inner_html( - selector: dom_id(@order), - html: render(partial: "spree/admin/orders/table_row", - locals: { order: @order.reload, success: true }) - ).broadcast - - morph :nothing + morph dom_id(@order), render(partial: "spree/admin/orders/table_row", + locals: { order: @order.reload, success: true }) else flash[:error] = with_locale{ payment_capture.gateway_error || I18n.t(:payment_processing_failed)