Skip to content

Hide price alerts on CoinGecko charts that do not publish a live price. - #432

Open
manish-rorotechnology wants to merge 1 commit into
masterfrom
feature/hide-coingecko-price-alerts
Open

manish-rorotechnology wants to merge 1 commit into
masterfrom
feature/hide-coingecko-price-alerts

Conversation

@manish-rorotechnology

Copy link
Copy Markdown

Swap and margin charts keep the alert tool.

Problem

What was wrong, and where (page, component, contract call). Link the task.

Change

What this PR changes. If it differs from what the task asked for, say so and why.

How it was verified

  • Typecheck and lint pass (the Checks workflow runs them on this PR)
  • Tried in the app: steps, network, and wallet used
  • Checked against the contract or subgraph when the change depends on on-chain behaviour

Left open

Anything not done, known limits, or follow-ups. Write "None" if nothing is open.

Retest

Who should retest after merge, and what to check. Merging to master updates test-app.dex223.io within a few minutes. app.dex223.io changes only when the staged production build is promoted in Vercel, after the retest passes on test-app.

Swap and margin charts keep the alert tool.

Co-authored-by: Cursor <cursoragent@cursor.com>
@vercel

vercel Bot commented Nov 9, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated
dex223-ui-blog Ready Ready Preview Nov 9, 2026 10:29am UTC
dex223-ui-web Ready Ready Preview Nov 9, 2026 10:29am UTC

Request Review

This branch was successfully deployed

2 active deployments
Preview – dex223-ui-web — d5e9a806 Deployed Nov 9, 2026 by vercel[bot]
Preview – dex223-ui-blog — d5e9a806 Deployed Nov 9, 2026 by vercel[bot]
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.

1 participant