XChange icon indicating copy to clipboard operation
XChange copied to clipboard

[Okex] Move the OKX streaming exchange to new module

Open makarid opened this issue 2 years ago • 2 comments

makarid avatar Oct 13 '22 11:10 makarid

why?

timmolter avatar Oct 15 '22 18:10 timmolter

There is a conflict when you call the exchange using string as parameters. Okcoin is a different exchange, so we need to have 2 modules one okcoin and one okex in my opinion. Also to be more organized

On Sat, Oct 15, 2022, 9:19 PM Tim Molter @.***> wrote:

why?

— Reply to this email directly, view it on GitHub https://github.com/knowm/XChange/pull/4597#issuecomment-1279800094, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHIWQ72RRTHLVA5BVP2JPHLWDLYUXANCNFSM6AAAAAAREGCA4M . You are receiving this because you authored the thread.Message ID: @.***>

makarid avatar Oct 15 '22 21:10 makarid

I change the name of the module from okx to okex in order to be consistent with xchange-okex module

makarid avatar Oct 19 '22 11:10 makarid