halo-sigs / plugin-links

Halo 2.0 的链接管理插件
https://www.halo.run/store/apps/app-hfbQg
GNU General Public License v3.0
30 stars 7 forks source link

添加分组描述,分组方式,友链卡片,友链标签,友链标签颜色 #29

Closed chengzhongxue closed 1 year ago

chengzhongxue commented 1 year ago

image image image image

f2c-ci-robot[bot] commented 1 year ago

@chengzhongxue: Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it.

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository.
f2c-ci-robot[bot] commented 1 year ago

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: To complete the pull request process, please assign ruibaby after the PR has been reviewed. You can assign the PR to them by writing /assign @ruibaby in a comment when ready.

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

Needs approval from an approver in each of these files: - **[OWNERS](https://github.com/halo-sigs/plugin-links/blob/main/OWNERS)** Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
ruibaby commented 1 year ago

Hi @chengzhongxue ,感谢你的贡献,但进行这个改动之前,我们建议最好先提交一个 issue 来讨论一下,可以参考:https://docs.halo.run/contribution/pr

此外,我认为添加的这些字段似乎是为了适配某个主题,感觉并不是太通用,所以我建议使用元数据来完成,可以参考:https://docs.halo.run/developer-guide/annotations-form

这样的话,主题就可以根据需要来定义所需要的字段,但目前在插件可能没有办法方便的使用在 Console 中提供的 AnnotationsForm 组件,如果有需要,我们会在 Console 自动全局注册此组件。

chengzhongxue commented 1 year ago

那可以添加Console 自动全局注册此组件? 

c @.***

 

------------------ 原始邮件 ------------------ 发件人: "Ryan @.>; 发送时间: 2023年7月3日(星期一) 中午11:22 收件人: @.>; 抄送: @.>; @.>; 主题: Re: [halo-sigs/plugin-links] 添加分组描述,分组方式,友链卡片,友链标签,友链标签颜色 (PR #29)

Hi @chengzhongxue ,感谢你的贡献,但进行这个改动之前,我们建议最好先提交一个 issue 来讨论一下,可以参考:https://docs.halo.run/contribution/pr

此外,我认为添加的这些字段似乎是为了适配某个主题,感觉并不是太通用,所以我建议使用元数据来完成,可以参考:https://docs.halo.run/developer-guide/annotations-form

这样的话,主题就可以根据需要来定义所需要的字段,但目前在插件可能没有办法方便的使用在 Console 中提供了 AnnotationsForm 组件,如果有需要,我们会在 Console 自动全局注册此组件。

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were mentioned.Message ID: @.***>

ruibaby commented 1 year ago

那可以添加Console 自动全局注册此组件?

嗯,但这可能得放在下一个 Halo 版本了,所以我建议这个 PR 可以先放放。

chengzhongxue commented 1 year ago

好的 

c @.***

 

------------------ 原始邮件 ------------------ 发件人: "Ryan @.>; 发送时间: 2023年7月3日(星期一) 中午11:32 收件人: @.>; 抄送: @.>; @.>; 主题: Re: [halo-sigs/plugin-links] 添加分组描述,分组方式,友链卡片,友链标签,友链标签颜色 (PR #29)

那可以添加Console 自动全局注册此组件?

嗯,但这可能得放在下一个 Halo 版本了,所以我建议这个 PR 可以先放放。

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were mentioned.Message ID: @.***>

ruibaby commented 1 year ago

Hi @chengzhongxue ,感谢你的贡献,但进行这个改动之前,我们建议最好先提交一个 issue 来讨论一下,可以参考:docs.halo.run/contribution/pr

此外,我认为添加的这些字段似乎是为了适配某个主题,感觉并不是太通用,所以我建议使用元数据来完成,可以参考:docs.halo.run/developer-guide/annotations-form

这样的话,主题就可以根据需要来定义所需要的字段,但目前在插件可能没有办法方便的使用在 Console 中提供的 AnnotationsForm 组件,如果有需要,我们会在 Console 自动全局注册此组件。

已经在 https://github.com/halo-sigs/plugin-links/pull/36 中支持。

/close

f2c-ci-robot[bot] commented 1 year ago

@ruibaby: Closed this PR.

In response to [this](https://github.com/halo-sigs/plugin-links/pull/29#issuecomment-1660003845): >> Hi @chengzhongxue ,感谢你的贡献,但进行这个改动之前,我们建议最好先提交一个 issue 来讨论一下,可以参考:[docs.halo.run/contribution/pr](https://docs.halo.run/contribution/pr) >> >> 此外,我认为添加的这些字段似乎是为了适配某个主题,感觉并不是太通用,所以我建议使用元数据来完成,可以参考:[docs.halo.run/developer-guide/annotations-form](https://docs.halo.run/developer-guide/annotations-form) >> >> 这样的话,主题就可以根据需要来定义所需要的字段,但目前在插件可能没有办法方便的使用在 Console 中提供的 AnnotationsForm 组件,如果有需要,我们会在 Console 自动全局注册此组件。 > >已经在 https://github.com/halo-sigs/plugin-links/pull/36 中支持。 > >/close Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository.