TumblThree
TumblThree copied to clipboard
Download Twitter profile and banner images
Is your feature request related to a problem? Please describe. At the moment the Twitter downloader isn't downloading the banner and profile images of the downloaded blog.
Describe the solution you'd like I would like that the profile and banner images of a blog are downloaded too. New images shouldn't overwrite the previous ones. And maybe they could be grouped together somehow so that they are not distributed across the whole folder.
Describe alternatives you've considered Downloading by hand is not easily possible as it needs actively looking for changes.
Additional context
As before this one is for first-timers-only. That means that we will only accept a PR for this one from someone who's never contributed to open source before. This one is particularly easy (but don't make that statement make you feel bad if you have a hard time with it, there's more to contributing to open source than changing lines of code, especially if it's your first time). We walk you through it if you need help. :-) Here are the steps to get a PR merged (see also our guide).
When downloading a Twitter blog its banner and profile images shall be downloaded too. Old versions shall not be overwritten and new versions shall only be downloaded/saved if they really changed.
- [x] Create a fork of the repository
- [x] Clone it to your local disk and create a new branch (e.g. 189-download-profile-images)
- [x] Look which class is dealing with the Twitter blog download. Where is the entry method of the Twitter crawler? What needs to be done to download these images? Do the filenames change or not if the blog owner changes the images? How can it be solved?
- [x] Add your solution
- [x] Build the app and run it
- [x] Create yourself a test blog where you can change the banner and profile images for testing
- [x] Alternatively: simulate previously downloaded banner/profile images of another blog by using different renamed test images
- [x] Crawl/Download the blog again
- [x] Verify that the changed profile and banner images have been downloaded
- [x] Commit your changes mentioning "
issue #189
" somewhere in your commit message title or description - [x] Push your changes to your fork, create a PR, get merged, celebrate??
We are happy to support you if you need help.
Hi, looking into trying to solve the issue.
Hello, thanks for your interest to work on this issue. I assign it to you. If you have a question or get stuck, feel free to ask.
@SantruptaMishra98 Hi, do you have any questions?
Ill give this issue a go as it seems @SantruptaMishra98 is inactive.
Yes, seems so. So it can be taken by a new volunteer.
Thanks for your interest. But if we'd stick closely to our rules, you'd not qualify as a first-time contributor. Do you like to solve e.g. #206 or #211 instead?
Okay, I'll try to solve #206.