Bug Report

[armenzg]

STR:

• Visit this link
  • Notice the 7 days param
• Look at the graph (make the window wide if the chart is not shown)

Expected results:

• See the last 7 days of results

Actual results:

• I see the results for the last 5 years

[drazisil-codecov]

@codecovdesign This report is an example of codecov/engineering-team#2170

[eliatcodecov]

@codecovdesign I think we're not being clear on what the interval selector is actually doing. This selector bins data points to the set interval. For example, selecting 7 days means that each data point is actually a binned time period and we're reporting a single result for that binned period. So the selector isn't a range, but rather a binning interval. This is quite confusing (and also referenced here)

We should likely clarify the language to at least make it clearer what this drop down is doing. Additionally, we should consider the addition of a true range selector as that's typically what users are looking for when they interact with this ui component.