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

Google analytics #73

Merged
merged 6 commits into from
Apr 23, 2024
Merged

Google analytics #73

merged 6 commits into from
Apr 23, 2024

Conversation

egekorkan
Copy link
Member

@egekorkan egekorkan commented Mar 8, 2024

Doing some tests first and checking eclipse compliance. Checklist:

  • All committers on a project must have access to the data collected for that project upon request -> Ege Korkan and Cristiano Aguzzi are admins. Everyone else can be viewer. @danielpeintner , @JKRhb and Lorenzo Gigli have viewer access.
  • Eclipse Webmaster (webmaster@eclipse-foundation.org) must be an administrator on all Google Analytics properties
  • The Eclipse Foundation has right to audit the use of Google analytics and the data collected, and the project must support the Eclipse Foundation in performing the audit and adjust their analytical tooling if required
  • A user must be requested to give their consent, and explicit consent must be given by the user before data is collected regarding that user
  • Google Analytics IP Anonymization feature must be turned on -> Enabled by default https://developers.google.com/analytics/devguides/migration/measurement/ip-anonymization
  • Projects must ensure that they are not transmitting PII to Google Analytics
  • Projects must accept and follow Google Analytics Terms of Service

Copy link

netlify bot commented Mar 8, 2024

Deploy Preview for thingweb-preview ready!

Name Link
🔨 Latest commit e94d750
🔍 Latest deploy log https://app.netlify.com/sites/thingweb-preview/deploys/662258fa6591d200087747b4
😎 Deploy Preview https://deploy-preview-73--thingweb-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@egekorkan egekorkan added the Discuss in Committer Meeting Labels issues and PRs that should be discussed in our regular Thingweb Committer Meeting. label Apr 19, 2024
@egekorkan egekorkan marked this pull request as ready for review April 19, 2024 11:44
@egekorkan
Copy link
Member Author

It is now approved by the Eclipse Webmaster. Merging

@egekorkan egekorkan merged commit 8969b15 into master Apr 23, 2024
5 checks passed
@egekorkan egekorkan deleted the egekorkan-patch-2 branch April 23, 2024 21:07
@egekorkan
Copy link
Member Author

After deployment, I can see my visit showing up in the analytics page after accepting and no new visitors when not accepting the tracking.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Discuss in Committer Meeting Labels issues and PRs that should be discussed in our regular Thingweb Committer Meeting.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants