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

Making it easier to download larger collections #18

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Fewshin
Copy link

@Fewshin Fewshin commented Sep 21, 2020

I formatted it the way I like since I didn't anticipate making a PR

// Whether the download tab should automatically be closed after the download has been started
var closeAfterDownload = true
var closeAfterDownload = false //some weirdo shit happens when this is true. Will need to investigate more
Copy link
Owner

Choose a reason for hiding this comment

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

What's the weirdo shit?

Copy link
Author

Choose a reason for hiding this comment

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

Not sure, I was missing downloads when close after download was true but when I left it off it seemed to work fine. I'm not confident on what the issue was so I just noted for myself the behavior seemed off so if I wanted to investigate it I could

var albumLoadDuration = 10000;
var albumLoadDuration = 30000; //30000 ms default - 30 sec default
//If true will make the time it takes to open new links entirely random. If false will use a set interval
const randomInterval = false;
Copy link
Owner

Choose a reason for hiding this comment

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

What's the purpose of having this option?

Copy link
Author

Choose a reason for hiding this comment

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

Was my initial approach. I left it in because I figured since I already programmed it having the option would be nice

@RyanBluth
Copy link
Owner

Can we remove the semicolons so that things are consistent? I don't really care if they're there or not, but I'd like to be consistent

@Fewshin
Copy link
Author

Fewshin commented Sep 22, 2020

Yeah, no complaints. I just program mostly in C and Verilog these days so it's force of habit. I also wouldn't be bothered if const and let were replaced by var. More force of habit stuff since I use predominantly ES6. I present the code as is for you to do whatever you want with. I mostly put it together for myself but I saw someone opened an issue asking for a fix I already made I figured I should share it.

@RyanBluth
Copy link
Owner

Sounds good to me. I'll make those few tweaks and add it in when I have a chance

@ligi
Copy link

ligi commented Jan 10, 2021

would love to see this merged. Just tried to download my collection and get the 429 error. Thanks for this software btw!

@johnsoca2
Copy link

I can't get this to work correctly. I update the script for the download format and select 6 albums. The page for the first album opens and the format is applied to the download, but nothing downloads and it does not continue to the next album. Did Bandcamp change something about their site that needs a different command to start the download? I would try to diagnose it, but I don't know java very well. Thanks.

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.

5 participants