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

3716 add data usage statement #3724

Merged
merged 11 commits into from
Jun 7, 2024

Conversation

aranas
Copy link
Collaborator

@aranas aranas commented Jun 6, 2024

Summary

Adding a User Awareness Statement about our use of Google Analytics to the Afterword

Fixes #3716

List of changes proposed in this PR (pull-request)

  • added a new markdown file with the statement
  • updated toc

What should a reviewer concentrate their feedback on?

  • Is there any information missing from the statement?
  • Are there better links to indicate in what ways Google Analytics might be problematic?

Acknowledging contributors

Copy link

welcome bot commented Jun 6, 2024

Thank You Banner
💖 Thanks for opening this pull request! 💖 The Turing Way community really appreciates your time and effort to contribute to the project. Please make sure you have read our Contributing Guidelines and filled in our pull request template to the best of your ability.
If you are submitting a new chapter, here are some things that will help get your pull request across the finish line! 🏁

  • Check you have removed all lorem ipsums from the chapter template (if you used it)
  • Check for any abbreviations or latin phrases (such as "e.g." or "i.e.") in your writing. See our style guide for more information on this topic.
  • Make sure you have added your new chapter to the Table of Contents

    We have Continuous Integration tests that check the writing style and will help you track down any slip-ups ♻ The Netlify bot will also comment with a preview of the book with your additions so you can see how it will look once it's merged! 🎉

    We get a lot of pull requests on this repo, so please be patient and we will get back to you as soon as we can. If we don't acknowledge this pull request after 7 days, feel free to chat to us about it in our Slack workspace.

@aranas aranas requested a review from malvikasharan June 6, 2024 12:41
Copy link

netlify bot commented Jun 6, 2024

Deploy Preview for the-turing-way ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 617e999
🔍 Latest deploy log https://app.netlify.com/sites/the-turing-way/deploys/6662bd99e1db670008812658
😎 Deploy Preview https://deploy-preview-3724--the-turing-way.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.

Copy link
Collaborator

@RichardJActon RichardJActon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good in the preview, glad to hear the plan is to ditch google analytics, the sooner the better.

Copy link
Collaborator

@acocac acocac left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@aranas - this is a great content that hopefully will lead us to a more GDPR-compliant service to track the web analytics of TTW book. I've made some minor suggestions.

book/website/afterword/data-usage-statement.md Outdated Show resolved Hide resolved

We want to collect data on the reach and findability of The Turing Way book.
For example, data on website traffic helps with reporting impact.
Data on search terms people use to find the website can help us understand readers' needs and preferences.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Data on search terms people use to find the website can help us understand readers' needs and preferences.
Data on search terms people use to find the website can help us understand readers' needs and preferences.

book/website/afterword/data-usage-statement.md Outdated Show resolved Hide resolved
book/website/afterword/data-usage-statement.md Outdated Show resolved Hide resolved
@aleesteele
Copy link
Member

Thank you for making this important contribution @aranas! Just wanted to flag that we also use the following website for our analytics on our github repository: https://cauldron.io/. I don't know if this is something we want to add here as well? :)

aranas and others added 2 commits June 7, 2024 08:16
Co-authored-by: Anne Lee Steele <aleesteele@gmail.com>
Co-authored-by: Alejandro © <acocac@gmail.com>
@aranas
Copy link
Collaborator Author

aranas commented Jun 7, 2024

Thank you for making this important contribution @aranas! Just wanted to flag that we also use the following website for our analytics on our github repository: https://cauldron.io/. I don't know if this is something we want to add here as well? :)

Yes, I think it would be nice for transparency's sake. I have added this!

book/website/afterword/data-usage-statement.md Outdated Show resolved Hide resolved
book/website/afterword/data-usage-statement.md Outdated Show resolved Hide resolved
@malvikasharan
Copy link
Collaborator

Thank you Sophie. This is such an important addition. ❤️

Co-authored-by: Malvika Sharan <malvikasharan@gmail.com>
Co-authored-by: Malvika Sharan <malvikasharan@gmail.com>
@aranas aranas merged commit 664f644 into the-turing-way:main Jun 7, 2024
15 checks passed
Copy link

welcome bot commented Jun 7, 2024

Congratulations Banner
Congrats on merging your first pull request! 🎉 We here at The Turing Way are proud of you! 💖 Thank you so much for your contribution 🎁

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.

[FOR REVIEW] Web Data Usage
6 participants