Skip to content

Docs: add Plausible #10086

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

Merged
merged 4 commits into from
Mar 6, 2023
Merged

Docs: add Plausible #10086

merged 4 commits into from
Mar 6, 2023

Conversation

humitos
Copy link
Member

@humitos humitos commented Feb 28, 2023

Use Plausible for analytics in our documentation.

Closes #9603


📚 Documentation previews 📚

Use Plausible for analytics in our documentation.
@humitos humitos requested a review from a team as a code owner February 28, 2023 21:57
@humitos humitos requested a review from benjaoming February 28, 2023 21:57
@humitos
Copy link
Member Author

humitos commented Feb 28, 2023

We have this code for Google Analytics, should we have something similar for Plausible?

if ({{ DO_NOT_TRACK_ENABLED | lower }} && navigator.doNotTrack === '1') {
console.debug('Respecting DNT with respect to analytics...');
} else {

@humitos
Copy link
Member Author

humitos commented Feb 28, 2023

Hrm, from their documentation I understand that it's not "required", tho -- https://plausible.io/docs/integration-guides#dnt-and-offline-functionality

Plausible by default doesn't take DNT (Do Not Track) into consideration as we're a privacy first analytics tool and DNT allows "data analysis techniques that protect the anonymity of the users".

@ericholscher
Copy link
Member

ericholscher commented Feb 28, 2023

Hrm, from their documentation I understand that it's not "required", tho -- https://plausible.io/docs/integration-guides#dnt-and-offline-functionality

Plausible by default doesn't take DNT (Do Not Track) into consideration as we're a privacy first analytics tool and DNT allows "data analysis techniques that protect the anonymity of the users".

Good question for @davidfischer, who I think did this work initially.

But it seems like we can just include it, since they're not storing data.

@davidfischer
Copy link
Contributor

Good question for @davidfischer, who I think did this work initially.

I think it's OK to have Plausible despite DNT although it also might be nice to just disable it for those users. I don't think there's any hard requirement so I'll leave it up to you.

At some point we will have to update the privacy policy and our section on GA in our docs although maybe those can wait until we roll it out more widely.

@benjaoming
Copy link
Contributor

I think it's OK to have Plausible despite DNT although it also might be nice to just disable it for those users. I don't think there's any hard requirement so I'll leave it up to you.

Oh interesting! Yup, they say so themselves here, stating that DNT says so, too.. https://plausible.io/docs/integration-guides#dnt-and-offline-functionality

@humitos humitos requested a review from ericholscher March 1, 2023 09:12
Copy link
Contributor

@benjaoming benjaoming left a comment

Choose a reason for hiding this comment

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

I like the approach, that's exactly what multiproject is for!! But think we need to do the below suggestion...

@humitos
Copy link
Member Author

humitos commented Mar 1, 2023

Docs are failing because some unrelated warnings.

@benjaoming
Copy link
Contributor

We could merge #10090 and then update this PR and check the PR build?

@humitos humitos merged commit 0829855 into main Mar 6, 2023
@humitos humitos deleted the humitos/plausible branch March 6, 2023 10:32
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.

Docs: use Plausible for analytics
4 participants