TumblThree icon indicating copy to clipboard operation
TumblThree copied to clipboard

Download Twitter profile and banner images

Open thomas694 opened this issue 3 years ago • 7 comments

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

thomas694 avatar Nov 20 '21 14:11 thomas694

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.

thomas694 avatar Nov 20 '21 14:11 thomas694

Hi, looking into trying to solve the issue.

SantruptaMishra98 avatar Nov 25 '21 17:11 SantruptaMishra98

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.

thomas694 avatar Nov 26 '21 00:11 thomas694

@SantruptaMishra98 Hi, do you have any questions?

thomas694 avatar Dec 23 '21 22:12 thomas694

Ill give this issue a go as it seems @SantruptaMishra98 is inactive.

kulesy avatar Feb 07 '22 03:02 kulesy

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?

thomas694 avatar Feb 07 '22 21:02 thomas694

Okay, I'll try to solve #206.

kulesy avatar Feb 07 '22 21:02 kulesy