Skip to content

Removed Google Analytics from the Druid Website#17661

Merged
gianm merged 1 commit intoapache:masterfrom
akulabs8:17647
Apr 1, 2025
Merged

Removed Google Analytics from the Druid Website#17661
gianm merged 1 commit intoapache:masterfrom
akulabs8:17647

Conversation

@akulabs8
Copy link
Contributor

Fixes #17647.

Description

To make Druid Website complaint with ASF Privacy Policy: Removed the details of google analytics and gtag from druid-website-src (Link to PR)
Thus, removed the gtag from here as well.

Removed the details of google analytics and gtag from druid-website-src (Link to PR)
Thus, removed the gtag from here as well.


This PR has:

  • been self-reviewed.
  • added documentation for new or modified features or behaviors.
  • a release note entry in the PR description.
  • added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
  • added or updated version, license, or notice information in licenses.yaml
  • added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
  • added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
  • added integration tests.
  • been tested in a test Druid cluster.

@akulabs8 akulabs8 marked this pull request as draft January 24, 2025 20:11
@akulabs8 akulabs8 marked this pull request as ready for review January 24, 2025 20:11
@techdocsmith techdocsmith requested a review from 317brian January 25, 2025 00:01
@github-actions
Copy link

This pull request has been marked as stale due to 60 days of inactivity.
It will be closed in 4 weeks if no further activity occurs. If you think
that's incorrect or this pull request should instead be reviewed, please simply
write any comment. Even if closed, you can still revive the PR at any time or
discuss it on the dev@druid.apache.org list.
Thank you for your contributions.

@github-actions github-actions bot added the stale label Mar 26, 2025
Copy link
Contributor

@gianm gianm left a comment

Choose a reason for hiding this comment

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

LGTM. Thank you!

@gianm gianm merged commit fb2f7ac into apache:master Apr 1, 2025
1 check passed
@kgyrtkirk kgyrtkirk added this to the 33.0.0 milestone Apr 14, 2025
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.

Remove Google Analytics from the Druid Website

3 participants