Skip to content

feat: 支持添加自定义内容用于follow认证 & 支持封面图 #36#37

Closed
forrany wants to merge 1 commit intohalo-dev:mainfrom
forrany:feat/follow-support
Closed

feat: 支持添加自定义内容用于follow认证 & 支持封面图 #36#37
forrany wants to merge 1 commit intohalo-dev:mainfrom
forrany:feat/follow-support

Conversation

@forrany
Copy link
Copy Markdown

@forrany forrany commented Oct 28, 2024

尝试解决 #36

  1. 添加自定义内容,追加在最后,用于 follow 订阅认证
image
  1. 对于封面图无法识别的问题,follow会自动抓取内容的第一个图片作为封面,因此尝试通过meta抓取封面图
image

@f2c-ci-robot
Copy link
Copy Markdown

f2c-ci-robot Bot commented Oct 28, 2024

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.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@f2c-ci-robot f2c-ci-robot Bot added the do-not-merge/release-note-label-needed Indicates that a PR should not merge because it's missing one of the release note labels. label Oct 28, 2024
@f2c-ci-robot f2c-ci-robot Bot requested review from guqing and wan92hen October 28, 2024 12:06
@f2c-ci-robot
Copy link
Copy Markdown

f2c-ci-robot Bot commented Oct 28, 2024

[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 guqing for approval. For more information see the Kubernetes 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

@Rising-Galaxy
Copy link
Copy Markdown

希望能添加自定义内容,追加在最后,或者是能够更改 字段用于 follow 订阅认证

@guqing
Copy link
Copy Markdown
Member

guqing commented Dec 5, 2024

Hi @Rising-Galaxy , 感谢你的贡献,在 #39 中我们重写了 feed 插件并已经完善了,因此此 PR 不再适合当前的场景,因此我将关闭此 PR

@guqing guqing closed this Dec 5, 2024
@guqing
Copy link
Copy Markdown
Member

guqing commented Dec 5, 2024

希望能添加自定义内容,追加在最后,或者是能够更改 字段用于 follow 订阅认证

在最新的代码中已经支持,等待 1.4.0 版本发布,参考 #39

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

Labels

do-not-merge/release-note-label-needed Indicates that a PR should not merge because it's missing one of the release note labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants