Skip to content

Fix for DribbbleRipper#2129

Merged
soloturn merged 2 commits intoRipMeApp:mainfrom
jorelm68:ethan
Aug 7, 2025
Merged

Fix for DribbbleRipper#2129
soloturn merged 2 commits intoRipMeApp:mainfrom
jorelm68:ethan

Conversation

@jorelm68
Copy link
Copy Markdown
Contributor

@jorelm68 jorelm68 commented Apr 23, 2025

Category

This change is exactly one of the following (please change [ ] to [x]) to indicate which:

Description

I went to https://dribbble.com/typogriff to figure out what changed on the website. I used inspect to investigate the images on the website. I found out that each thumbnail image was actually a link to a dedicated webpage with the full image. On that dedicated webpage, there was a link to the image address. Going back to the code, I had to scrap the old system. The old system involved grabbing the image addresses directly from the main page. Once finishing my implementation, the new system involved grabbing the link to the dedicated webpage for each thumbnail, opening the link, then grabbing the image address within each webpage.

Testing

Required verification:

  • I've verified that there are no regressions in gradlew test (there are no new failures or errors).
  • I've verified that this change works as intended.
    • Downloads all relevant content.
    • Downloads content from multiple pages (as necessary or appropriate).
    • Saves content at reasonable file names (e.g. page titles or content IDs) to help easily browse downloaded content.
  • I've verified that this change did not break existing functionality (especially in the Ripper I modified).

Optional but recommended:

  • I've added a unit test to cover my change.

@jorelm68 jorelm68 changed the title Ethan Fix for DribbbleRipper Apr 24, 2025
@soloturn soloturn merged commit c1c2a7c into RipMeApp:main Aug 7, 2025
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.

DribbbleRipper not working - site changed

2 participants