Skip to content

[Stats Refresh] correct Posting Activity data - #11765

Merged
ScoutHarris merged 1 commit into
developfrom
fix/11763-posting_activity_data
May 23, 2019
Merged

[Stats Refresh] correct Posting Activity data#11765
ScoutHarris merged 1 commit into
developfrom
fix/11763-posting_activity_data

Conversation

@ScoutHarris

Copy link
Copy Markdown
Contributor

Fixes #11763

So, the limit parameter bit us again. An unspecified limit defaults to 10. limit: 0 doesn't return all. StatsPostingStreakInsight.queryProperties uses 5000, so I set the limit to 5000. And voila, we have a year's worth of data once again.

To test:

  • Go to Stats > Insights > Posting Activity for an active site.
    • Tip: Hogwarts is a good one.
  • Select View more.
    • The Insights card and details view use the same data, so verifying details should do it.
  • Compare the data with Calypso (https://wordpress.com/stats/insights/<site_url>).

Calypso for Hogwarts:
calypso

Insights card:
insights

Details:
details

Update release notes:

  • If there are user facing changes, I have added an item to RELEASE-NOTES.txt.

@ScoutHarris ScoutHarris added this to the 12.6 milestone May 23, 2019
@ScoutHarris
ScoutHarris requested a review from a user May 23, 2019 17:02
@ScoutHarris ScoutHarris self-assigned this May 23, 2019
@ScoutHarris ScoutHarris mentioned this pull request May 23, 2019
45 tasks

@ghost ghost left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ScoutHarris : looks good to me!
:shipit:

@ScoutHarris
ScoutHarris merged commit 0df22aa into develop May 23, 2019
@ScoutHarris
ScoutHarris deleted the fix/11763-posting_activity_data branch May 23, 2019 21:55
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.

[Stats Refresh] Posting Activity data is no longer correct

1 participant