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

new site #24

Open
wants to merge 27 commits into
base: master
Choose a base branch
from
Open

new site #24

wants to merge 27 commits into from

Conversation

alzamer2
Copy link
Collaborator

@alzamer2 alzamer2 commented Apr 9, 2017

+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

alzamer2 added 6 commits April 8, 2017 19:49
+fix the USA unblock option by replacing the old site because its no
longer working
+add Node.js Requires modules because some platform need it to login
now it work with usa blocked by using rss feed
@alzamer2
Copy link
Collaborator Author

+improve the auto catch
+improve usa unblocker option so that it does not overload the site

alzamer2 added 11 commits April 12, 2017 11:20
+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
@jaw20
Copy link
Owner

jaw20 commented Apr 21, 2017

please explain what auto cash is, do you mean auto catch? Because on my end this feature works fine.

@jaw20
Copy link
Owner

jaw20 commented Apr 21, 2017

Please make your pull request more focused.

@Dokman
Copy link
Contributor

Dokman commented Apr 21, 2017

@jaw20 it means for USA proxy :) i am using his version and then make massive download

@jaw20
Copy link
Owner

jaw20 commented Apr 21, 2017 via email

@Dokman
Copy link
Contributor

Dokman commented Apr 21, 2017

he fixed various things i think that he need to remake the commit description hahaha

@shinji257
Copy link
Contributor

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.

@jaw20
Copy link
Owner

jaw20 commented Apr 21, 2017 via email

@alzamer2
Copy link
Collaborator Author

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)
but later i found problem after other and i tried to fixed them that why the comment is jammed

so what do you want me to do?
open branch for each major modfied?
or fix the pull description?

@alzamer2
Copy link
Collaborator Author

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
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.

4 participants