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

feat: add anonymized analytics to docs #6573

Merged
merged 1 commit into from
Mar 21, 2024

Conversation

jeluard
Copy link
Member

@jeluard jeluard commented Mar 21, 2024

Description

Add anonymized analytics to the docusaurus based docs website.

Fixes #6549

@jeluard jeluard requested a review from a team as a code owner March 21, 2024 08:51
@philknows
Copy link
Member

Already discussed this with Julien as a way to get better metrics on our docs for improved dev ex. Understanding users and what they are consuming and how they are consuming our pages will help us create a better experience for them. Not intended for anything more than that and this can be revoked at anytime if there are concerns.

Copy link
Member

@nflaig nflaig 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 to me but for transparency we should probably highlight this in our next release.

@jeluard jeluard added the scope-documentation All issues related to the Lodestar documentation. label Mar 21, 2024
Copy link

codecov bot commented Mar 21, 2024

Codecov Report

Merging #6573 (b552ab1) into unstable (da69a7b) will decrease coverage by 0.01%.
Report is 1 commits behind head on unstable.
The diff coverage is n/a.

Additional details and impacted files
@@             Coverage Diff              @@
##           unstable    #6573      +/-   ##
============================================
- Coverage     61.48%   61.48%   -0.01%     
============================================
  Files           556      556              
  Lines         58891    58891              
  Branches       1854     1853       -1     
============================================
- Hits          36212    36211       -1     
- Misses        22638    22639       +1     
  Partials         41       41              

@jeluard jeluard merged commit 9b1f0af into ChainSafe:unstable Mar 21, 2024
19 of 21 checks passed
@jeluard jeluard deleted the jeluard/docusaurus-analytics branch March 21, 2024 16:45
@wemeetagain
Copy link
Member

🎉 This PR is included in v1.18.0 🎉

@philknows
Copy link
Member

Sidenote: Google Analytics has been removed via #6680 before publishing the new docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
scope-documentation All issues related to the Lodestar documentation.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add analytics support to documentation website
4 participants