Fix NaN on zap

Third argument of update is reserved for context
stackernews · Jan 20, 2024 · 7543e08 · 7543e08
1 parent ae19e53
commit 7543e08
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/components/invoice.js b/components/invoice.js
@@ -221,7 +221,7 @@ export const useInvoiceable = (onSubmit, options = defaultOptions) => {
       provider,
       pollInvoice,
       updateCache: () => update?.(client.cache, { data: optimisticResponse }),
-      undoUpdate: () => update?.(client.cache, { data: optimisticResponse }, true)
+      undoUpdate: () => update?.(client.cache, { data: { ...optimisticResponse, undo: true } })
     })
     const { webLN, modalClose } = payment
 

diff --git a/components/item-act.js b/components/item-act.js
@@ -172,8 +172,8 @@ export function useAct ({ onUpdate } = {}) {
 }
 
 export function useZap () {
-  const update = useCallback((cache, args, undo) => {
-    const { data: { act: { id, sats, path, amount } } } = args
+  const update = useCallback((cache, args) => {
+    const { data: { act: { id, sats, path, amount, undo } } } = args
 
     // determine how much we increased existing sats by by checking the
     // difference between result sats and meSats