search

SheltonZhu / 115driver

This is a 115 cloud driver package.
MIT License
67 stars 25 forks source link

Fix GetFiles #2

Closed northsea4 closed 2 years ago

northsea4 commented 2 years ago
  1. ApiFileListByName 数据异常问题（如缺失、重复、多出），暂时未找到解决方法，可以考虑放弃该接口；
  2. 改用ApiFileList，默认排序FileOrderByTime，natsort=1参数可能会导致请求出错，所以移除了该参数。

另外有个小建议，可以考虑增大limit的数值，减少请求和提高响应速度。

SheltonZhu commented 2 years ago
  1. ApiFileListByName 数据异常问题（如缺失、重复、多出），暂时未找到解决方法，可以考虑放弃该接口；
  2. 改用ApiFileList，默认排序FileOrderByTime，natsort=1参数可能会导致请求出错，所以移除了该参数。

另外有个小建议，可以考虑增大limit的数值，减少请求和提高响应速度。

看上去只改了3个地方， 最近加班有点忙，等周末我验证一下吧

northsea4 commented 2 years ago

我这边的分支有点问题，提交后才发现。实际只需要改一下我上面描述的几个地方即可。

SheltonZhu commented 2 years ago

我这边的分支有点问题，提交后才发现。实际只需要改一下我上面描述的几个地方即可。

已经squash merge 了，本来想用github的merge功能合并的，不会😂，弄的有点乱，只会用命令行

SheltonZhu commented 2 years ago

下次先rebase然后squash再提pr吧，这样我就可以直接web ui 合并了

northsea4 commented 2 years ago

👌🏻 我对这些也不太熟，有空学习一下