Skip to content

Conversation

@timmyc
Copy link
Contributor

@timmyc timmyc commented Dec 29, 2016

This branch is based on a somewhat informed hunch between some investigating that @jblz and I have done in regards to #9529

While attempting to calculate the size of the indexDb instance, Jeff was hitting some oddness around the statsStreak persisted data. In this branch, I have disabled the serialization of the stats list items to see if this might be a possible fix.

To Test
Open a site stats insights page, and ensure the "Post Trends" data still loads as expected

@timmyc timmyc added the [Feature] Stats Everything related to our analytics product at /stats/ label Dec 29, 2016
@timmyc timmyc requested a review from jblz December 29, 2016 18:43
@matticbot
Copy link
Contributor

@jblz
Copy link
Member

jblz commented Dec 29, 2016

Not sure if it's related to this change or not, but if I do a localStorage.clear() & reload the insights page, the statsStreak state key never gets set (until I do a full page reload or navigate away and back).

Otherwise, this seems to work ok.

@timmyc
Copy link
Contributor Author

timmyc commented Dec 30, 2016

closing in favor of #10326

@timmyc timmyc closed this Dec 30, 2016
@timmyc timmyc deleted the fix/stats/state branch December 30, 2016 00:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Feature] Stats Everything related to our analytics product at /stats/

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants