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

[BUG] 使用artalk2.8.2 docker版本,调用出现404,已初步发现问题,求优化解决 #415

Closed
a512471 opened this issue Feb 8, 2024 · 3 comments
Assignees
Labels
bug Something isn't working
Milestone

Comments

@a512471
Copy link

a512471 commented Feb 8, 2024

Describe the bug 描述你遇到的错误

  • 自部署了artalk2.8.2,使用fixlt配置模板启用artalk后页面出现404错误,如下图:

image

  • 查看artalk后台报错如下:

image

  • 然后我试着使用artalk官方推荐的前端引用直接修改hugo的public中页面源码,写法如下:

image

  • 之后刷新页面发现正常了,可以评论:

image

  • 推测应该市fixlt中artalk模板使用的内置前端 CSS、JS 资源导致的,版本比较老

Expected behavior 期待的行为

希望fixlt对artalk支持的模板上能再改一改,artalk官网现在使用的css和js文件直接在后端提供,不需要内置到fixlt中了,这样灵活性更高点
image

Screenshots 屏幕截图

No response

Build Environment 构建环境

No response

Preview Environment 预览环境

No response

Additional Information 补充信息

No response

@a512471 a512471 added the bug Something isn't working label Feb 8, 2024
@JamesFlare1212
Copy link

暂时可以手动在这里改一下

{{- $source := $cdn.artalkCSS | default "lib/artalk/Artalk.css" -}}
{{- dict "Source" $source "Minify" true "Fingerprint" $fingerprint | dict "Scratch" .Scratch "Data" | partial "scratch/style.html" -}}
{{- $source := $cdn.artalkJS | default "lib/artalk/Artalk.js" -}}

@Lruihao
Copy link
Member

Lruihao commented Apr 13, 2024

抱歉,最近业余时间都在写 FixIt 的文档,这个问题我下个 fix 版本改一下。

@Lruihao Lruihao self-assigned this Apr 13, 2024
@Lruihao Lruihao added this to the v0.3.x milestone Apr 13, 2024
@Lruihao Lruihao pinned this issue Apr 17, 2024
@Lruihao Lruihao moved this from To do to In progress in Roadmap of FixIt Apr 23, 2024
@Lruihao
Copy link
Member

Lruihao commented Apr 23, 2024

  1. FixIt 0.3.3 开始默认读取 params.page.comment.artalk.server 的 JS 和 CSS,如果启用公共 CDN 时,则使用 CDN 资源。
  2. FixIt 主题默认使用前端配置项设置 Artalk 界面,如需在 Artalk 后台配置界面,将 params.page.comment.artalk.useBackendConf 设置为 true

@Lruihao Lruihao moved this from In progress to Done in Roadmap of FixIt Apr 23, 2024
@Lruihao Lruihao unpinned this issue Apr 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Archived in project
Development

No branches or pull requests

3 participants