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

【nav 主题】将slug设置为外链报错 #2521

Open
panghu960 opened this issue Jun 21, 2024 · 11 comments · Fixed by #2530
Open

【nav 主题】将slug设置为外链报错 #2521

panghu960 opened this issue Jun 21, 2024 · 11 comments · Fixed by #2530
Assignees
Labels
bug Something isn't working good first issue Good for newcomers help wanted Extra attention is needed resolved

Comments

@panghu960
Copy link

描述bug
nav 主题 将slug设置为外链报错
image

复现步骤
【此项必填】你的操作步骤,按此步骤理应在我的开发环境出现一样的bug。

环境

最新分支

补充说明
上一个分支不报错

@panghu960 panghu960 added the bug Something isn't working label Jun 21, 2024
@panghu960
Copy link
Author

@panghu960
Copy link
Author

image

@W4ddl3
Copy link

W4ddl3 commented Jun 21, 2024

对,今天我也发现了这个问题
Shot 2024-06-21 at 17 26 38@2x

Shot 2024-06-21 at 17 29 26@2x

@sakuraee
Copy link

按照文档,slug不是说作为路径的一个别名么,为什么要填写成一个外链呢,我在这个文件中处理url的地方修改了下,编译可以成功编译 ,但是那篇文章的展示会有问题

@W4ddl3
Copy link

W4ddl3 commented Jun 24, 2024

按照文档,slug不是说作为路径的一个别名么,为什么要填写成一个外链呢,我在这个文件中处理url的地方修改了下,编译可以成功编译 ,但是那篇文章的展示会有问题

你说的对,出现这个报错后,我就去检查了 作者的部署文档 与 仓库 ,发现根本找不到 slug 可以设置为 外链 的说法。
(但是你可以从 closed issue 中看到相关功能的问题)

作为证据,这是一个非常完美的网站:https://ct.ued.cat
你可以看到,他的文章都是跳转到外链的。那你觉得怎么实现呢?正是 slug 填外链实现的。

但是现在文档没了外链这个功能介绍,只说是作为文章的 文章短路径 ,所以 @tangly1024 是决定去掉这个功能了吗?

@sakuraee
Copy link

是哦,我直接退回到二月份4.24版本 发现是没有任何问题的

@sakuraee
Copy link

@tangly1024 你在前几天的一笔提交上引入了这个问题,你看看需要把这部分改回去么,已经提了PR了

@panghu960
Copy link
Author

按照文档,slug不是说作为路径的一个别名么,为什么要填写成一个外链呢,我在这个文件中处理url的地方修改了下,编译可以成功编译 ,但是那篇文章的展示会有问题

你说的对,出现这个报错后,我就去检查了 作者的部署文档 与 仓库 ,发现根本找不到 slug 可以设置为 外链 的说法。 (但是你可以从 closed issue 中看到相关功能的问题)

作为证据,这是一个非常完美的网站:https://ct.ued.cat 你可以看到,他的文章都是跳转到外链的。那你觉得怎么实现呢?正是 slug 填外链实现的。

但是现在文档没了外链这个功能介绍,只说是作为文章的 文章短路径 ,所以 @tangly1024 是决定去掉这个功能了吗?

请问一下,获取图标的方案你是如何解决的,手动传上去吗

@tangly1024 tangly1024 linked a pull request Jun 26, 2024 that will close this issue
@tangly1024
Copy link
Owner

保留外链功能,并在此次提交中修复bug: #2530

@tangly1024 tangly1024 added help wanted Extra attention is needed good first issue Good for newcomers labels Jun 26, 2024
@panghu960
Copy link
Author

修复后,无法正常获取文章内容

@tangly1024
Copy link
Owner

感谢反馈,已提交新的 修复补丁 ;3220bf4

@tangly1024 tangly1024 changed the title nav 主题 将slug设置为外链报错 【nav 主题】将slug设置为外链报错 Jun 28, 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 good first issue Good for newcomers help wanted Extra attention is needed resolved
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants