Giscus fails when repository contains capitals
See original GitHub issueDiscovered this when using Giscus with Jekyll as part of https://github.com/mmistakes/minimal-mistakes/pull/3022
The PR takes the repository name from a setting, which in my case was SeanKilleen/seankilleen.github.io
.
Giscus failed with a 400 error when retrieving comments, so thought there was no discussion topic, and therefore kept creating new discussion topics.
I think the issue is that the GitHub API or similar might be failing with a repo name that contains uppercase characters. I’ll see if I can submit a PR to make that behavior case insensitive.
Issue Analytics
- State:
- Created 2 years ago
- Reactions:1
- Comments:9 (4 by maintainers)
Top Results From Across the Web
Build from GitHub fails when repo name includes capitalized ...
Create a GitHub repository that includes capital letters in the name. Create a new Cloud Run service; Select "Continuously deploy new ...
Read more >Add support for private repositories · Issue #292 - GitHub
Currently new comment creates new discussion, but discussion search is not working and no comments are displayed. This is a failed request ......
Read more >Replacing Disqus with GitHub discussions using giscus
Configuring giscus · Enter the repository you configured earlier · Choose Discussion title contains page pathname. · Check Use strict title ...
Read more >How to use GitHub issues for blog post comments (with ...
List issue comments. From the issue endpoint GET /repos/{owner}/{repo}/issues/{issue_number} we want to have the information if the issue has ...
Read more >
Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free
Top Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
@laymonage confirmed – #124 resolved this issue. 👍 Thanks!
@LazyRen It turns out that the issue is unrelated, I’ve shipped https://github.com/laymonage/giscus/commit/bca513c72a5a3b309cce4a864b042a2e82c66c48 which should fix it.
@SeanKilleen I cannot reproduce the error, the following works fine:
Could you copy the
<iframe>
's src when it happens and paste it here?