Skip to content
This repository has been archived by the owner on Feb 16, 2020. It is now read-only.

Updated config file to comply with new Bitstamp API version. Simple erro... #102

Closed
wants to merge 1 commit into from

Conversation

willeeklund
Copy link

Updated config file to comply with new Bitstamp API version.

Please also consider creating a new release of the npm module. The 0.0.3 release today does not take the new version of npm module "bitstamp" into consideration, making it uncompliant with new updated Bitstamp API.

Thanks for a great repo!

@askmike
Copy link
Owner

askmike commented Dec 31, 2013

Thanks, I will merge this in even though the current dataprovider for bitstamp (bitcoincharts) is not recommended ( see #77 ).

We are working hard on a new version which keeps a local database for historical data so we can leverage the Bitstamp's own API. Check it out here (note you should consider it alpha). Here you can read up about the discussion (it started out as a problem with CEX.io, but the same goes for bitstamp).

When that branch is nearing a stable release we'll make sure that the Bitstamp trader is updated to work again.

Please also consider creating a new release of the npm module.

I am the author of the module and it's already updated for a month or so, however due this new API we couldn't get Gekko running on it anymore reliably. Because the module changed to much updating the required one for Gekko would break Gekko.

@askmike
Copy link
Owner

askmike commented Feb 13, 2014

This has been fixed since 0.1.0.

@askmike askmike closed this Feb 13, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants