search

MuckRock / muckrock

MuckRock's source code - Please report bugs, issues and feature requests to info@muckrock.com
https://www.muckrock.com
GNU Affero General Public License v3.0
114 stars 22 forks source link

Do we use top viewed requests anymore? #457

Closed mitchelljkotler closed 9 years ago

mitchelljkotler commented 9 years ago

So we eliminated google analytic calls from the daily stats, but its still trying to be used for setting the top view requests of the day. Can I just rip this code out? Also, should I remove the corresponding database column for page_views from FOIARequests?

https://app.getsentry.com/muckrock/app4858795herokucom/group/66820704/

morisy commented 9 years ago

I think we'll eventually want to make use of both of these features, but neither is a high priority nor, if they get ripped out, hard to put back down the road. Not sure why GA always gave bad data, but deleting that column wouldn't be a problem.