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

add basic site analytics + check for which posts to update after bulk import #36

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

bnjmacdonald
Copy link
Contributor

Implements basic versions of two new utilities:

(1) A request_report utility that makes it easy to request a simple Google Analytics report (using the reporting API v4).

Note: this is implemented as an installed application, as opposed to a service application (which means that every time you use the utility, it opens a web browser and you have to sign in with an OWID google account). This should be implemented as a service application in the future, but I don't have admin access to Google Analytics in order to add the service account address.

(2) A posts_to_update utility that is intended to make it easy to see which public OWID pages need to be checked for textual inconsistencies after one or more suggested chart revisions have been approved as part of a bulk dataset update (ordered by number of page views in the past 90 days).

These two utilities are in the same PR b/c (2) is dependent on (1).

Adds the ability to make a simple Google Analytics report request using
the reporting API v4.
Adds a `posts_to_update` module, which constructs a dataframe of the
OWID pages that should be checked for textual inconsistencies following
the approval of suggested chart revisions as part of a bulk dataset
update.

Each row is an OWID page, including the number of chart updated on the
page and the number of page views (from Google Analytics) over the past
90 days.
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.

1 participant