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

More tickers #2700

Merged
merged 13 commits into from
Mar 18, 2018
Merged

More tickers #2700

merged 13 commits into from
Mar 18, 2018

Conversation

bziemek
Copy link
Contributor

@bziemek bziemek commented Mar 18, 2018

Add: Binance, Crypto-Bridge, Mercatox, SouthXChange, TradeOgre, and Yobit
Remove: Cryptsy
Fix: BTC-e

It's late, and I only eyeballed these. Needs tests.

@scottmacrae
Copy link

Good work any chance to add Trade Orge we need to pick up nah value in strayapool.com

https://tradeogre.com/help/api

@bziemek
Copy link
Contributor Author

bziemek commented Mar 18, 2018

The target for TradeOgre should be i.e /api/v1/ticker/BTC-NAH

@scottmacrae
Copy link

scottmacrae commented Mar 18, 2018 via email

@bziemek
Copy link
Contributor Author

bziemek commented Mar 18, 2018

@scottmacrae TradeOgre's webserver 404s if you have doubled up slashes.

$config['price']['url'] = 'https://tradeogre.com';
$config['price']['target'] ='/api/v1/ticker/BTC-NAH';

@bziemek
Copy link
Contributor Author

bziemek commented Mar 18, 2018

All tested; ready for merge

--

To be added to the Wiki later:

https://api.crypto-bridge.org
/api/v1/ticker

https://api.binance.com
/api/v3/ticker/price?symbol={BASE}{QUOTE}

https://yobit.com
/api/3/ticker/{base}_{quote}

https://www.southxchange.com
/api/price/{BASE}/{QUOTE}

https://mercatox.com
/public/json24

https://tradeogre.com
/api/v1/ticker/{QUOTE}-{BASE}

@TheSerapher TheSerapher merged commit a0f06e7 into MPOS:development Mar 18, 2018
@scottmacrae
Copy link

scottmacrae commented Mar 19, 2018 via email

@TheSerapher
Copy link
Contributor

Are you on the correct branch? Master does not have the change and Development just recently got it. Unless you checked out this PR specifically, this wasn't working anyway. And now it only works on development branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants