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

Adding Binance / Other Exchanges #29

Open
3 of 6 tasks
theimo1221 opened this issue Feb 17, 2018 · 12 comments
Open
3 of 6 tasks

Adding Binance / Other Exchanges #29

theimo1221 opened this issue Feb 17, 2018 · 12 comments
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed

Comments

@theimo1221
Copy link
Contributor

theimo1221 commented Feb 17, 2018

An issue to collect data and tasks needed for adding binance

  • Decrease Callback/ Load Amount to 1 in Reduction to one callback (All pairs) #40

  • Analyze compatibility
    Needed changes in Data call?
    Needed changes in Data storage (Multiple times same pair e.g. "ETH-USD")

  • Discuss form of presentation
    Original:Send all data to client, and hide/ show selection on clientside with js?
    Update: Sending all data should be no problem due to changes in Reduction to one callback (All pairs) #40
    Maybe rework show/ hide in the future, to show/ hide complete exchanges.

  • Add Dropdown to select exchange
    Update Show/ Hide Button is much easier

  • define format for get_data function
    Goal is to keep get_data modular for future.

  • exclude api calls from get_data function

When we change this we should change #28 aswell

Thanks and Greets

Theimo

@pmaji
Copy link
Owner

pmaji commented Feb 18, 2018

Yep, good idea and a big step forward. I read the Binance API info last night and it looks like a VERY different process that require a lot more work.

I think it best to put this on the back-burner in terms of development priorities and ensure we lock down the methodology for GDAX first, but anyone is welcome to give other-exchange integration a shot.

@theimo1221
Copy link
Contributor Author

If it´s so different, we should maybe consinder changing the pairs to Objejtc

  • Pair to Object
  • Passing Object to get_data to get individual hooks from apis

Additionally we should consider getting the update into 1 Package, except of having 1 for each pair, to highly decrease server load for Web Hosting (Further improvement of #10)

@MarkyVee
Copy link

Hey guys...doing great work. Any way of "time stamping" the transactions?

@pmaji
Copy link
Owner

pmaji commented Apr 14, 2018

@MarkyVee definitely something that has been considered and is not permanently something I don't PLAN to do, but I have tested the timestamp validity of the websocket multiple times and it is outside of the realm of what I would consider useful in terms of its trustworthiness. I want to be sure that I can stand behind whatever we report, and I don't have that degree of comfort yet with the timestamps. This is well documented issue on Stack and also in discussions with the folks at GDAX, so it may change in the future, but isn't an update I'm planning short-term.

@MarkyVee
Copy link

MarkyVee commented Apr 14, 2018 via email

@pmaji
Copy link
Owner

pmaji commented Apr 14, 2018

@MarkyVee It may actually may sense not to wait; the present state of hosting is perfectly stable, it's just a question of adding further additions and increased speed. Let's chat more via telegram so I can explain, but for your purposes I don't believe it makes sense to wait to mention it.

@pmaji pmaji added the enhancement New feature or request label Apr 29, 2018
@pmaji
Copy link
Owner

pmaji commented Apr 14, 2019

Will keep this open in case a developer comes along who would like to tackle the challenge of integrating with Binance / other exchanges beyond coinbase, but I don't presently have time to tackle this. I also think that such an endeavor would be best accomplished via a separate fork of this code as the app is pretty computationally heavy as is.

@pmaji pmaji added good first issue Good for newcomers help wanted Extra attention is needed labels Apr 14, 2019
@pmaji pmaji changed the title Adding Binance Adding Binance / Other Exchanges Apr 14, 2019
@3raserHead
Copy link

Hi, pmagi, Gdax is not really used anymore by traders, the majority moved to Binance and Bybit... It would be awesome to have the possibility to have this amazing tool on Binance and have the possibility to monitor each pairs on that exchange...

I hope one this will be implemented in future!

Thank you all for developing this precious tool!

@pmaji
Copy link
Owner

pmaji commented Jan 25, 2021

Thanks for this @3raserHead. I agree that that would be supremely helpful, but sadly, it requires a decent amount of time which I don't currently have to refactor the repo to make it extensible to all other exchanges. I keep this issue open hoping for other volunteer contributors to take a wheel, but sadly don't have time to invest in this myself right now.

@3raserHead
Copy link

3raserHead commented Jan 25, 2021 via email

@jihadd1987
Copy link

hi - thank you all - i want to ask how to get this tool ? whale crypto ? the website not working ( https://whales.cracklord.com)
thank you - i was using this tool before some years - now the website not working

@jihadd1987
Copy link

hi

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Development

No branches or pull requests

5 participants