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.

Sign up for GitHub

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

Jump to bottom

文章详情页评论使用gitment和gitalk都会报错 #139

Closed
enjoyWeb opened this issue Jul 12, 2022 · 6 comments · Fixed by #195
Closed

文章详情页评论使用gitment和gitalk都会报错 #139

enjoyWeb opened this issue Jul 12, 2022 · 6 comments · Fixed by #195
Assignees
@Candinya
Labels
enhancement New feature or request question Further information is requested

Comments

@enjoyWeb
Copy link

yml配置如下所示
image
应用和密钥在github上均已创建
image
（1）使用gitment报错如下
image
（2）使用gitalk报错如下
image
@ArcticLampyrid
Copy link
Collaborator

这些评论系统似乎需要您手动创建 issue

@enjoyWeb
Copy link
Author

这些评论系统似乎需要您手动创建 issue

我之前好像是可以评论成功，但是后面用正式域名就不行了（应用里的callback url地址也改成正是域名的了）
image

@ArcticLampyrid
Copy link
Collaborator

您好，这一主题只是简单的将相关参数传输给了Gitalk v1的脚本，相关使用说明和答疑还请前往 https://github.com/gitalk/gitalk

@ArcticLampyrid ArcticLampyrid added the question Further information is requested label Jul 16, 2022
@Lotuses-robot
Copy link
Contributor

Gitalk 出现 Validation Failed. 是因为文章链接长度过长 （>50 4bit/1char）

在 Gitalk repo issue 中有提到

@Lotuses-robot
Copy link
Contributor

有一个比较简单的方法解决：传入 gitalk 的参数为源地址的 md5

@ArcticLampyrid
Copy link
Collaborator

传入 gitalk 的参数为源地址的 md5

Note: This will be a breaking change.

@Candinya Candinya added the enhancement New feature or request label Oct 26, 2022
@ArcticLampyrid ArcticLampyrid mentioned this issue Jan 4, 2024
Closed
3 tasks
jht3QAQ added a commit to jht3QAQ/Kratos-Rebirth that referenced this issue Jan 5, 2024
@jht3QAQ
fix: gitalk id too long to create create issue
db9ccb9 
allow gitalk to hash id

fix Candinya#139
@jht3QAQ jht3QAQ mentioned this issue Jan 5, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Candinya Candinya

Labels
enhancement New feature or request question Further information is requested
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: gitalk id too long to create create issue jht3QAQ/Kratos-Rebirth
4 participants
@enjoyWeb @ArcticLampyrid @Candinya @Lotuses-robot