@exodus/activity-txs 4.6.4 → 4.6.5

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (3) hide show
  1. package/CHANGELOG.md +6 -0
  2. package/package.json +9 -9
  3. package/redux/selectors/create-with-nfts-activity.js +32 -17
package/CHANGELOG.md CHANGED
@@ -3,6 +3,12 @@
3
3
  All notable changes to this project will be documented in this file.
4
4
  See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
5
5
 
6
+ ## [4.6.5](https://github.com/ExodusMovement/exodus-hydra/compare/@exodus/activity-txs@4.6.4...@exodus/activity-txs@4.6.5) (2025-04-03)
7
+
8
+ ### Bug Fixes
9
+
10
+ - fix: filter out tx activities associated with a spam nft (#11950)
11
+
6
12
  ## [4.6.4](https://github.com/ExodusMovement/exodus-hydra/compare/@exodus/activity-txs@4.6.3...@exodus/activity-txs@4.6.4) (2025-02-12)
7
13
 
8
14
  ### Performance
package/package.json CHANGED
@@ -1,6 +1,6 @@
1
1
  {
2
2
  "name": "@exodus/activity-txs",
3
- "version": "4.6.4",
3
+ "version": "4.6.5",
4
4
  "description": "The activity-txs feature",
5
5
  "author": "Exodus Movement, Inc.",
6
6
  "repository": {
@@ -44,23 +44,23 @@
44
44
  "devDependencies": {
45
45
  "@exodus/assets": "^8.0.95",
46
46
  "@exodus/assets-base": "^8.1.10",
47
- "@exodus/assets-feature": "^6.1.0",
47
+ "@exodus/assets-feature": "^7.2.1",
48
48
  "@exodus/available-assets": "^8.7.0",
49
49
  "@exodus/bitcoin-plugin": "^1.4.2",
50
- "@exodus/connected-origins": "^4.1.1",
51
- "@exodus/error-tracking": "^1.3.1",
52
- "@exodus/fiat-ramp": "^12.2.0",
50
+ "@exodus/connected-origins": "^4.3.0",
51
+ "@exodus/error-tracking": "^1.5.1",
52
+ "@exodus/fiat-ramp": "^12.7.1",
53
53
  "@exodus/logger": "^1.2.3",
54
- "@exodus/nfts": "^9.5.0",
55
- "@exodus/orders": "^4.15.0",
54
+ "@exodus/nfts": "^9.5.1",
55
+ "@exodus/orders": "^4.16.7",
56
56
  "@exodus/personal-notes": "^3.7.0",
57
57
  "@exodus/redux-dependency-injection": "^4.1.1",
58
- "@exodus/wallet-accounts": "^17.3.0",
58
+ "@exodus/wallet-accounts": "^17.5.0",
59
59
  "@types/minimalistic-assert": "^1.0.1",
60
60
  "redux": "^4.2.1"
61
61
  },
62
62
  "publishConfig": {
63
63
  "access": "public"
64
64
  },
65
- "gitHead": "16fb1f9330a353da411f5023911cf60075ab0a54"
65
+ "gitHead": "37ecf6b50e43f67fb416ab04e376e020450e7e09"
66
66
  }
package/redux/selectors/create-with-nfts-activity.js CHANGED
@@ -4,7 +4,7 @@ import assert from 'minimalistic-assert'
4
4
  import { formatNftTx, formatUnindexedNftTx } from '../utils/activity-formatters/format-nft'
5
5
 
6
6
  const selectorFactory =
7
- (createAssetSourceNftTxsById, assetsSelector) =>
7
+ (createAssetSourceNftTxsById, activeNftsSelector, assetsSelector) =>
8
8
  ({ createActivitySelector, nftsNetworkNameToAssetName }) => {
9
9
  assert(createActivitySelector, 'please provide activitySelector')
10
10
  assert(nftsNetworkNameToAssetName, 'please provide nftsNetworkNameToAssetName')
@@ -12,7 +12,7 @@ const selectorFactory =
12
12
  return memoize(
13
13
  ({ assetName, walletAccount, ...rest }) => {
14
14
  const activitySelector = createActivitySelector({ assetName, walletAccount, ...rest })
15
- if (createAssetSourceNftTxsById.isFallback) {
15
+ if (createAssetSourceNftTxsById.isFallback || activeNftsSelector.isFallback) {
16
16
  return activitySelector
17
17
  }
18
18
 
@@ -20,9 +20,17 @@ const selectorFactory =
20
20
  assert(nftsSelector, 'nfts redux feature must be installed to use this selector')
21
21
  return createSelector(
22
22
  nftsSelector,
23
+ activeNftsSelector,
23
24
  assetsSelector,
24
25
  activitySelector,
25
- (nftsByTxId, assets, activity) => {
26
+ (nftsByTxId, activeNfts, assets, activity) => {
27
+ const spamNftIdsSet = new Set(
28
+ Object.values(activeNfts)
29
+ .flat()
30
+ .filter((nft) => nft.isSpam)
31
+ .map(({ id }) => id && id.toLowerCase())
32
+ )
33
+
26
34
  const indexedNftTxIds = new Set()
27
35
  const resultActivity = []
28
36
  let hasChanges = false
@@ -30,28 +38,34 @@ const selectorFactory =
30
38
  const nft = nftsByTxId.get(activityItem.txId)
31
39
 
32
40
  if (nft) {
33
- hasChanges = true
34
- indexedNftTxIds.add(nft.txId)
35
- resultActivity.push(
36
- formatNftTx({
37
- ...activityItem,
38
- nft,
39
- asset: assets[activityItem.assetName],
40
- assetName: activityItem.assetName,
41
- })
42
- )
41
+ const nftIdLower = nft.nftId && nft.nftId.toLowerCase()
42
+ if (!nftIdLower || !spamNftIdsSet.has(nftIdLower)) {
43
+ hasChanges = true
44
+ indexedNftTxIds.add(nft.txId)
45
+ resultActivity.push(
46
+ formatNftTx({
47
+ ...activityItem,
48
+ nft,
49
+ asset: assets[activityItem.assetName],
50
+ assetName: activityItem.assetName,
51
+ })
52
+ )
53
+ }
43
54
  } else {
44
55
  resultActivity.push(activityItem)
45
56
  }
46
57
  }
47
58
 
48
59
  ;[...nftsByTxId.values()]
49
- .filter(
50
- (nftTx) =>
60
+ .filter((nftTx) => {
61
+ const nftIdLower = nftTx.nftId && nftTx.nftId.toLowerCase()
62
+ return (
51
63
  Object.hasOwn(nftsNetworkNameToAssetName, nftTx.network) &&
52
64
  assetName === nftsNetworkNameToAssetName[nftTx.network] &&
53
- !indexedNftTxIds.has(nftTx.txId)
54
- )
65
+ !indexedNftTxIds.has(nftTx.txId) &&
66
+ (!nftIdLower || !spamNftIdsSet.has(nftIdLower))
67
+ )
68
+ })
55
69
  .forEach((nftTx) => {
56
70
  hasChanges = true
57
71
  const assetName = nftsNetworkNameToAssetName[nftTx.network]
@@ -77,6 +91,7 @@ const createWithNftsActivitySelectorDefinition = {
77
91
  selectorFactory,
78
92
  dependencies: [
79
93
  { module: 'nfts', selector: 'createAssetSourceNftTxsById', optional: true },
94
+ { module: 'nfts', selector: 'activeNfts', optional: true },
80
95
  { module: 'assets', selector: 'all' },
81
96
  ],
82
97
  }