Skip to content

fix(candles): forward initial websocket snapshot - #2122

Open
antonbelous0v wants to merge 1 commit into
dydxprotocol:mainfrom
antonbelous0v:anton/fix-tradingview-candle-race
Open

antonbelous0v wants to merge 1 commit into
dydxprotocol:mainfrom
antonbelous0v:anton/fix-tradingview-candle-race

Conversation

@antonbelous0v

Copy link
Copy Markdown

Summary

  • forward the latest candle from the initial WebSocket snapshot to TradingView
  • close the gap between the REST history request and WebSocket subscription
  • continue forwarding updates from the last seen candle without replaying older history
  • add regression coverage for the initial snapshot and subsequent updates

Testing

  • pnpm vitest run src/bonsai/websocket/__test__/candlesForTradingView.test.ts
  • pnpm eslint src/bonsai/websocket/candlesForTradingView.ts src/bonsai/websocket/__test__/candlesForTradingView.test.ts
  • pnpm tsc --noEmit

@antonbelous0v
antonbelous0v requested a review from a team as a code owner September 2, 2026 15:25
@vercel

vercel Bot commented Sep 2, 2026

Copy link
Copy Markdown

@AntonBelousovWEB is attempting to deploy a commit to the dYdX Trading Team on Vercel.

A member of the Team first needs to authorize it.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant