-
Notifications
You must be signed in to change notification settings - Fork 48
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
Comments
I get the same message. It was working last week! |
It seems the UI was updated. I'll fix it soon. |
Actually, you don’t need to. I found a workaround. If you use python3 download_reloaded.py -user -p, it still downloads all the songs from your playlist without any error message. So I have created a dummy account for this purpose. Btw, I love this script, thanks for it. And keep it alive for as long as possible!
On 19Jun, 2020, at 12:28 PM, Monu <[email protected]> wrote:
It seems the UI was updated. I'll fix it soon.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub <#27 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AQAISZSFNQUWE4PRGAZLWGDRXMOUVANCNFSM4OCMCCTA>.
|
@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. |
Duplicate of #26 |
Please fix this error soon. |
@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. |
Thank you! It's working for me.now |
@monuyadav016 awesome!! I've been refreshing the tab since morning for update. |
@monuyadav016 it is not working!!! the terminal is stuck at enter url: |
can anyone help, I have reinstalled the script |
It got stuck for a minute or two. Then it produced following output:
|
@faheembhati I don't even get that output |
@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! |
@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. |
@monuyadav016 Thanks for the quick update. Let me download and test it. |
@monuyadav016 If an album contains ":" it throws following error: |
The text was updated successfully, but these errors were encountered: