Skip to content

Comments

feat(downloader): 添加下载转发微博的原微博图片的功能#323

Merged
dataabc merged 1 commit intodataabc:masterfrom
schaepher:download_retweet_picture
Mar 28, 2021
Merged

feat(downloader): 添加下载转发微博的原微博图片的功能#323
dataabc merged 1 commit intodataabc:masterfrom
schaepher:download_retweet_picture

Conversation

@schaepher
Copy link
Collaborator

当前仅下载了原创微博的图片,没有下载转发微博的原微博图片。

@dataabc dataabc merged commit 0efd360 into dataabc:master Mar 28, 2021
@dataabc
Copy link
Owner

dataabc commented Mar 28, 2021

感谢贡献代码。

非常好的特性,其他修改也修复了很多bug,太感谢啦

@schaepher
Copy link
Collaborator Author

@dataabc
我也感谢你创建了这样一个优秀的项目,帮我节省了很多时间

@gudaocode
Copy link

@dataabc
我也感谢你创建了这样一个优秀的项目,帮我节省了很多时间

这样是否意味着这个爬虫,可以下载转发微博的全部图片?当大约9张时,也可以正常下载?

@schaepher
Copy link
Collaborator Author

这样是否意味着这个爬虫,可以下载转发微博的全部图片?当大约9张时,也可以正常下载?

@gudaocode 你想问的是图片大于9张时能否正常下载是吧?可以的。

@gudaocode
Copy link

这样是否意味着这个爬虫,可以下载转发微博的全部图片?当大约9张时,也可以正常下载?

@gudaocode 你想问的是图片大于9张时能否正常下载是吧?可以的。

OK,赞!!终于不用拿着爬虫爬一遍,再用免Cookie版再来一遍了,感谢!

@gudaocode
Copy link

这样是否意味着这个爬虫,可以下载转发微博的全部图片?当大约9张时,也可以正常下载?

@gudaocode 你想问的是图片大于9张时能否正常下载是吧?可以的。

但是有个问题,如何开关下载转发图片?
原来那个免Cookie版在config里用的是:
"original_pic_download": 0,
"retweet_pic_download": 1,
"original_video_download": 0,
"retweet_video_download": 0,

这个后面也会用类似的配置来供用户选择是否下载转发图片吗?

@gudaocode
Copy link

另外,转发的和非转发的也和那个类似,下载后放在不同的文件夹里面吗?
好像给你增加了不少麻烦,汗

@schaepher
Copy link
Collaborator Author

这个后面也会用类似的配置来供用户选择是否下载转发图片吗?

把 filter 设置为 1 ,会过滤掉转发的图片。

@schaepher
Copy link
Collaborator Author

另外,转发的和非转发的也和那个类似,下载后放在不同的文件夹里面吗?

你可以提 issue。这个可做可不做,看有多大的必要性。

1 similar comment
@schaepher
Copy link
Collaborator Author

另外,转发的和非转发的也和那个类似,下载后放在不同的文件夹里面吗?

你可以提 issue。这个可做可不做,看有多大的必要性。

@schaepher schaepher changed the title feat(downloader): 添加下载原微博图片的功能 feat(downloader): 添加下载转发微博的原微博图片的功能 Mar 31, 2021
@gudaocode
Copy link

另外,转发的和非转发的也和那个类似,下载后放在不同的文件夹里面吗?

你可以提 issue。这个可做可不做,看有多大的必要性。

建议是做区分比较好
其实如果是我个人建议,会建议在id list里面做区分是否下载转发的图片,毕竟当id list比较多时,有些id需要下载转发,有些不需要。不过,这个估计有点麻烦,就一支没说

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.

3 participants