dytt-reptitle
dytt-reptitle copied to clipboard
有点小bug
具体在readme里写了。不知道改得好不好
Codecov Report
Merging #8 into master will not change coverage. The diff coverage is
n/a
.
@@ Coverage Diff @@
## master #8 +/- ##
=====================================
Coverage 100% 100%
=====================================
Files 3 3
Lines 12 12
=====================================
Hits 12 12
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 4d7ef25...ecd3e9c. Read the comment docs.
看起来没有特别大的问题,说下我的两点观点
- 磁力链可以作为额外参数传入,不需要去改动原来的downloadUrl格式
- 跳页的需求场景暂时不明确,望说明~
可以加个微信交流 18979122835 ;)
看起来没有特别大的问题,说下我的两点观点
- 磁力链可以作为额外参数传入,不需要去改动原来的downloadUrl格式
- 跳页的需求场景暂时不明确,望说明~
可以加个微信交流 18979122835 ;)
一开始我在dytt那里看到的下载FTP地址是提示要下载迅雷的。我试着想在自己的Demo项目中利用windown.open('ftp://xxx')
来实现调起迅雷下载,但实际上实现不了。所以我改成了磁力链接,这样在用户体验上,比起用户自己复制ftp地址到迅雷上再下载可能会更好。不过磁力链确实可以考虑一个额外参数传入,因为有时候ftp的速度可能会比磁力链更快。
@luchenwei9266 有时间的话可以考虑改一下 pr,专门针对 skip 和磁力链即可
如果没有时间的话我之后稍作处理吧.. ;)