Skip to content

Conversation

@premiumjibles
Copy link
Collaborator

@premiumjibles premiumjibles commented Jan 15, 2026

Description

Portals api is throwing errors that it currently doesn't support Gnosis and Monad even though the docs say it does. The API on the explore page on mobile returns status 400 : "each value in networks must be one of the following values: ethereum, optimism, arbitrum, polygon, avalanche, bsc, base, sonic, hyperevm". Removing it for now to un-break the explore page but we can check later if this gets fixed on their end and revert.

Issue (if applicable)

closes #11652

Risk

Low risk, we could lose access to some gnosis stuff on portals but it's already completely breaking some pages.

High Risk PRs Require 2 approvals

What protocols, transaction types, wallets or contract interactions might be affected by this PR?

Testing

  • Explore page on mobile loads ok when clicking into sub sections
  • Sanity check swaps to and from gnosis show up with quotes
  • Make sure one click defi still loads as expected on the markets page

Engineering

Operations

  • 🏁 My feature is behind a flag and doesn't require operations testing (yet)

Screenshots (if applicable)

image

@premiumjibles premiumjibles requested a review from a team as a code owner January 15, 2026 05:47
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 15, 2026

📝 Walkthrough

Walkthrough

This PR removes Gnosis and Monad chain support from the Portals swapper integration by deleting chain ID imports and their corresponding mappings across three files. The changes eliminate chains that are no longer supported in the swap routing logic.

Changes

Cohort / File(s) Summary
Portals chain mapping removals
packages/swapper/src/swappers/PortalsSwapper/constants.ts, src/lib/portals/constants.ts, src/state/apis/portals/utils.ts
Removed Gnosis and Monad chain support by deleting chain ID mappings from CHAIN_ID_TO_PORTALS_NETWORK and PORTALS_NETWORKS_TO_CHAIN_ID_MAP constants, and removed SupportedPortalsNetwork.Gnosis enum member. Chains no longer resolve in Portals swapper lookups.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Suggested reviewers

  • gomesalexandre
  • NeOMakinG

Poem

🐰 Gnosis and Monad wave goodbye,
As Portals chains streamline and fly,
Mappings removed with surgical care,
Cleaner networks float through the air! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: removing Monad and Gnosis support from Portals, which directly addresses the root cause of the 'No results found' issue on the Explore page.
Linked Issues check ✅ Passed The code changes successfully remove Monad and Gnosis from Portals support, directly addressing the Portals API errors that were causing the 'No results found' issue on the Explore page.
Out of Scope Changes check ✅ Passed All changes are scoped to removing Monad and Gnosis from Portals support across three files, with no unrelated modifications or out-of-scope alterations detected.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch 11652_fix_portals

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Collaborator

@NeOMakinG NeOMakinG left a comment

Choose a reason for hiding this comment

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

Clean removal, makes sense to disable until Portals actually supports these networks

Here we go, now it's clickable again!

image

@NeOMakinG NeOMakinG merged commit 16c6b67 into develop Jan 15, 2026
4 checks passed
@NeOMakinG NeOMakinG deleted the 11652_fix_portals branch January 15, 2026 09:43
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.

Subheadings on the explore page on mobile are showing no results

3 participants