From 8b83f729d7394837a3be231bbeeea44f6a01e9a1 Mon Sep 17 00:00:00 2001
From: Florian Dold <florian@dold.me>
Date: Fri, 17 Feb 2023 02:25:46 +0100
Subject: -comments

---
 .../taler-wallet-core/src/operations/pay-peer.ts   |  2 ++
 .../src/operations/transactions.ts                 | 23 ++++++++++++++--------
 packages/taler-wallet-core/src/wallet-api-types.ts | 15 +++++++++++---
 3 files changed, 29 insertions(+), 11 deletions(-)

(limited to 'packages/taler-wallet-core/src')

diff --git a/packages/taler-wallet-core/src/operations/pay-peer.ts b/packages/taler-wallet-core/src/operations/pay-peer.ts
index dd5ea13c9..c1cacead9 100644
--- a/packages/taler-wallet-core/src/operations/pay-peer.ts
+++ b/packages/taler-wallet-core/src/operations/pay-peer.ts
@@ -1222,3 +1222,5 @@ export async function initiatePeerPullPayment(
     ),
   };
 }
+
+
diff --git a/packages/taler-wallet-core/src/operations/transactions.ts b/packages/taler-wallet-core/src/operations/transactions.ts
index 0dbbc56c1..d2a7e9d41 100644
--- a/packages/taler-wallet-core/src/operations/transactions.ts
+++ b/packages/taler-wallet-core/src/operations/transactions.ts
@@ -1361,13 +1361,20 @@ export async function abortTransaction(
 ): Promise<void> {
   const { type, args: rest } = parseId("txn", transactionId);
 
-  if (type === TransactionType.Payment) {
-    const proposalId = rest[0];
-    await abortPay(ws, proposalId, forceImmediateAbort);
-  } else {
-    const unknownTxType: any = type;
-    throw Error(
-      `can't abort a '${unknownTxType}' transaction: not yet implemented`,
-    );
+  switch (type) {
+    case TransactionType.Payment: {
+      const proposalId = rest[0];
+      await abortPay(ws, proposalId, forceImmediateAbort);
+      break;
+    }
+    case TransactionType.PeerPushDebit: {
+      break;
+    }
+    default: {
+      const unknownTxType: any = type;
+      throw Error(
+        `can't abort a '${unknownTxType}' transaction: not yet implemented`,
+      );
+    }
   }
 }
diff --git a/packages/taler-wallet-core/src/wallet-api-types.ts b/packages/taler-wallet-core/src/wallet-api-types.ts
index fab7632f2..3895c944d 100644
--- a/packages/taler-wallet-core/src/wallet-api-types.ts
+++ b/packages/taler-wallet-core/src/wallet-api-types.ts
@@ -591,8 +591,9 @@ export type ExportBackupPlainOp = {
 // group: Peer Payments
 
 /**
- * Check if initiating a peer push payment is possible.
- * 
+ * Check if initiating a peer push payment is possible
+ * based on the funds in the wallet.
+ *
  * FIXME: Rename to CheckPeerPushPaymentInitiation
  */
 export type PreparePeerPushPaymentOp = {
@@ -612,6 +613,8 @@ export type InitiatePeerPushPaymentOp = {
 
 /**
  * Check an incoming peer push payment.
+ * 
+ * FIXME: Rename to "PrepareIncomingPeerPushPayment"
  */
 export type CheckPeerPushPaymentOp = {
   op: WalletApiOperation.CheckPeerPushPayment;
@@ -630,6 +633,8 @@ export type AcceptPeerPushPaymentOp = {
 
 /**
  * Initiate an outgoing peer pull payment.
+ * 
+ * FIXME: This does not check anything, so rename to CheckPeerPullPaymentInitiation
  */
 export type PreparePeerPullPaymentOp = {
   op: WalletApiOperation.PreparePeerPullPayment;
@@ -648,6 +653,8 @@ export type InitiatePeerPullPaymentOp = {
 
 /**
  * Prepare for an incoming peer pull payment.
+ *
+ * FIXME: Rename to "PreparePeerPullPayment"
  */
 export type CheckPeerPullPaymentOp = {
   op: WalletApiOperation.CheckPeerPullPayment;
@@ -656,7 +663,9 @@ export type CheckPeerPullPaymentOp = {
 };
 
 /**
- * Accept an incoming peer pull payment.
+ * Accept an incoming peer pull payment (i.e. pay the other party).
+ *
+ * FIXME: Rename to ConfirmPeerPullPayment
  */
 export type AcceptPeerPullPaymentOp = {
   op: WalletApiOperation.AcceptPeerPullPayment;
-- 
cgit v1.2.3