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

修改帮助链接 #2461

Open
wants to merge 21 commits into
base: main
Choose a base branch
from
Open

修改帮助链接 #2461

wants to merge 21 commits into from

Conversation

zkitefly
Copy link
Member

@zkitefly zkitefly commented Aug 9, 2023

需要合并 HMCL-dev/HMCL-docs#57https://docs.hmcl.net/help 能正常访问到页面后才能合并该 PR

https://docs.hmcl.net/help 无法访问,故将还原至带 .html 的链接

- 将通过按钮打开的 https://docs.hmcl.net/help.html 链接修改为 https://docs.hmcl.net/help

  • 将展示给用户的需要手动在浏览器访问的 https://docs.hmcl.net/help.html 修改为 docs.hmcl.net/help
  • 将展示给用户的需要手动在浏览器访问的 https://docs.hmcl.net/help.html 修改为 docs.hmcl.net/help.html

@zkitefly zkitefly changed the title editlink 修改帮助链接 Aug 9, 2023
@burningtnt
Copy link
Member

冲突了

@burningtnt
Copy link
Member

冲突了

@Glavo Glavo changed the base branch from javafx to main February 7, 2024 10:42
@Glavo
Copy link
Member

Glavo commented Feb 7, 2024

怀疑删除 https:// 前缀的必要性。https:// 前缀可以明确提示用户这是链接,隐藏该前缀不见得有什么正面价值。

@zkitefly
Copy link
Member Author

zkitefly commented Feb 7, 2024

怀疑删除 https:// 前缀的必要性。https:// 前缀可以明确提示用户这是链接,隐藏该前缀不见得有什么正面价值。

改回去了(

@zkitefly
Copy link
Member Author

zkitefly commented Feb 7, 2024

我再加个东西稍等

可以了

@zkitefly
Copy link
Member Author

ping @Glavo

@3gf8jv4dv
Copy link
Contributor

Can I incorporate the changes from this PR into #3287? I will put co-author in the commit message.

3gf8jv4dv added a commit to 3gf8jv4dv/HMCL that referenced this pull request Sep 22, 2024
Revision from HMCL-dev#2461 and revised again based on it.

Co-authored-by: Zkitefly <2573874409@qq.com>
@3gf8jv4dv
Copy link
Contributor

d5086da (#3287)

Check it out!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants