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

Support commenting using giscus #543

Merged
merged 3 commits into from
Feb 28, 2022
Merged

Support commenting using giscus #543

merged 3 commits into from
Feb 28, 2022

Conversation

Rihoj
Copy link
Contributor

@Rihoj Rihoj commented Feb 28, 2022

Issue

Fixes #536

Description

Adds Giscus support
I do think a guide post should be added, but I wanted to make sure you like how this is done before I start writing out a post. (Unless someone else wants to write it)

Test Evidence

image

Copy link
Member

@hossainemruz hossainemruz left a comment

Choose a reason for hiding this comment

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

Thank you @Rihoj. Looks good to me. Just a little change. Can you please acknowledge the comments?

layouts/partials/giscus.html Outdated Show resolved Hide resolved
layouts/partials/giscus.html Outdated Show resolved Hide resolved
@hossainemruz
Copy link
Member

Thank you @Rihoj . Looks good to me. Would you like to have a PR in guide repo? I can merge this PR.

@hossainemruz hossainemruz changed the title add giscus Support [giscus](https://giscus.app/) comment Feb 28, 2022
@hossainemruz hossainemruz changed the title Support [giscus](https://giscus.app/) comment Support commenting using giscus Feb 28, 2022
@Rihoj
Copy link
Contributor Author

Rihoj commented Feb 28, 2022

@hossainemruz I am currently working on the guide PR. Since the structure was a single page for Analytics and Comments, I am working on detailing the whole page out, so it will be a little longer as I am not always elegant with my words.

@hossainemruz
Copy link
Member

Yes. It will be good idea to separate the analytics and comment post.

@kodiakhq kodiakhq bot merged commit 711d4c7 into hugo-toha:main Feb 28, 2022
Rihoj added a commit to Rihoj/guides that referenced this pull request Feb 28, 2022
@Rihoj Rihoj deleted the giscus branch February 28, 2022 18:05
hossainemruz pushed a commit to Rihoj/guides that referenced this pull request Mar 15, 2022
@roumail
Copy link

roumail commented Mar 11, 2023

I'm unable to see comments on my website following these instructions for giscus. Any ideas what could have gone wrong?

@roumail
Copy link

roumail commented Mar 11, 2023

Switched to utteranc and it still doesn't work...

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

Successfully merging this pull request may close these issues.

Add support for giscuss a utterances alternative that uses GitHub discussion
3 participants