-
Notifications
You must be signed in to change notification settings - Fork 70
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
CLI support for websites #28
Comments
First of all, sorry for the very late reply. If somehow it is required by more people, I will add functionality for CLI as well. |
Ah, no worries. I did end up using the GUI, so it worked out. If that's the case, it would be handy if the documentation mentioned that the CLI only supports 9anime, and to use the GUI for other sites. The README makes it sound like you can use either for any of the supported sites. Is there a reason you didn't write it as a CLI and then the GUI just being a wrapper? I'd be interested in helping with that, but depends on how much would need to be rewritten. |
Yes, I have edited the readme already in the dev branch just not in the current master :D This is originally a CLI and yes, it has then GUI been wrapped around it. However as I added more website support (after GUI) I quite didn't add the same website support for CLI. However, if it's needed to add all the website supported using CLI, it will be easy (you just have to add another parameter to the CLI commands list to input the website (and other new functionalities added)). Almost same as GUI work since I have used the same format for both. Have a look at it! :D Another dev will really help in maintaining this since I have been busy. Also, you don't have to actually rewrite many things, just add some more parameters and call the relevant scraper as for that :D |
Cool, sounds like fun. Is the |
Nope, you can use the dev branch to start working on it. I will only be working on the update files and scrapers :D Edit: If you're to implement any new features, open an issue first so we can keep track of them :) |
The entire CLI functionality was removed in 4ee3c33. It also looks like you're putting everything into one file ( To clean up the code a bit, I think it would be worth it to move the I am attempting this change and will let you know how it works out. |
Attempting to download a show from animefreak using the CLI causes errors, but seems to work with the GUI. This appears to be because
Anime_Downloader.py
only importsscrapers.nineanime
and does not check which scaper to use for different links inside of themain
function.The text was updated successfully, but these errors were encountered: