Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

deribit implementation #1032

Closed
wants to merge 4 commits into from
Closed

deribit implementation #1032

wants to merge 4 commits into from

Conversation

geseq
Copy link
Contributor

@geseq geseq commented Sep 14, 2022

PR Description

Is someone still working on the deribit implementation?

I have some work here and thought I'd share in case there's interest. It is based on #705 but also has added websocket implementation for creating orders.

It has been tested against the live exchange, but will of course need to be built upon to add the other endpoints to complete the implementation. Happy to work on fleshing this out further (as time permits) if it's of interest and no one is working on it already.

@gloriousCode gloriousCode requested a review from a team September 14, 2022 22:18
@gloriousCode gloriousCode added the review me This pull request is ready for review label Sep 14, 2022
@thrasher-
Copy link
Collaborator

Firstly, thank you for looking into this! ATM no one is working on Deribit aside from the PR you have already seen, but after OKX v5/Kucoin and Gateiov4 this is next on the list. The goal is to cover all the highest volume derivative exchanges based on https://ftx.com/volume-monitor

@gloriousCode gloriousCode marked this pull request as draft September 19, 2022 00:17
@gloriousCode gloriousCode added help wanted and removed review me This pull request is ready for review labels Sep 19, 2022
@thrasher-
Copy link
Collaborator

@samuael has took this exchange implementation on and is working on it here: https://github.com/samuael/gocryptotrader/tree/gderibit :)

@thrasher-
Copy link
Collaborator

Superseded by #1082

@thrasher- thrasher- closed this Nov 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants