Skip to content

Added toTs, allData, and aggregatePredictableTimePeriods to options#31

Merged
RyanZim merged 2 commits into
ExodusOSS:masterfrom
ourarash:master
Nov 7, 2018
Merged

Added toTs, allData, and aggregatePredictableTimePeriods to options#31
RyanZim merged 2 commits into
ExodusOSS:masterfrom
ourarash:master

Conversation

@ourarash
Copy link
Copy Markdown
Contributor

@ourarash ourarash commented Nov 1, 2018

No description provided.

@RyanZim
Copy link
Copy Markdown
Contributor

RyanZim commented Nov 1, 2018

Where are the official docs for these options?

@ourarash
Copy link
Copy Markdown
Contributor Author

ourarash commented Nov 1, 2018

They are here

https://min-api.cryptocompare.com

Endpoint Parameters:
tryConversion: If set to false, it will try to get only direct trading values
REQUIRED fsym: The cryptocurrency symbol of interest [Max length: 10]
REQUIRED tsym: The currency symbol to convert into [Max length: 10]
e: The exchange to obtain data from (our aggregated average - CCCAGG - by default) [Max length: 30]
aggregate: Time period to aggregate the data over (for daily it's days, for hourly it's hours and for minute histo it's minutes)
aggregatePredictableTimePeriods: True by default, only used when the aggregate param is also in use. If false it will aggregate based on the current time. If the param is false and the time you make the call is 1pm - 2pm, with aggregate 2, it will create the time slots: ... 9am, 11am, 1pm. If the param is false and the time you make the call is 2pm - 3pm, with aggregate 2, it will create the time slots: ... 10am, 12am, 2pm. If the param is true (default) and the time you make the call is 1pm - 2pm, with aggregate 2, it will create the time slots: ... 8am, 10am, 12pm. If the param is true (default) and the time you make the call is 2pm - 3pm, with aggregate 2, it will create the time slots: ... 10am, 12am, 2pm.
limit: The number of data points to return
allData: Returns all data (only available on histo day)
toTs: Last unix timestamp to return data for
extraParams: The name of your application (we recommend you send it) [Max length: 2000]
sign: If set to true, the server will sign the requests (by default we don't sign them), this is useful for usage in smart contracts

@RyanZim RyanZim requested a review from dooglio November 1, 2018 19:20
Copy link
Copy Markdown

@dooglio dooglio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM utACK

@RyanZim RyanZim merged commit 4be81f7 into ExodusOSS:master Nov 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants