@exodus/assets-feature 4.1.0 → 4.1.2

package/CHANGELOG.md

@@ -3,6 +3,20 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [4.1.2](https://github.com/ExodusMovement/exodus-hydra/compare/@exodus/assets-feature@4.1.1...@exodus/assets-feature@4.1.2) (2024-02-20)
+
+### Bug Fixes
+
+- make sure the check does not throw ([#5070](https://github.com/ExodusMovement/exodus-hydra/issues/5070)) ([b145ecc](https://github.com/ExodusMovement/exodus-hydra/commit/b145ecce8d5e2720edc9f422b913268e5f5b8364))
+- native tokens auto-disabling when already in registry ([#5736](https://github.com/ExodusMovement/exodus-hydra/issues/5736)) ([feb9e8c](https://github.com/ExodusMovement/exodus-hydra/commit/feb9e8cf79f43f5be3987108bb2bc307c3648365))
+
+## [4.1.1](https://github.com/ExodusMovement/exodus-hydra/compare/@exodus/assets-feature@4.1.0...@exodus/assets-feature@4.1.1) (2024-01-19)
+
+### Bug Fixes
+
+- merge mem ([#5448](https://github.com/ExodusMovement/exodus-hydra/issues/5448)) ([a6fb606](https://github.com/ExodusMovement/exodus-hydra/commit/a6fb6065bdabf0b9ebf0a678c5e748198f2dd100))
+- remove duplicated analytics plugin ([#5442](https://github.com/ExodusMovement/exodus-hydra/issues/5442)) ([e453417](https://github.com/ExodusMovement/exodus-hydra/commit/e453417ee9f11574655031d91f060bff86bd0894))
+
 ## [4.1.0](https://github.com/ExodusMovement/exodus-hydra/compare/@exodus/assets-feature@4.0.2...@exodus/assets-feature@4.1.0) (2024-01-12)
 
 ### Features

package/client/asset-client-interface.js

@@ -102,15 +102,15 @@ class AssetClientInterface {
     accountState,
     batch = this.blockchainMetadata.batch(),
   }) => {
-    if (!isEmpty(newData)) {
-      return batch.updateAccountState({ assetName, walletAccount, newData })
-    }
-
     // merge mem to keep the previous accountMem behavior
-    if (newData.mem && (!accountState || accountState.mem)) {
+    if (!isEmpty(newData?.mem) && (!accountState || accountState.mem)) {
       newData = { ...newData, mem: { ...accountState?.mem, ...newData.mem } }
     }
 
+    if (!isEmpty(newData)) {
+      return batch.updateAccountState({ assetName, walletAccount, newData })
+    }
+
     return batch
   }
 

package/index.js

@@ -7,7 +7,6 @@ import assetModuleDefinition from './module'
 import customTokensMonitorDefinition from './monitor'
 import assetPreferencesDefinition from './module/asset-preferences'
 import assetsAtomDefinition from './atoms/assets'
-import enabledAssetsAnalyticsPluginDefinition from './plugin/analytics'
 
 const assets = ({ config = {} } = {}) => {
   return {
@@ -36,10 +35,6 @@ const assets = ({ config = {} } = {}) => {
       },
       { definition: assetPreferencesDefinition },
       { definition: customTokensMonitorDefinition },
-      {
-        if: { registered: ['analytics'] },
-        definition: enabledAssetsAnalyticsPluginDefinition,
-      },
     ],
   }
 }

package/module/assets-module.js

@@ -318,8 +318,8 @@ export class AssetsModule {
 
     if (isEmpty(tokens)) return
 
-    const validTokens = tokensToAdd.filter(this.#validateCustomToken).map(normalizeToken)
-    if (validTokens.length !== tokensToAdd.length) this.#logger.warn('Invalid Custom Token schema')
+    const validTokens = tokens.filter(this.#validateCustomToken).map(normalizeToken)
+    if (validTokens.length !== tokens.length) this.#logger.warn('Invalid Custom Token schema')
 
     const { fetchedAndHandledTokens, added, updated } = this.#handleFetchedTokens(validTokens)
 
@@ -481,8 +481,8 @@ export class AssetsModule {
 
   #assertSupportsCustomTokens = (baseAssetName) => {
     const baseAsset = this.getAsset(baseAssetName)
-    if (!baseAsset.api.hasFeature('customTokens')) {
-      const reason = `BUG: network ${baseAsset.name} does not support custom tokens.`
+    if (!baseAsset?.api?.hasFeature('customTokens')) {
+      const reason = `BUG: network ${baseAsset?.name} does not support custom tokens.`
       this.#logger.warn(reason)
       throw new Error(reason)
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@exodus/assets-feature",
-  "version": "4.1.0",
+  "version": "4.1.2",
   "license": "UNLICENSED",
   "description": "Assets module, clients and apis",
   "main": "index.js",
@@ -48,6 +48,7 @@
     "@exodus/bitcoin-plugin": "^1.0.3",
     "@exodus/bitcoinregtest-plugin": "^1.0.3",
     "@exodus/bitcointestnet-plugin": "^1.0.3",
+    "@exodus/blockchain-metadata": "^15.1.0",
     "@exodus/combined-assets-meta": "^1.2.5",
     "@exodus/cosmos-plugin": "^1.0.0",
     "@exodus/ethereum-meta": "^1.1.0",
@@ -55,7 +56,7 @@
     "@exodus/osmosis-plugin": "^1.0.0",
     "@exodus/redux-dependency-injection": "^3.0.0",
     "@exodus/storage-memory": "^2.1.1",
-    "@exodus/wallet-accounts": "^15.1.0",
+    "@exodus/wallet-accounts": "^16.0.0",
     "@exodus/wild-emitter": "^1.0.0",
     "eslint": "^8.44.0",
     "events": "^3.3.0",
@@ -63,5 +64,5 @@
     "msw": "^2.0.0",
     "redux": "^4.0.0"
   },
-  "gitHead": "dae8c9d4c3dc27060210398351c9c3d9ed0caab7"
+  "gitHead": "27ed0586ad800043afc77ebe8192259fcfb966cc"
 }

package/plugin/analytics.js

@@ -1,26 +0,0 @@
-const createEnabledAssetsAnalyticsPlugin = ({ analytics, enabledAssetsAtom }) => {
-  let unsubscribe
-
-  const onStart = () => {
-    analytics.requireDefaultProperties(['numberOfAssetsEnabled'])
-
-    unsubscribe = enabledAssetsAtom.observe((enabledAssets) => {
-      const numberOfAssetsEnabled = Object.keys(enabledAssets).length
-
-      analytics.setDefaultProperties({ numberOfAssetsEnabled })
-    })
-  }
-
-  const onStop = () => unsubscribe()
-
-  return { onStart, onStop }
-}
-
-const enabledAssetsAnalyticsPluginDefinition = {
-  id: 'enabledAssetsAnalyticsPlugin',
-  type: 'plugin',
-  factory: createEnabledAssetsAnalyticsPlugin,
-  dependencies: ['analytics', 'enabledAssetsAtom'],
-}
-
-export default enabledAssetsAnalyticsPluginDefinition