-
Notifications
You must be signed in to change notification settings - Fork 1
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
Refactor translation functions #1
Comments
Hi, |
Thanks for your kind words! I'd be happy for you to merge once done. However, I'm a self-taught programmer, and I've never done any work on NVDA addons before. I'm also more comfortable in C#, and this is my first real work in Python. So it might take me longer than expected to do this refactor. That's why the code is a mess right now. :-) I needed something that worked, because Google blocked me from the translate service for making too many calls. My current plan is:
Questions I'm not sure about:
If you have any advice or feedback, I would appreciate it! |
In order to make it possible to add multiple translation services, a code refactor is needed. We need to create a class to wrap the various API's, expose a universal translate method, and a standard set of config getters and setters. Once that's done, we can add google, yandex, and Libre translate.
The text was updated successfully, but these errors were encountered: