Skip to content
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 1 addition & 2 deletions src/components/trade/tradebox/across-bridge-modal.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -31,8 +31,7 @@ const COUNTERFACTUAL_CHAINS = [
{ id: polygon.id, name: "Polygon", shortName: "POLY" },
] as const;

// const POPULAR_SYMBOLS = new Set(["USDC", "USDT", "USDT0", "ETH", "WETH", "WBTC", "DAI", "cbBTC"]);
const POPULAR_SYMBOLS = new Set(["USDT", "USDT0"]);
const POPULAR_SYMBOLS = new Set(["USDC"]);
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

Restricting the popular symbols to only "USDC" might be too narrow for chains like Polygon and Arbitrum, where the bridged version of USDC (often symbolized as "USDC.e") is still commonly used. If the Across SDK returns the bridged variant for these chains, it will be filtered out by the logic on line 123, potentially leaving the user with no selectable tokens in the modal. Consider including "USDC.e" in the set to ensure better compatibility and a smoother user experience across all supported networks.

Suggested change
const POPULAR_SYMBOLS = new Set(["USDC"]);
const POPULAR_SYMBOLS = new Set(["USDC", "USDC.e"]);


const INTEGRATOR_ID = (import.meta.env.VITE_ACROSS_INTEGRATOR_ID ?? "0xdead") as `0x${string}`;
const ACROSS_API_KEY = import.meta.env.VITE_ACROSS_API_KEY as string | undefined;
Expand Down
Loading