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

[Feature Request] Parse data in response headers #68

Open
iuvbio opened this issue Jul 23, 2020 · 3 comments
Open

[Feature Request] Parse data in response headers #68

iuvbio opened this issue Jul 23, 2020 · 3 comments

Comments

@iuvbio
Copy link

iuvbio commented Jul 23, 2020

CoinAPI provides data on request limits, exchanges used in aggregation of OHLCV data, and possibly more that I'm not aware of in the response headers. As far as I can see, the response header is currently neither parsed nor stored. I think it would be good to capture that information, either by parsing it for the relevant request or by just giving the CoinAPIv1 class an attribute response_header and leaving it to the user to parse it. I have only looked at the Python SDK, but I imagine the other languages are similar.

Do you think this makes sense generally and one of the implementations above specifically? If so, I'm happy to make a PR for the Python SDK.

@Svisstack
Copy link
Contributor

This suggestion is valid as we are tracking it, however can't provide a timeline for that at this moment.

@coinapi coinapi deleted a comment from qqqq-collab Jun 18, 2021
@Svisstack
Copy link
Contributor

@iuvbio Will you be making PR for this?

@iuvbio
Copy link
Author

iuvbio commented Jun 19, 2021

I hope to get some time to take a look in the next weeks, this has been completely off my radar for a while.

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

No branches or pull requests

2 participants