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

WIP kraken support #21

Merged
merged 2 commits into from
Aug 16, 2016
Merged

WIP kraken support #21

merged 2 commits into from
Aug 16, 2016

Conversation

grigio
Copy link
Contributor

@grigio grigio commented Aug 16, 2016

No description provided.

@grigio
Copy link
Contributor Author

grigio commented Aug 16, 2016

Minimal support #9

  • it calls the public api to get BTCUSD rates

I'm not sure if Kraken supports &limit parameter

@carlos8f carlos8f merged commit 1e3af89 into DeviaVir:master Aug 16, 2016
@grigio
Copy link
Contributor Author

grigio commented Aug 16, 2016

@carlos8f I see teh data processing, but I don't see the graph on localhost.. I get "no data"

@carlos8f
Copy link
Contributor

hmm, I'll test it out

@grigio
Copy link
Contributor Author

grigio commented Aug 16, 2016

The backfill time is always the same

@carlos8f
Copy link
Contributor

fixed it in master, the since param is actually microtime, not a standard timestamp

@grigio
Copy link
Contributor Author

grigio commented Aug 16, 2016

Thanks!

supersabbath pushed a commit to supersabbath/zenbot that referenced this pull request Oct 2, 2017
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