Skip to content

(test-only) trading costs dashboard#2499

Closed
qwinndev wants to merge 18 commits into
releasefrom
trading-costs-dashboard
Closed

(test-only) trading costs dashboard#2499
qwinndev wants to merge 18 commits into
releasefrom
trading-costs-dashboard

Conversation

@qwinndev

Copy link
Copy Markdown
Contributor

Summary

Draft test-only PR for the trading costs dashboard work.

This includes:

  • core trading-cost helpers and market matching
  • Hyperliquid market/book data and cost estimates
  • GMX protocol fee, price impact, rate, and network fee estimates
  • the /costs dashboard route and UI
  • fixes for Arbitrum production market sourcing and the holding-period app dropdown

Validation

  • yarn test:ci src/domain/tradingCosts src/App/MainRoutes.spec.tsx
  • yarn test:ct src/pages/TradingCosts/__tests__/TradingCostsView.ct.spec.tsx
  • yarn tscheck:ci
  • Focused ESLint on changed TS/TSX files
  • Pre-push hook also ran full unit/component checks successfully during branch push.

Notes

Test-only draft. Not ready for review/merge.

@qwinndev qwinndev changed the title Test-only: trading costs dashboard (test-only) trading costs dashboard May 27, 2026
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented May 28, 2026

Copy link
Copy Markdown

Deploying gmx-interface-home with  Cloudflare Pages  Cloudflare Pages

Latest commit: 3b6e7b9
Status: ✅  Deploy successful!
Preview URL: https://3ffdb357.gmx-interface-home.pages.dev
Branch Preview URL: https://trading-costs-dashboard.gmx-interface-home.pages.dev

View logs

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented May 28, 2026

Copy link
Copy Markdown

Deploying gmx-interface with  Cloudflare Pages  Cloudflare Pages

Latest commit: 3b6e7b9
Status: ✅  Deploy successful!
Preview URL: https://1d6f6614.gmx-interface.pages.dev
Branch Preview URL: https://trading-costs-dashboard.gmx-interface.pages.dev

View logs

@github-actions

Copy link
Copy Markdown

Playwright Component Test Report

All tests passed.

View Report

Commit 3b6e7b9

@qwinndev

Copy link
Copy Markdown
Contributor Author

Combined into #2556.

@qwinndev qwinndev closed this Jun 10, 2026
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