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

[pixiv] support tag for user downloads #17

Merged
merged 3 commits into from
Jun 4, 2017
Merged

[pixiv] support tag for user downloads #17

merged 3 commits into from
Jun 4, 2017

Conversation

aiasdfd
Copy link
Contributor

@aiasdfd aiasdfd commented Jun 2, 2017

Pixiv user links can include a tag (https://www.pixiv.net/member_illust.php?id={user}&tag={tag}), which means to only show images with that tag. This PR adds support for that to the extractor.

@mikf
Copy link
Owner

mikf commented Jun 3, 2017

Thanks for your contribution.

Looking at this made me realize that the pixiv module really needs an overhaul.

There are some minor issues with your code (regex, for example), but I'm just going to merge this as is and fix these myself while rewriting the pixiv module, if that is OK with you.

@aiasdfd
Copy link
Contributor Author

aiasdfd commented Jun 4, 2017

Sounds good to me. I didn't really know what I was doing, just wanted the feature so I threw something together that mostly works. Looking forward to the new and improved module when you do the rewrite! And thanks for making this great tool!

@mikf mikf merged commit 338f791 into mikf:master Jun 4, 2017
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