Skip to content
This repository was archived by the owner on Mar 7, 2025. It is now read-only.

Add analytics#147

Merged
katiejiang merged 2 commits intomasterfrom
google-analytics
Jan 31, 2018
Merged

Add analytics#147
katiejiang merged 2 commits intomasterfrom
google-analytics

Conversation

@katiejiang
Copy link
Contributor

@katiejiang katiejiang commented Jan 31, 2018

  • Include Google analytics by setting environment variable GOOGLE_ANALYTICS_ID
  • Include Segment analytics by setting environment variable SEGMENT_API_KEY
  • Update documentation for analytics

@sandlerben sandlerben temporarily deployed to flask-base-h4i-pr-147 January 31, 2018 15:42 Inactive
@katiejiang katiejiang requested a review from abhisuri97 January 31, 2018 15:42
@sandlerben
Copy link
Contributor

amazing

@sandlerben
Copy link
Contributor

would we consider using segment.com?

@katiejiang
Copy link
Contributor Author

@sandlerben I can add an option for segment!

@katiejiang
Copy link
Contributor Author

We can probably leave the decision of which analytics platform to our clients!

@katiejiang katiejiang changed the title Add Google analytics Add analytics Jan 31, 2018
@sandlerben
Copy link
Contributor

The nice thing about segment is it forwards the data to any analytics provider you want (google, mixpanel, etc)!

@katiejiang
Copy link
Contributor Author

^This would be a great thing to include in our hand-off document actually

@katiejiang katiejiang merged commit 05aac45 into master Jan 31, 2018
@katiejiang katiejiang deleted the google-analytics branch January 31, 2018 20:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants