From 31c537c5e095cf72d8aaa9cf86b6ad9f6680ee97 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 fe59036fb8..765438d7cc 100644 --- a/app/reflexes/admin/orders_reflex.rb +++ b/app/reflexes/admin/orders_reflex.rb @@ -8,13 +8,8 @@ module Admin 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] = payment_capture.gateway_error || I18n.t(:payment_processing_failed) morph_admin_flashes