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

[mangasee] add support for 'mangalife' #3086

Merged
merged 2 commits into from
Oct 21, 2022
Merged

Conversation

enduser420
Copy link
Contributor

Copy link
Owner

@mikf mikf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

I kind of dislike how verbose the code to handle two different domains is, but I guess it is the same for kemono and coomer and there really is no better way at the moment.

Using BaseExtractor like in a7d23f1 would be nice, except it most likely does not at all work with ChapterExtractor and MangaExtractor base classes.

Anyway, we can get rid of self.domain by using only part of self.root when needed.

gallery_dl/extractor/mangasee.py Outdated Show resolved Hide resolved
gallery_dl/extractor/mangasee.py Outdated Show resolved Hide resolved
gallery_dl/extractor/mangasee.py Outdated Show resolved Hide resolved
Co-authored-by: Mike Fährmann <[email protected]>
@mikf mikf merged commit ac6111e into mikf:master Oct 21, 2022
@enduser420 enduser420 deleted the extractor/mangasee branch October 21, 2022 14:50
mikf pushed a commit that referenced this pull request Oct 29, 2022
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