From dd4d1a6d4f2952616b714631d689baa84997c02b Mon Sep 17 00:00:00 2001
From: Bryce <bryce@brycecovertoperations.com>
Date: Wed, 20 May 2026 21:29:03 -0700
Subject: [PATCH] Add error message to require-approval validator

Ensure the validation error message shows up properly when users try
to approve a manual transaction without assigning financial accounts.
---
 src/clj/auto_ap/ssr/transaction/edit.clj | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/src/clj/auto_ap/ssr/transaction/edit.clj b/src/clj/auto_ap/ssr/transaction/edit.clj
index fed16332..b2a7a8c3 100644
--- a/src/clj/auto_ap/ssr/transaction/edit.clj
+++ b/src/clj/auto_ap/ssr/transaction/edit.clj
@@ -62,9 +62,10 @@
 
 (defn require-approval [s]
   [:and s
-   [:fn (fn [{:transaction/keys [accounts approval-status]}]
-          (or (not= approval-status :transaction-approval-status/approved)
-              (seq accounts)))]])
+   [:fn {:error/message "Approved transactions must have accounts assigned."}
+    (fn [{:transaction/keys [accounts approval-status]}]
+      (or (not= approval-status :transaction-approval-status/approved)
+           (seq accounts)))]])
 
 (def edit-form-schema
   (mc/schema