Skip to content

Conversation

@Holybasil
Copy link
Member

No description provided.

@Holybasil Holybasil requested a review from Copilot November 25, 2025 06:12
@vercel
Copy link

vercel bot commented Nov 25, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
frontends Ready Ready Preview Comment Nov 25, 2025 6:12am

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the ecosystem list base URL from the Netlify-hosted endpoint to a Vercel-hosted endpoint by introducing an environment variable NEXT_PUBLIC_ECOSYSTEM_BASE_URI. All hardcoded references to https://scroll-eco-list.netlify.app are replaced with dynamic references to the ecosystemOrigin variable.

Key changes:

  • Centralized the ecosystem base URL configuration using an environment variable
  • Updated all hardcoded URLs across multiple files to use the new ecosystemOrigin variable
  • Updated Next.js image configuration to allow the new Vercel hostname

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
src/apis/ecosystem.ts Changed ecosystemOrigin from hardcoded Netlify URL to environment variable and added new fetchEcosystemCategoryToTagsURL export
src/constants/networks.ts Replaced hardcoded bridge icon URLs with dynamic ecosystemOrigin references
src/constants/bridge.ts Replaced hardcoded icon URLs with dynamic ecosystemOrigin references
src/app/ecosystem/page.tsx Replaced hardcoded fetch URL with new fetchEcosystemCategoryToTagsURL constant
src/app/_sessions/Protocols/protocolList.ts Replaced hardcoded logo URLs with dynamic ecosystemOrigin references
src/app/_SCR-sSCR/Header/data.ts Replaced hardcoded logo URLs with dynamic ecosystemOrigin references
next.config.mjs Updated image hostname from Netlify to Vercel domain

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Holybasil Holybasil merged commit a268b30 into sepolia Nov 25, 2025
2 checks passed
@Holybasil Holybasil deleted the update-eco-list-base-url branch November 25, 2025 06:13
Holybasil added a commit that referenced this pull request Nov 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants