Skip to content

Conversation

@codycjy
Copy link

@codycjy codycjy commented Oct 24, 2023

仿照QB下载器的形式利用transmission-rpc 完成了基本的功能,其中QBDownloader的部分功能并未使用,在tr中也就没有实现。
可能未来做一个BaseDownloader来管理更多的下载器会更好?如果有需要我可以提一个RFC并在后续版本完成他。

@vercel
Copy link

vercel bot commented Oct 24, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
auto-bangumi ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 24, 2023 6:20am

@EstrellaXD
Copy link
Owner

EstrellaXD commented Oct 24, 2023

感谢贡献!不过目前 3.2 规划上会迁移到 httpx 协程运行。 #561 因此后端的下载器的同步库也进行了一部分重构,详情可以参考 #567 中 QB 下载器的写法。

具体 BaseDownloader 是一个怎么样的构思呢,目前来说后端有一个统一的下载器 API 入口,可以简单说说设计么。

@codycjy
Copy link
Author

codycjy commented Oct 24, 2023

transmission是rpc实现的,我认为将函数修改为异步即可,应该不需要用httpx实现?
关于tr的更新应当直接从aio-refactor 叉出然后再合入aio-refactor吗?

BaseDownloader本意是作为一个基类,不过我看在 #567 已经做了修改(删除了很多未使用的方法),只留下了必要的方法。目前我认为应该不需要做额外处理,谢谢.

@EstrellaXD
Copy link
Owner

关于tr的更新应当直接从aio-refactor 叉出然后再合入aio-refactor吗?

是的,辛苦啦~

@codycjy codycjy closed this Oct 26, 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.

2 participants