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

Added analytics tags to relevant pages. #201

Merged
merged 1 commit into from
Apr 30, 2024

Conversation

MaanavD
Copy link
Contributor

@MaanavD MaanavD commented Apr 29, 2024

This pull request involves changes to multiple HTML files to include Google Tag Manager (gtag.js) in the head section of each page. This will allow for tracking and analytics on these pages.

The key changes are:

  • 404.html: Google Tag Manager script added to the head section.
  • about.html: Google Tag Manager script added to the head section.
  • codeofconduct.html: Google Tag Manager script added to the head section.
  • get-started.html: Google Tag Manager script added to the head section.
  • index.html: Google Tag Manager script added to the head section.

Additionally, the same changes were applied to the following pages: meetups/june2022.html [1] news.html [2] sc-candidates.html [3] and supported-tools.html [4].

@MaanavD MaanavD requested a review from ramkrishna2910 April 29, 2024 22:23
@ramkrishna2910 ramkrishna2910 added this pull request to the merge queue Apr 30, 2024
Merged via the queue into onnx:main with commit 293967c Apr 30, 2024
2 checks passed
@andife
Copy link
Member

andife commented Apr 30, 2024

One question? Where do we see the results?

@MaanavD
Copy link
Contributor Author

MaanavD commented Apr 30, 2024

One question? Where do we see the results?

Hey @andife , please reach out to @ramkrishna2910 for the google analytics account username and password. I'm still working on setting it all up, but information should start to slowly propagate there :)

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

Successfully merging this pull request may close these issues.

3 participants