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

doc: add google analytics tracking to docs #16666

Merged
merged 1 commit into from
Jun 7, 2019
Merged

doc: add google analytics tracking to docs #16666

merged 1 commit into from
Jun 7, 2019

Conversation

dbkinder
Copy link
Contributor

@dbkinder dbkinder commented Jun 6, 2019

This adds code to each page needed for gathering google analytics
tracking data to the LF-provided Global site tag.

Signed-off-by: David B. Kinder david.b.kinder@intel.com

This adds code to each page needed for gathering google analytics
tracking data to the LF-provided Global site tag.

Signed-off-by: David B. Kinder <david.b.kinder@intel.com>
@zephyrbot
Copy link
Collaborator

zephyrbot commented Jun 6, 2019

All checks are passing now.

Review history of this comment for details about previous failed status.
Note that some checks might have not completed yet.

@dbkinder
Copy link
Contributor Author

dbkinder commented Jun 6, 2019

@nashif When this demonstrates as working, it would be good to backport this to the LTS (1.14) documentation too (probably don't need to go back to any earlier release).

@nashif nashif merged commit aa9b602 into zephyrproject-rtos:master Jun 7, 2019
@aescolar
Copy link
Member

aescolar commented Jun 7, 2019

@dbkinder Why do we need this?

@dbkinder
Copy link
Contributor Author

dbkinder commented Jun 7, 2019

@aescolar collecting google analytics data will let us know more about how visitors are using the documentation and visitor flow through the site. It also let’s us find, for example, broken incoming links from external sites. Our intention is to use this to improve our documentation and experience.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants