-
Notifications
You must be signed in to change notification settings - Fork 244
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
添加作者文章列表 #995
添加作者文章列表 #995
Conversation
谢谢!从效果上看似乎只有联合署名文章会被列上去,另外我估计同样会存在 #950 的重名问题,可能要加些条件判断。 |
已修复。重名问题暂时没头绪。 |
先记一下思路:
如果有人愿意背这口锅,可以冒个泡。 |
我这样按照 id 筛选文章:
两位赵鹏的列表是对的,但其他人就有一长串文章了,不知道怎么解决。 @tcgriffith 能不能帮忙改改这个 pull request,解决下重名问题? |
有空我看看! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
谢谢!
Co-authored-by: Yihui Xie <[email protected]>
@tcgriffith TC兄有进展否?最近清理历史遗留问题 #663,感觉这个PR如果可以应该可以顺道解决 |
@yufree 预计一周后(8.6以后)我可以!先记入我的todo |
… the article list by author_id
一年半以后终于撸起袖子解决这个pr了… 年代久远 rebase又不大熟悉干脆重新按最新分支另写起草了一个PR,请移步查看预览效果和讨论 #1097 |
感谢 @tcgriffith! 同时感谢 @CyrusYip 的原型 #995 * Follow-up on #995, authors with duplicated names will generate the article list by author_id * tweak the logic for edge cases: when author_id doesn't exist, use name instead --------- Co-authored-by: tc <[email protected]>
感谢 @CyrusYip !最终解决方案在 #1097,感谢 @tcgriffith ! |
效果图: