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 Cloudflare Web Analytics #15

Merged
merged 3 commits into from
May 10, 2024
Merged

Conversation

grantwforsythe
Copy link
Owner

@grantwforsythe grantwforsythe commented May 10, 2024

Pull Request Template

Title

Add Cloudflare Web Analytics

Type of Change

  • Feature
  • Fix
  • Refactor
  • UI/UX improvement

Description

Create a service that injects the Cloudflare Web Analytics script into the DOM on init of app-root when in the production environment.

Testing

The logic is simple so there is nothing worth testing.

Impact

Add Cloudflare Web Analytics to the application providing insight on the usage of the application.

Create a service to inject the Cloudflare Web Analytics script into the
DOM as it is only need in production.
Update privacy policy with the addition of Cloudflare Web Analytics and
in preparation for asking my OAuth client ID to be removed from
restricted mode.
Copy link

render bot commented May 10, 2024

@grantwforsythe grantwforsythe marked this pull request as ready for review May 10, 2024 19:56
@grantwforsythe grantwforsythe merged commit 08cecde into main May 10, 2024
7 checks passed
@grantwforsythe grantwforsythe deleted the feat/add-cloudflare-analytics branch May 10, 2024 19:56
@grantwforsythe grantwforsythe added the enhancement New feature or request label May 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant