-
Notifications
You must be signed in to change notification settings - Fork 4
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
Broken? #12
Comments
Fixed, plz pull the newest master branch and try again... |
Hello, I cloned the latest master and it still returns the same error.
|
If possible, may you provide the gallery link? Maybe pagination selector is not suited for every gallery... |
http://oevrlord.deviantart.com/ |
I've tried http://oevrlord.deviantart.com/gallery/, it works. Remember to use GALLERY's PAGE, not author's page... |
@nonamethanks Is it ok now? If everything is find, I would like to close this issue. |
Yes, if I use the /gallery/ url it works @xofred |
Glad to hear it! |
Requesting re-open. I purged the install of this script, still not working.
|
Please try |
Same result. |
|
I ran the same command you did and got the same error. |
@vxbinaca Could you provide the detailed error message, such as error line of code? |
@xofred ptrace? That's the output I got:
|
@vxbinaca Plz check out dev branch, it may solve the issue. And it can provide more error messages if not. |
I was having the same issue on my debian system, and the dev branch fixed it. |
EDIT: It's only fetching the first page (usually around 22 or so) then it thinks it's done. With or without the "/?catpath=/" Tested on 4 different user's galleries, same results. |
Without it. If it still not work, Could you provide the gallery link for me to debug? @1n5aN1aC |
Well, something like I've tried it on 3 other artists' galleries as well, and all of them I've tried so far, either with personal credential, or with -n seem to do the same thing. |
Turns out to be an account setting issue. Please go to https://www.deviantart.com/settings/browsing. In "General Browsing" section, make sure "Display" as "24 Thumbnails per Page" and "Paging" as "Click through pages" |
Ahh, so "scroll through pages" isn't compatible. Makes sense, even if it is disappointing. I'd suggest you add that tidbit to a FAQ somewhere, but it's solved for me! |
Good point, README updated. |
Requesting a re-open this has cropped back up, or the open issue might possibly be a fix. Why he diff'd instead of a PR is beyond me. |
Sadly, can confirm. As of maybe a little over a week ago, even the dev branch is broken again. |
The generated credentials are brand new. I can log in thru the browser using them just fine (I'm unable to test them in other software such as 4pr0n/ripme due to changes in DA's website (see 4pr0n/ripme#425); I'm not sure if perhaps this issue is caused by the same changes?
The text was updated successfully, but these errors were encountered: