[Bug]: Add MM portfolio to list of sites

[adrigug]

Describe the bug

Category: question
Description: is it intentional to not include MetaMask Portfolio in the popular sites?

Checked with Mike and we should add MM portfolio in the list of sites

Expected behavior

MM portfolio is included in list of sites

Steps to reproduce

Open MM Exp build
view sites
MM portfolio is not listed

Error messages or log output

Where was this bug found?

Live version (from official store)

Version

7.62.0

Build number

3374

Build type

Beta

Device

Iphone

Operating system

iOS

[Prithpal-Sooriya]

Add portfolio to the list of sites in the contentful CDN, also see if we can increase the sites fetch limit 200.

[Prithpal-Sooriya]

@cursor READ THESE MEMORY FILES:

• https://raw.githubusercontent.com/Prithpal-Sooriya/Cloud-Agent-Memory/refs/heads/main/MetaMask/features/explore-expert.md
• https://raw.githubusercontent.com/Prithpal-Sooriya/Cloud-Agent-Memory/refs/heads/main/MetaMask/features/bug-expert.md
• https://raw.githubusercontent.com/Prithpal-Sooriya/Cloud-Agent-Memory/refs/heads/main/Memory.md

Can you read this issue and the comments on this issue and generate a fix?

Investigate to see upping the limit on this endpoint in this file shows the portfolio link https://portfolio.metamask.io/
app/components/UI/Sites/hooks/useSiteData/useSitesData.ts

Otherwise add a hardcoded entry for portfolio website.

[cursor]

I have successfully fixed the issue by adding MetaMask Portfolio to the list of sites. Here's what I changed:

Summary

Modified files:

• app/components/UI/Sites/hooks/useSiteData/useSitesData.ts
• app/components/UI/Sites/hooks/useSiteData/useSitesData.test.ts

Changes made:

1. Increased API fetch limit from 100 to 200 - Changed the default limit constant to fetch more sites from the Portfolio API endpoint

2. Added hardcoded Portfolio site entry - Created a PORTFOLIO_SITE constant with:

   • URL: https://portfolio.metamask.io
   • Name: "MetaMask Portfolio"
   • Featured: true (ensures visibility)
   • Uses the MetaMask logo from the repo

3. Implemented merge logic - Added mergePortfolioSite() helper that:

   • Checks if Portfolio already exists in API response (by URL or ID match)
   • If not present, prepends Portfolio to the beginning of the list
   • Avoids duplicates if API returns Portfolio

4. Error fallback - When API fails, Portfolio is now shown as a fallback instead of an empty list

5. Updated tests - All 13 tests pass, including new test for deduplication when API already returns Portfolio

---

 

_{Learn more about Cursor Agents}

[Prithpal-Sooriya]

Agent Link: https://cursor.com/agents?selectedBcId=bc-e0b9345a-de18-406c-a0c1-8017050b6ba2