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

Update time horizons to show more data (#716) #928

Merged
merged 1 commit into from
Oct 2, 2020
Merged

Conversation

robhudson
Copy link
Member

@robhudson robhudson commented Oct 1, 2020

I consider this a working, work in progress.

TODO:

  • Investigate triggering a re-fetch when the time horizon changes to > MONTH, and default back to a smaller number of versions requested from the API to keep the data response smaller by default
  • Investigate making the API align more with the UI by request a number of builds rather than number of versions

@robhudson
Copy link
Member Author

Talked to hamillton and this is good enough for now. We agreed adjusting the API or re-querying the API may be better, but fetching 10 versions and slicing the data via the time horizon buttons is an good fix for now.

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.

1 participant