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

Add Giscus comments system #267

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from
Draft

Add Giscus comments system #267

wants to merge 5 commits into from

Conversation

fflaten
Copy link
Collaborator

@fflaten fflaten commented May 3, 2023

Implements Giscus as comment system for all Docs and Commands-pages. Only index-page is currently excluded.
Comments are stored and managed as discussions in specific category in the docs-repo.

Fix #251

TODO:

  • Install Giscus app in docs-repo
  • Enable discussion in docs-repo. Remove default categories and create a "Comments" or "Website Comments"-category of Announcement-type
  • Update config
  • Reswizzle after Docusaurus v3 upgrade Docusaurus v3 #272

@bravo-kernel
Copy link
Collaborator

Looks amazing. Perhaps comments could also be excluded for the Additional Resources section but to be honest, I think that whole section should either be dropped or totally reworked anyway. It's just a lot of very (years old) content for which a google search provides better results.

@fflaten
Copy link
Collaborator Author

fflaten commented May 4, 2023

Great minds. See #264 😁
Until then, comments to help crowdsource what works or not is an improvement.

@fflaten fflaten changed the title Add Giscus comments to docs pages Add Giscus comments system May 5, 2023
@fflaten
Copy link
Collaborator Author

fflaten commented May 5, 2023

A few sample pages:

What do you think @nohwnd?

@bravo-kernel
Copy link
Collaborator

LGTM shall we just merge this?

@fflaten
Copy link
Collaborator Author

fflaten commented May 16, 2023

Can't, see task list in PR details. Using my fork's discussion atm. So waiting on approval and actions from @nohwnd 🙂

@bravo-kernel
Copy link
Collaborator

Ahhh 🤦🏻

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.

Giscus comments
2 participants