Merge pull request #4705 from EdgeApp/matthew/bitpayRbf

Disable RBF for payment protocol transactions

CHANGELOG.md

@@ -16,6 +16,7 @@
 - changed: Use new EdgeTxAction data for Thorchain and Tron stake plugins
 - changed: Make useAsyncEffect tags required
 - changed: Rettry failed WalletConnect initializations
+- changed: Disable RBF for payment protocol transactions
 - fixed: USP vs legacy landing experiment distribution
 - fixed: Paybis sell from Tron USDT
 - fixed: Remove `minWidth` style from stake option card

src/actions/PaymentProtoActions.tsx

@@ -222,6 +222,16 @@ export async function launchPaymentProto(navigation: NavigationBase, account: Ed
     }),
     metadata
   }
+
+  // RBF transactions aren't supported so it needs to be disabled
+  if (selectedWallet.currencyInfo.canReplaceByFee) {
+    Object.assign(spendInfo, {
+      otherParams: {
+        enableRbf: false
+      }
+    })
+  }
+
   if (requiredFeeRate != null) {
     spendInfo.networkFeeOption = 'custom'
     spendInfo.customNetworkFee = { satPerByte: Math.ceil(requiredFeeRate) }

src/util/DeepLinkParser.ts

@@ -43,6 +43,8 @@ export function parseDeepLink(uri: string, opts: { aztecoApiKey?: string } = {})
     // If the URI started with 'bitcoin:', etc.
     uri = betterUrl.query.r
     return { type: 'paymentProto', uri }
+  } else if (betterUrl.hostname.includes('bitpay.com')) {
+    return { type: 'paymentProto', uri }
   }
 
   // Handle the edge:// scheme: