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

支持生成 RSS 订阅 #264

Merged
merged 9 commits into from
Feb 5, 2025
Merged

支持生成 RSS 订阅 #264

merged 9 commits into from
Feb 5, 2025

Conversation

Jinvic
Copy link
Contributor

@Jinvic Jinvic commented Jan 27, 2025

这个 PR 做了什么?

当自定义RSS为空时,请求/rss接口生成rss内容。
image
image

  • markdown格式的原文将解析为html
  • 其余如视频音乐豆瓣等内容将以超链接形式展示
  • 音乐只支持网易云和QQ音乐

示例:
image
image

@dinphy
Copy link
Contributor

dinphy commented Jan 30, 2025

RSS功能,被MAIL功能的部分代码覆盖了。

@Jinvic
Copy link
Contributor Author

Jinvic commented Jan 30, 2025

RSS功能,被MAIL功能的部分代码覆盖了。

因为dev分支上有rss的pr,所以暂时没有将邮件功能merge到dev分支。

你可以拉取到本地后手动将feat/mail分支merge到dev,然后使用dev分支。

@kingwrcy kingwrcy merged commit 23c32fc into kingwrcy:dev Feb 5, 2025
@dianso
Copy link
Contributor

dianso commented Feb 10, 2025

RSS功能,被MAIL功能的部分代码覆盖了。

因为dev分支上有rss的pr,所以暂时没有将邮件功能merge到dev分支。

你可以拉取到本地后手动将feat/mail分支merge到dev,然后使用dev分支。

这个默认是展示最近的8条memos么?我看再往前的没有

@Jinvic
Copy link
Contributor Author

Jinvic commented Feb 10, 2025

这个默认是展示最近的8条memos么?我看再往前的没有

默认展示最近的10条未置顶的memo,考虑到rss的主要用于订阅追更,所以没有展示太多。

未置顶这个条件好像是cv时忘了删(挠头)

我再提个bugfix的pr吧。话说置顶的memo在rss里还需要置顶吗?

@dianso
Copy link
Contributor

dianso commented Feb 10, 2025

这个默认是展示最近的8条memos么?我看再往前的没有

默认展示最近的10条未置顶的memo,考虑到rss的主要用于订阅追更,所以没有展示太多。

未置顶这个条件好像是cv时忘了删(挠头)

我再提个bugfix的pr吧。话说置顶的memo在rss里还需要置顶吗?

我感觉没必要在rss里面也置顶,要不就加一个选项

@xuewenG xuewenG added the feature New feature label Feb 14, 2025
@xuewenG xuewenG changed the title feature: 实现RSS功能 支持生成RSS订阅 Feb 14, 2025
@xuewenG xuewenG added this to the v0.2.9 milestone Feb 14, 2025
@xuewenG
Copy link
Collaborator

xuewenG commented Feb 14, 2025

close #8

@xuewenG xuewenG changed the title 支持生成RSS订阅 支持生成 RSS 订阅 Feb 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants