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: Giscus comments based on GH discussions #11132

Closed
wants to merge 2 commits into from
Closed

feat: Giscus comments based on GH discussions #11132

wants to merge 2 commits into from

Conversation

a-hariti
Copy link
Collaborator

@a-hariti a-hariti commented Aug 23, 2024

enable users to login with GH account and leave comments on docs,

which will show up under the repo discussions and on the docs page ofc.

Setup guide: https://giscus.app: basically install the Giscus app and enable Discussions

Currently I have it setup to my personal fork for the preview

TODO:

  • Point to actual repo
  • Adjust to dark mode when dark mode PR is merged
  • is the pathname -> discussion mapping enough given the dev docs/user docs situation?

Copy link

vercel bot commented Aug 23, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
changelog ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 23, 2024 0:04am
develop-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 23, 2024 0:04am
sentry-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 23, 2024 0:04am

Copy link

codecov bot commented Aug 23, 2024

Bundle Report

Changes will increase total bundle size by 58.66kB ⬆️

Bundle name Size Change
sentry-docs-server 8.27MB 27.59kB ⬆️
sentry-docs-edge-server 254.34kB 3 bytes ⬇️
sentry-docs-client 6.28MB 31.07kB ⬆️

Copy link
Member

@Jeffreyhung Jeffreyhung left a comment

Choose a reason for hiding this comment

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

@a-hariti we have a commitment to all of our public facing websites that we do not use any sort of tracking technology to track users and does not drop cookies other than the one we listed here.
Can you provide some insights on if Giscus will track users or drop any cookies?

@a-hariti
Copy link
Collaborator Author

Does that include iframes @Jeffreyhung ?

@Jeffreyhung
Copy link
Member

@a-hariti iframe is a common way that drops 3rd party cookies, but it doesn't necessary do so

@a-hariti
Copy link
Collaborator Author

It does drop these cookies, the question is does the no cookie policy cover iframes or not?

cookies from giscus.app

@a-hariti
Copy link
Collaborator Author

Cancelling this due to cookie policy

@a-hariti a-hariti closed this Aug 28, 2024
@github-actions github-actions bot locked and limited conversation to collaborators Sep 13, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants