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

Keep getting this index out of range error. Any ideas? #27

Closed
hyndraslic opened this issue Jun 19, 2020 · 18 comments
Closed

Keep getting this index out of range error. Any ideas? #27

hyndraslic opened this issue Jun 19, 2020 · 18 comments
Labels
bug Something isn't working

Comments

@hyndraslic
Copy link


Error

@vaishakhgarg
Copy link

I get the same message. It was working last week!

@inovachrono
Copy link
Owner

It seems the UI was updated. I'll fix it soon.

@vaishakhgarg
Copy link

vaishakhgarg commented Jun 19, 2020 via email

@inovachrono
Copy link
Owner

@vaishakhgarg Actually it does. Since many have reported it. Also the functionality for users with accounts is different and does not rely on web scraping the ID for playlist and albums. So changes in UI can break functionality for those using it without having a jiosaavn account.

@inovachrono inovachrono added the bug Something isn't working label Jun 19, 2020
@inovachrono
Copy link
Owner

Duplicate of #26

@amail101
Copy link

Please fix this error soon.

@inovachrono
Copy link
Owner

@hyndraslic @vaishakhgarg @amail101 A small fix for downloading albums and playlist has been done. I'll have to figure out a fix for artist and individual songs. Download and update the scripts.

@hyndraslic
Copy link
Author

Thank you! It's working for me.now

@faheembhati
Copy link

@monuyadav016 awesome!! I've been refreshing the tab since morning for update.
Please don't ever discontinue this beautiful script.
Thanks a lot!

@vaishakhgarg
Copy link

@monuyadav016 it is not working!!! the terminal is stuck at enter url:

@vaishakhgarg
Copy link

can anyone help, I have reinstalled the script

@faheembhati
Copy link

It got stuck for a minute or two. Then it produced following output:

Unable to get albumID:
DONE

@vaishakhgarg
Copy link

@faheembhati I don't even get that output

@inovachrono
Copy link
Owner

@faheembhati @vaishakhgarg I know this fix is not perfect because the updated UI is making it slow to obtain the ID. The json embedded in their web pages is broken. Fixing that is slow and the script would feel stuck and at times may not work. I'll have to look more into it. For now I recommend creating a temporary account and using it to download the playlist.

@vaishakhgarg
Copy link

@faheembhati @vaishakhgarg I know this fix is not perfect because the updated UI is making it slow to obtain the ID. The json embedded in their web pages is broken. Fixing that is slow and the script would feel stuck and at times may not work. I'll have to look more into it. For now I recommend creating a temporary account and using it to download the playlist.

no problem man! the user playlist command is still working, you're awesome!

@inovachrono
Copy link
Owner

@faheembhati @hyndraslic @vaishakhgarg @amail101 I have fixed it properly now. You can update the scripts and test it. Now the downloads should be as fast as it was before.

@faheembhati
Copy link

faheembhati commented Jun 20, 2020

@monuyadav016 Thanks for the quick update. Let me download and test it.
UPDATE: It's working fine now.

@faheembhati
Copy link

faheembhati commented Jun 20, 2020

@monuyadav016 If an album contains ":" it throws following error:
NotADirectoryError: [WinError 267] The directory name is invalid: 'C:\Users\xxxxxx\Non-Artist\xxxx: xxxxxx'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

5 participants