Cryptorush API Price Ticker #1931
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Simple yet effective way to get ticker cron to use the Cryptorush.in API via get=all parameter. Just enter on your global cfg file: $config['price']['url'] = 'https://cryptorush.in'; $config['price']['target'] = '/api.php?get=all&key=YOUR_API_KEY&id=YOUR_USER_ID'; $config['price']['currency'] = 'COIN'; and run the cron. Enjoy!
Contributor
There was a problem hiding this comment.
I don't think you need to do this variable assignment here. Just address it in the array directly.
Contributor
|
As you said, 5 lines of code. Nice work! Will test tomorrow. |
Contributor
Author
|
👍 |
Contributor
|
Once we merge this, can you add the Wiki documentation for your new option? |
Contributor
|
I checked their website, they say no BTC trading online right now. Is that maintenance? |
Contributor
Author
|
cryptorush is currently under maint ye. Should be back up by this eve UTC i think. BTC wise anywho |
TheSerapher
added a commit
that referenced
this pull request
Mar 14, 2014
Cryptorush API Price Ticker
Contributor
|
Merged but added the proper target currency too, we already have it as an option so we may as well use it: 914bc6b |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Simple yet effective way to get ticker cron to use the Cryptorush.in API
via get=all parameter.
MIGHT NOT BE THE MOST EFFECTIVE WAY! comments highly welcome 👍
After you have added the commit
Just add the following in your global cfg file:
$config['price']['url'] = 'https://cryptorush.in';
$config['price']['target'] ='/api.php?get=all&key=YOUR_API_KEY&id=YOUR_USER_ID';
$config['price']['currency'] = 'BTC';
The above represents the price for your coin vs Bitcoin.
Run the cron
Enjoy!