Google-Analytics-Popular-Posts icon indicating copy to clipboard operation
Google-Analytics-Popular-Posts copied to clipboard
verified publisher icon INN

Improvements.

Open foliovision opened this issue 8 years ago • 2 comments

Increased the stats from 2 days to 90 day. Fixing SQL generator when url_to_postid() returns 0. Adding stats_get_csv() to replace Jetpack stats Adding quick stats view to the options screen.

foliovision avatar Jan 16 '17 13:01 foliovision

Can you split this up into separate pull requests for:

  • Making the stat retention duration into 90 days (we'd like to make this filterable)
  • Fixing the SQL generation
  • The shim for Jetpack's stats_get_csv()
  • Adding quick stats view to the options screen
    • this should use stats_get_csv() instead of fv_ga_stats_get_csv()

Thanks,

Ben

benlk avatar Feb 01 '17 17:02 benlk

Hello Ben,

  1. Increased the stats from 2 days to 90 day - we made it an option, pull request is here: https://github.com/INN/Google-Analytics-Popular-Posts/pull/67

  2. SQL generation is fixed here: https://github.com/INN/Google-Analytics-Popular-Posts/pull/68

  3. JetPack Stats bridge (long description there): https://github.com/INN/Google-Analytics-Popular-Posts/pull/69

  4. Adding quick stats view to the options screen (ticky, as above pull request is part of it): https://github.com/INN/Google-Analytics-Popular-Posts/pull/70

Thanks, Martin

foliovision avatar Feb 02 '17 12:02 foliovision