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

show episode num #582

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

show episode num #582

wants to merge 2 commits into from

Conversation

xxfttkx
Copy link
Contributor

@xxfttkx xxfttkx commented Jan 10, 2025

类似#510,显示各个番剧源的集数
image
querychapterRoads请求似乎会增加很多,这pr能被接受吗

@Predidit
Copy link
Owner

非常抱歉,但是这种实现在我看来是不可接受的

@xxfttkx
Copy link
Contributor Author

xxfttkx commented Jan 10, 2025

@Predidit
是因为querychapterRoads消耗太大了吗。预览番剧源的集数对于我是很常用的功能,项目有考虑添加吗。

@Predidit
Copy link
Owner

是的,这个实现的消耗太大了

@Predidit
Copy link
Owner

不是我们的消耗,这里指对源站的消耗

Girigiri 的站长之前已经开过 Issue 来抗议相关问题

@xxfttkx
Copy link
Contributor Author

xxfttkx commented Jan 10, 2025

对源站的消耗应该更集中在视频的观看吧

@Predidit
Copy link
Owner

触发 rate_limit 被拉黑就不好玩了,那样的用户体验很差的。

@xxfttkx
Copy link
Contributor Author

xxfttkx commented Jan 10, 2025

目前的写法是点击具体源时才去才去进行相关查询。害怕rate_limit或许可以在每次查询时延时1秒甚至更长?或者就只查询资源的前几个?不过这样用户体验或许也不好。

@Predidit
Copy link
Owner

可以先标记为草稿

我在考虑实现一个像安卓开发者选项那样的隐藏菜单

在隐藏菜单中提供这些功能的开关

@xxfttkx xxfttkx marked this pull request as draft January 11, 2025 00:13
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.

2 participants