Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Eroshare naming enhancements and XhamsterRipper method refactoring. #358

Open
wants to merge 22 commits into
base: master
Choose a base branch
from
Open

Eroshare naming enhancements and XhamsterRipper method refactoring. #358

wants to merge 22 commits into from

Conversation

sleaze
Copy link

@sleaze sleaze commented Oct 27, 2016

  • Eroshare naming now includes username, gallery-id, and gallery name.

e.g. for url=https://eroshare.com/aB12C3d uploaded by sample-user with
title="hello there", the directory name will be:

eroshare_sample-user_aB12C3d_hello-there

The raw HTML page is also now saved for eroshare.

  • Refactored utility methods from XhamsterRipper into base AlbumRipper.

Kastang and others added 22 commits June 24, 2016 18:07
- Eroshare naming now includes username, gallery-id, and gallery name.

e.g. for url=https://eroshare.com/aB12C3d uploaded by sample-user with
title="hello  there", the directory name will be:

    eroshare_sample-user_aB12C3d_hello-there

The raw HTML page is also now saved for eroshare.

- Refactored utility methods from XhamsterRipper into base AlbumRipper.
@metaprime
Copy link
Collaborator

Hi @sleaze - it looks like you've got a bunch of PRs open that mostly are unrelated but include a lot of previous unrelated commits. Is it feasible to split them out into independent commits and PRs that I can test and merge one at a time?

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.

2 participants