queried cleared balance for tooltip (#1678)

actualbudget · Sep 13, 2023 · b6100cf · b6100cf
1 parent d835b11
commit b6100cf
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 4 deletions.
diff --git a/packages/desktop-client/src/components/accounts/Reconcile.js b/packages/desktop-client/src/components/accounts/Reconcile.js
@@ -96,15 +96,20 @@ export function ReconcilingMessage({
 }
 
 export function ReconcileTooltip({ account, onReconcile, onClose }) {
-  let balance = useSheetValue(queries.accountBalance(account));
+  let balanceQuery = queries.accountBalance(account);
+  let clearedBalance = useSheetValue({
+    name: balanceQuery.name + '-cleared',
+    value: null,
+    query: balanceQuery.query.filter({ cleared: true }),
+  });
   let format = useFormat();
 
   function onSubmit(e) {
     e.preventDefault();
     let input = e.target.elements[0];
     let amount = currencyToInteger(input.value);
     if (amount != null) {
-      onReconcile(amount == null ? balance : amount);
+      onReconcile(amount == null ? clearedBalance : amount);
       onClose();
     } else {
       input.select();
@@ -119,10 +124,10 @@ export function ReconcileTooltip({ account, onReconcile, onClose }) {
           reconcile with:
         </Text>
         <form onSubmit={onSubmit}>
-          {balance != null && (
+          {clearedBalance != null && (
             <InitialFocus>
               <Input
-                defaultValue={format(balance, 'financial')}
+                defaultValue={format(clearedBalance, 'financial')}
                 style={{ margin: '7px 0' }}
               />
             </InitialFocus>

diff --git a/upcoming-release-notes/1678.md b/upcoming-release-notes/1678.md
@@ -0,0 +1,6 @@
+---
+category: Bugfix
+authors: [shaankhosla]
+---
+
+Fixes the reconciliation tooltip to use the cleared balance instead of the balance from all transactions