Merge pull request #1212 from hirosystems/release/memo-warnings

Release/memo warnings

Author: kyranjamie

.github/workflows/debug-build.yml

@@ -92,7 +92,6 @@ jobs:
 
       - uses: actions/setup-python@v4
         with:
-          python-version: '2.7.18'
           architecture: x64
 
       #

CHANGELOG.md

@@ -1,3 +1,10 @@
+## [4.10.1-dev.1](https://github.com/blockstack/stacks-wallet/compare/v4.10.0...v4.10.1-dev.1) (2023-06-22)
+
+
+### Bug Fixes
+
+* only show memo for sent transactions ([bbfeb04](https://github.com/blockstack/stacks-wallet/commit/bbfeb0498f63da50a3ff7141d225f8f13e694071))
+
 # [4.10.0](https://github.com/blockstack/stacks-wallet/compare/v4.9.1...v4.10.0) (2023-05-25)
 
 

app/components/home/transaction-list/transaction-list-item.tsx

@@ -21,6 +21,7 @@ import {
   isDelegatedStackingTx,
   inferSendManyTransferOperation,
   isSendManyTx,
+  getMemoForTx,
 } from '@utils/tx-utils';
 import { toHumanReadableStx } from '@utils/unit-convert';
 import React, {
@@ -106,12 +107,8 @@ export const TransactionListItem: FC<TransactionListItemProps> = props => {
   }, [direction, poxInfo?.contract_id, tx]);
 
   const sumPrefix = direction === 'sent' && !isStackingTx(tx, poxInfo?.contract_id) ? '−' : '';
-  const memo =
-    tx.tx_type === 'token_transfer' &&
-    Buffer.from(
-      tx.token_transfer.memo.replace('0x', '').replace(/^(0{2})+|(0{2})+$/g, ''),
-      'hex'
-    ).toString('utf8');
+  const memo = getMemoForTx(tx, direction);
+
   const txDate = new Date(tx.burn_block_time_iso || (tx as any).parent_burn_block_time_iso);
   const txDateShort = txDate.toLocaleString();
 

app/utils/tx-utils.ts

@@ -112,3 +112,16 @@ export const truncateMiddle = (input: string, offset = 5): string => {
     return `${start}…${end}`;
   }
 };
+
+export const getMemoForTx = (tx: Transaction, direction: StxTxDirection) => {
+  if (tx.tx_type !== 'token_transfer' || direction !== 'sent') {
+    return undefined;
+  }
+
+  const utf8Memo = Buffer.from(
+    tx.token_transfer.memo.replace('0x', '').replace(/^(0{2})+|(0{2})+$/g, ''),
+    'hex'
+  ).toString('utf8');
+
+  return utf8Memo;
+};

package.json

@@ -1,6 +1,6 @@
 {
   "name": "stacks-wallet",
-  "version": "4.10.0",
+  "version": "4.10.1-dev.1",
   "description": "Hiro Wallet 2.0 — Stacking",
   "prettier": "@stacks/prettier-config",
   "homepage": "https://hiro.so/wallet",