fix: Can remove marketplace or listing from modal (#5362)

* removing asset doesn't change other listings

* remove marketplace from modal removes asset

* working multiple markets

* better fix

* track signed listing

* simpler solution

* remove unused field

* correct transition
Co-authored-by: Charles Bachmeier <charlie@genie.xyz>

src/nft/components/bag/profile/ListingButton.tsx

@@ -51,9 +51,7 @@ export const ListingButton = ({ onClick, buttonText, showWarningOverride = false
     listingsAboveSellOrderFloor,
     invalidPrices,
   ] = useMemo(() => {
-    const noMarketplacesSelected = sellAssets.some(
-      (asset: WalletAsset) => asset.marketplaces === undefined || asset.marketplaces.length === 0
-    )
+    const noMarketplacesSelected = sellAssets.some((asset: WalletAsset) => asset.marketplaces === undefined)
     const missingExpiration = sellAssets.some((asset) => {
       return asset.expirationTime != null && asset.expirationTime * 1000 - Date.now() < ms`60 seconds`
     })

src/nft/components/bag/profile/utils.ts

@@ -147,6 +147,7 @@ export const getListings = (sellAssets: WalletAsset[]): [CollectionRow[], Listin
         status: ListingStatus.DEFINED,
         asset,
         marketplace,
+        price: asset.newListings?.find((listing) => listing.marketplace.name === marketplace.name)?.price,
       }
       newListings.push(newListing)
       if (

src/nft/hooks/useNFTList.ts

@@ -33,7 +33,32 @@ export const useNFTList = create<NFTListState>()(
       }),
     setListings: (listings) =>
       set(() => {
-        return { listings }
+        const updatedListings = listings.map((listing) => {
+          const oldStatus = get().listings.find(
+            (oldListing) =>
+              oldListing.asset.asset_contract.address === listing.asset.asset_contract.address &&
+              oldListing.asset.tokenId === listing.asset.tokenId &&
+              oldListing.marketplace.name === listing.marketplace.name &&
+              oldListing.price === listing.price
+          )?.status
+          const status = () => {
+            switch (oldStatus) {
+              case ListingStatus.APPROVED:
+                return ListingStatus.APPROVED
+              case ListingStatus.FAILED:
+                return listing.status === ListingStatus.SIGNING ? ListingStatus.SIGNING : ListingStatus.FAILED
+              default:
+                return listing.status
+            }
+          }
+          return {
+            ...listing,
+            status: status(),
+          }
+        })
+        return {
+          listings: updatedListings,
+        }
       }),
     setCollectionsRequiringApproval: (collections) =>
       set(() => {

src/nft/types/sell/sell.ts

@@ -104,6 +104,7 @@ export interface AssetRow {
 export interface ListingRow extends AssetRow {
   asset: WalletAsset
   marketplace: ListingMarket
+  price?: number
 }
 
 export interface CollectionRow extends AssetRow {