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

CEX.io syntax error #61

Closed
wants to merge 2 commits into from
Closed

CEX.io syntax error #61

wants to merge 2 commits into from

Conversation

leefernandes
Copy link
Contributor

The "pair" syntax for CEX.io was incorrect. It should be formatted like: /GHS/BTC

17NuF432QCV2vF4fdqYPYMdCuK9jHVDSNG

17NuF432QCV2vF4fdqYPYMdCuK9jHVDSNG added 2 commits December 4, 2013 03:24
cexio pair syntax fix - GET https://cex.io/api/trade_history/GHS/BTC

17NuF432QCV2vF4fdqYPYMdCuK9jHVDSNG
remove trailing slash
@askmike
Copy link
Owner

askmike commented Dec 6, 2013

Thanks, unfortunately I am offline this weekend, but will def take a look as soon as I get back.

@danpe
Copy link

danpe commented Dec 7, 2013

Confirmed Issue + Fix 👍

@silverbucket
Copy link

That patch is not correct though, the cexio module uses the ghs_btc format for the pair string, then does a split and converts to uppercase when it needs to. So it's expecting the _ to split on.

I think the problem is using USD as a currency, as cex.io does not deal with USD.

However, once setting the currency to GHS or BTC, I get the error reported in this issue:
#72

@askmike
Copy link
Owner

askmike commented Dec 21, 2013

Fetching historical trades is currently broken at CEX.io because of a change in API behaviour. See this issue for more information.

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.

None yet

4 participants