Skip to content

提交halo-theme-asorn主题#184

Open
XiaoWoUi wants to merge 1 commit intohalo-sigs:mainfrom
XiaoWoUi:main
Open

提交halo-theme-asorn主题#184
XiaoWoUi wants to merge 1 commit intohalo-sigs:mainfrom
XiaoWoUi:main

Conversation

@XiaoWoUi
Copy link

仓库地址

https://github.com/XiaoWoUi/halo-theme-asorn

应用市场

3458721468

Updated the pull request template to include a repository link and modified the release note section with additional content.
@f2c-ci-robot f2c-ci-robot bot added the release-note Denotes a PR that will be considered when it comes time to generate release notes. label Nov 30, 2025
@f2c-ci-robot
Copy link

f2c-ci-robot bot commented Nov 30, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign ruibaby for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ruibaby
Copy link
Member

ruibaby commented Dec 1, 2025

感谢提交,但在合并之前期望优化以下问题:

  1. 导航菜单直接使用 menuFinder.getPrimary 即可,没有必要在主题设置中去设置。
  2. 不是很清楚使用主题设置去添加文章链接的需求,建议直接在模板中用 postFinder 去查询文章。https://github.com/XiaoWoUi/halo-theme-asorn/blob/8ecd22c594bd0b0ec0c8622ff7198d617ee20691/settings.yaml#L340-L343
  3. 文章列表模板没有必要去新建页面,使用自定义页面实现,可以直接实现 archives.html 模板。

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

Labels

release-note Denotes a PR that will be considered when it comes time to generate release notes.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants