Skip to content
This repository has been archived by the owner on Sep 28, 2020. It is now read-only.

Added the ratebypass flag to the download url. This drastically speeds u... #94

Merged
merged 2 commits into from Jan 16, 2015
Merged

Conversation

ghost
Copy link

@ghost ghost commented Dec 23, 2014

Hey, hopefully this fixes Issue #64

@flagbug
Copy link
Owner

flagbug commented Dec 23, 2014

Do this in the ExtractDownloadUrls function and check if the ratebypass parameter exists before we add it, so we don't add a duplicate parameter

Also your indentation is a bit off, are you using tabs instead of spaces?

@ghost
Copy link
Author

ghost commented Dec 23, 2014

Yeah, I am using tabs, you'd rather I used spaces? I'll get onto it

@flagbug
Copy link
Owner

flagbug commented Dec 23, 2014

@thesquareof4 Yup, always use spaces

@ghost
Copy link
Author

ghost commented Dec 23, 2014

Hopefully that's closer to what you want. Sorry, I'm going to be away for a while so if you want me to fix something else I might not respond too quickly

@flagbug flagbug self-assigned this Dec 30, 2014
flagbug added a commit that referenced this pull request Jan 16, 2015
Added the ratebypass flag to the download url. This drastically speeds u...
@flagbug flagbug merged commit a46d929 into flagbug:master Jan 16, 2015
@flagbug
Copy link
Owner

flagbug commented Jan 16, 2015

Thanks @thesquareof4 and sorry it took so long to merge this!

@fallaciousreasoning
Copy link

Hey no worries, this is an awesome library you've got going here, I'm glad I got a chance to contribute

@ghost
Copy link
Author

ghost commented Jan 16, 2015

Eh sorry, wrong account

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants