dytt-reptitle icon indicating copy to clipboard operation
dytt-reptitle copied to clipboard

有点小bug

Open luchenwei9266 opened this issue 5 years ago • 4 comments

具体在readme里写了。不知道改得好不好

luchenwei9266 avatar May 27 '19 15:05 luchenwei9266

Codecov Report

Merging #8 into master will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@          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.

codecov-io avatar May 27 '19 15:05 codecov-io

看起来没有特别大的问题,说下我的两点观点

  • 磁力链可以作为额外参数传入,不需要去改动原来的downloadUrl格式
  • 跳页的需求场景暂时不明确,望说明~

可以加个微信交流 18979122835 ;)

HuangXiZhou avatar May 30 '19 08:05 HuangXiZhou

看起来没有特别大的问题,说下我的两点观点

  • 磁力链可以作为额外参数传入,不需要去改动原来的downloadUrl格式
  • 跳页的需求场景暂时不明确,望说明~

可以加个微信交流 18979122835 ;)

一开始我在dytt那里看到的下载FTP地址是提示要下载迅雷的。我试着想在自己的Demo项目中利用windown.open('ftp://xxx')来实现调起迅雷下载,但实际上实现不了。所以我改成了磁力链接,这样在用户体验上,比起用户自己复制ftp地址到迅雷上再下载可能会更好。不过磁力链确实可以考虑一个额外参数传入,因为有时候ftp的速度可能会比磁力链更快。

luchenwei9266 avatar May 30 '19 08:05 luchenwei9266

@luchenwei9266 有时间的话可以考虑改一下 pr,专门针对 skip 和磁力链即可

如果没有时间的话我之后稍作处理吧.. ;)

HuangXiZhou avatar Jun 05 '19 16:06 HuangXiZhou