From 351659f8eb36a3a23e8881dae0d18bd560c44197 Mon Sep 17 00:00:00 2001 From: Bryce Date: Wed, 20 May 2026 23:18:35 -0700 Subject: [PATCH] Fix toggle-amount-mode: properly update request state with converted accounts and new mode The assoc-in call had too many arguments, so the request state wasn't being updated with the new mode or converted accounts. Using -> threading with separate assoc-in calls ensures both the accounts and mode are properly set before re-rendering the grid. --- src/clj/auto_ap/ssr/transaction/edit.clj | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/clj/auto_ap/ssr/transaction/edit.clj b/src/clj/auto_ap/ssr/transaction/edit.clj index c7677c33..1441176d 100644 --- a/src/clj/auto_ap/ssr/transaction/edit.clj +++ b/src/clj/auto_ap/ssr/transaction/edit.clj @@ -362,8 +362,9 @@ accounts (convert-accounts-mode (:transaction/accounts snapshot) old-mode new-mode total)] (html-response [:div#account-grid-body - (account-grid-body* (assoc-in request [:multi-form-state :snapshot :transaction/accounts] accounts - [:multi-form-state :snapshot :amount-mode] new-mode))]))) + (account-grid-body* (-> request + (assoc-in [:multi-form-state :snapshot :transaction/accounts] accounts) + (assoc-in [:multi-form-state :snapshot :amount-mode] new-mode)))]))) (defrecord BasicDetailsStep [linear-wizard] mm/ModalWizardStep