-
Notifications
You must be signed in to change notification settings - Fork 14
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
new site #24
base: master
Are you sure you want to change the base?
new site #24
Conversation
now it work with usa blocked by using rss feed
+improve the auto catch |
+remove usa blocker improvvment because it not working
thanks shinji257 for fix
auto-cash now work in this order if one failed go to other: 1-RSS 2-Site Html 3-Site Html+USA unblocker
remove extra lines
…all seasons can some give feedback for this fix
+ Total size calculater +downladed size update +download speed +multi download connection to boosth speed
thanks to Bandido06
fix the huge files appending
the speed calculation was in bytes while the unit as in bit its fixed now
please explain what auto cash is, do you mean auto catch? Because on my end this feature works fine. |
Please make your pull request more focused. |
@jaw20 it means for USA proxy :) i am using his version and then make massive download |
That's what it was originally for, but he has made multiple new commits
which do not relate to the original pull request.
…On Apr 21, 2017 1:03 PM, "Sergi Canaleta Fondó" ***@***.***> wrote:
@jaw20 <https://github.com/jaw20> it means for USA proxy :) i am using
his version and then make massive download
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#24 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AE0jc93coZ5KCl1kLSRmLXharNvTmmErks5ryOFVgaJpZM4M4DK->
.
|
he fixed various things i think that he need to remake the commit description hahaha |
I'm going to be fairly honest here. What he should have done was done a branch for the intended feature then submitted the pull request against that. If he wanted to he could have merged that branch back into his fork for further unrelated features. That is what I do when submitting pull requests for either fixes or new features. |
Exactly. He needs to make new pull request for each new feature or fix or
he needs to be more specific about what is happening here.
|
ok first of all welcome back about my pull i intended at first to only fix the USA Proxy problem ( i live outside the us and the that site no longer work) so what do you want me to do? |
ok i tried to split them up and made new pull request |
this is fix for long path name in the script as in issue #1
# Conflicts: # crunchy-xml-decoder.bat
use reqquest function if urllib2 SSL verification failed
replace not working Crunchyroll unblocker
+fix the USA unblock option by replacing the old site because its no
longer working #13
+add Node.js Requires modules because some platform need it to login