search

Synthetixio / synthetix-exchange

The Synthetix Exchange dApp code
https://synthetix.exchange
121 stars 92 forks source link

feat(TradeChart): add user trade markers to the trade section chart #450

Closed dvd-schwrtz closed 4 years ago

vercel[bot] commented 4 years ago

This pull request is being automatically deployed with Vercel (learn more). To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/synthetixio/synthetix-exchange/az2w77727 ✅ Preview: https://synthetix-exchange-git-trade-markers.synthetixio.vercel.app

evgenyboxer commented 4 years ago

@clementbalestrat could you have a go - its a bit hard to test without the graph on Kovan... I actually don't have a recent trades on MainNet...

0xclem commented 4 years ago

@evgenyboxer yes I'll have a look at it today.

evgenyboxer commented 4 years ago

@dvd-schwrtz LGTM. just need @clementbalestrat to test this behaviour.