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

feat(#973): 发种数量统计 #1517

Merged
merged 2 commits into from
Aug 7, 2023
Merged

feat(#973): 发种数量统计 #1517

merged 2 commits into from
Aug 7, 2023

Conversation

IITII
Copy link
Member

@IITII IITII commented Aug 4, 2023

feat(#973): 发种数量统计

示例

image

@fzlins @Rhilip @pjw20911

需要注意的 commit: cfe072b

已知发种数量有问题的站点 (已修复)

如果遇到下面类型的问题,可以先尝试重新刷新个人数据

  • jpop 强制转换后正常

image

  • azusa 刷新后正常

image

@fzlins
Copy link
Collaborator

fzlins commented Aug 6, 2023

可能也需要把字符串改成数字。或者在累加发种数的时候强制转换。

@IITII
Copy link
Member Author

IITII commented Aug 7, 2023

可能也需要把字符串改成数字。或者在累加发种数的时候强制转换。

emm, 虽然 UserInfo 里面定义是 number, 但是实际上还是有 string 的存在. 应该是站点 config.json 里面定义的 uploads 的问题. 对于这种, 貌似只能在累计的时候加强制转换. @fzlins

至于 jpop, 这样尝试了一下, 并没有生效... 欢迎 PR

image

@Rhilip Rhilip merged commit 0b8a5fa into pt-plugins:dev Aug 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants