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

Older URI parameter for pageSize, should be page-size #42

Open
krowvin opened this issue Aug 19, 2024 · 0 comments
Open

Older URI parameter for pageSize, should be page-size #42

krowvin opened this issue Aug 19, 2024 · 0 comments

Comments

@krowvin
Copy link
Collaborator

krowvin commented Aug 19, 2024

I believe an older version of CDA ( RADAR ? ) used camelCase instead of kebab-case for the URI parameters.

"pageSize": -1, # always fetch all results

New Version

image

This came up because a user was wanting to fetch a months worth of hourly data and interlace / pull out the 8am values for the report. (They don't have a daily timeseries) and it is faster to pull one large call for hourly than make 31 8am calls.

In that spirit, should we try to handle this somehow and/or tell the user via WARN message that they hit the limit?

Related to

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

No branches or pull requests

1 participant