fix: adds updatePersistedCollections in setNftsForAccount

bloomwalletio · May 22, 2024 · e907885 · e907885
1 parent 77f1965
commit e907885
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 11 deletions.
diff --git a/packages/shared/src/lib/core/nfts/actions/loadNftsForActiveProfile.ts b/packages/shared/src/lib/core/nfts/actions/loadNftsForActiveProfile.ts
@@ -8,7 +8,7 @@ import { get } from 'svelte/store'
 import { Nft } from '../interfaces'
 import { addNftsToDownloadQueue } from './addNftsToDownloadQueue'
 import { buildNftFromNftOutput } from './buildNftFromNftOutput'
-import { setNftsForAccount } from '../stores'
+import { addOrUpdateNftsForAccount } from '../stores'
 
 export async function loadNftsForActiveProfile(): Promise<void> {
     let nftsToDownload: Nft[] = []
@@ -60,7 +60,7 @@ export async function loadNftsForAccount(account: IAccountState): Promise<Nft[]>
             }
         }
     }
-    setNftsForAccount(account.index, accountNfts)
+    addOrUpdateNftsForAccount(account.index, accountNfts)
 
     return accountNfts
 }
diff --git a/packages/shared/src/lib/core/nfts/actions/updatePersistedCollections.ts b/packages/shared/src/lib/core/nfts/actions/updatePersistedCollections.ts
@@ -7,7 +7,7 @@ export async function updatePersistedCollections(nfts: Nft[]): Promise<void> {
     const collectionsToAdd: PersistedCollection[] = []
     for (const nft of nfts) {
         if (!nft.collectionId || get(persistedCollections)[nft.collectionId]) {
-            return
+            continue
         }
 
         const collection = await buildPersistedCollectionFromNft(nft)

diff --git a/packages/shared/src/lib/core/nfts/stores/active-profile-nfts-per-account.store.ts b/packages/shared/src/lib/core/nfts/stores/active-profile-nfts-per-account.store.ts
@@ -18,13 +18,6 @@ export function getNftsForAccount(accountIndex: number): Nft[] {
     return get(activeProfileNftsPerAccount)[accountIndex] ?? []
 }
 
-export function setNftsForAccount(accountIndex: number, nfts: Nft[]): void {
-    activeProfileNftsPerAccount.update((state) => {
-        state[accountIndex] = nfts
-        return state
-    })
-}
-
 export function addOrUpdateNftsForAccount(accountIndex: number, nfts: Nft[]): void {
     const addedNfts: Nft[] = []
     activeProfileNftsPerAccount.update((state) => {

diff --git a/packages/shared/src/lib/core/nfts/utils/buildPersistedErc721Nft.ts b/packages/shared/src/lib/core/nfts/utils/buildPersistedErc721Nft.ts
@@ -54,7 +54,9 @@ export async function buildPersistedErc721Nft(
 
             persistedNft.mediaUrl = metadata.image
         } catch (err) {
-            throw new Error(`Unable to get metadata of token ${tokenId} from contract ${contractMetadata.address}`)
+            throw new Error(
+                `Unable to get metadata of token ${tokenId} from contract ${contractMetadata.address} due to: ${err}`
+            )
         }
     }