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

Stats: Add Last Updated Time Stats #10911

Closed
timmyc opened this issue Jan 25, 2017 · 3 comments · Fixed by #11035
Closed

Stats: Add Last Updated Time Stats #10911

timmyc opened this issue Jan 25, 2017 · 3 comments · Fixed by #11035
Assignees
Labels
Stats Everything related to our analytics product at /stats/ [Type] Enhancement

Comments

@timmyc
Copy link
Contributor

timmyc commented Jan 25, 2017

From pKdGS-181-p2 - In old stats, a timestamp at the top of the page denoted the time that the stats displayed on the page were last updated. At one point early in Calypso Stats, we attempted to re-create a similar timestamp on stats pages, but due to the various stats-lists and dozens of render()s on a stats page, it proved difficult.

We currently do not track the time each state/stats/list tree was last updated, but perhaps we could implement some tracking in the state tree level, or maybe track changes of the data in a state variable in connected-list?

@youknowriad any thoughts on the best way to track this?

Some of the user requests are specifically around the chart data, so perhaps we could start there.

@timmyc timmyc added Stats Everything related to our analytics product at /stats/ [Type] Enhancement labels Jan 25, 2017
@timmyc timmyc changed the title Stats: Add Last Updated Time Stamps Stats: Add Last Updated Time Stats Jan 25, 2017
@youknowriad
Copy link
Contributor

Yep, we could definitely track the last requested date on the state tree by stat query I guess.
But sometimes there are different components (Queries) on the same page, that would make it a bit difficult to show a timestamp by page.

@youknowriad
Copy link
Contributor

Maybe we could add a "time" icon with a hover showing this date for each component instead of at the top of each page?

@timmyc
Copy link
Contributor Author

timmyc commented Jan 25, 2017

Maybe we could add a "time" icon with a hover showing this date for each component instead of at the top of each page?

I was kind of thinking the same thing. And yes since the data can update independently, per module it is kind of a difficult design task.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Stats Everything related to our analytics product at /stats/ [Type] Enhancement
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants