Closed longjuan closed 1 year ago
/feed/categories/{slug}.xml
/feed/authors/{metadata-name}.xml
都是用的非展示名称,需要给用户提示。
如果没对应的分类或用户,就会返回404。存在分类或用户但没文章会正常返回,只是没item。
item.title和item.description均做了转义,与halo1.x一致。
lastBuildDate需要更新halo的依赖版本,我这边暂时写了TODO。
另外https://github.com/halo-dev/halo/pull/3168 我没看懂怎么优化的,不知道我这个写法会不会也有性能问题。
我这边测试没什么问题了,不知道需不需要增加一点单元测试类。 /unhold /ping @halo-sigs/halo
经过测试可以符合预期。
/cc @halo-sigs/halo
顺便希望https://github.com/halo-dev/halo/issues/2730 能发布,这样升级halo依赖比较方便,就不用单独放个jar包了。
// TODO lastBuildDate need upgrade halo dependency version
顺便希望https://github.com/halo-dev/halo/issues/2730 能发布
这个功能我们还在规划中
原来有作者的单独的链接啊,主题路由里没看到以为没有,我把按作者的链接也替换成作者单独的链接吧。 又是需要更新依赖版本的,我加TODO吧。
可以先发一个测试版本?
可以,你们搞吧,README什么的
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: JohnNiang, ruibaby
The full list of commands accepted by this bot can be found here.
The pull request process is described here
2.1.0-加入user permalink 2.2.0-加入lastBuildDate
现在就提PR吗,还是过几天没人反馈问题再提?
2.1.0-加入user permalink 2.2.0-加入lastBuildDate
现在就提PR吗,还是过几天没人反馈问题再提?
可以再等等反馈吧。
Fixes https://github.com/halo-sigs/plugin-feed/issues/1 所有: 按作者: 按分类:
/hold 先让我多测试会