off-peak icon indicating copy to clipboard operation
off-peak copied to clipboard

Feature/more time periods

Open hendrikdotse opened this issue 1 year ago • 2 comments

Added: last-year, this-year, last-month as time periods to choose from. Did some dynamic work on titles with new options. Fixed an issue when no data was returned from API you would get stuck in Error state.

Im new to React, so if something is unclear or seems way off or you just don't like the PR please reach out.

regards Hendrik

hendrikdotse avatar Dec 15 '22 15:12 hendrikdotse

It's a nice start and I have no objections to this - but getting around tibbers limitation of ~ 750 hours per request will be key.

There is if I'm not mistaken there is a cursor property on the consumption and pricing data returned from Tibbers GraphQL API, so using a stepper month-by-month is probably the better option. That will both assure that we don't hammer the API and get people rate limited, and the implementation app-side will be quite a bit easier. The current rate limit is 100 requests per 5 minutes, changing page a couple of times could risk that of complete reloads are performed each time.

dhogborg avatar Dec 15 '22 20:12 dhogborg

Understand, will read up on tibbers quotas and update with new solution.

hendrikdotse avatar Dec 15 '22 21:12 hendrikdotse