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

Better error checking for offline use.. Add in more reliability #7

Closed
willwade opened this issue Aug 20, 2023 · 1 comment
Closed

Comments

@willwade
Copy link
Contributor

willwade commented Aug 20, 2023

  • If you are offline are trying to translate - or its too slow or some other error - the error capturing right now isn't nice. We should try and do things to capture and quietly present errors
  • Likewise with Azure TTS
  • We should have an option to turn on or off these easygui message boxes. SOme people who have disabilities wont be able to close them (or alternatively, a way of it autoclosing the dialog after a set time
  • In general I think the code could do with some refactoring to look nicer. It started off as a script but its getting quite hard to read now
@acerjay
Copy link
Contributor

acerjay commented Aug 27, 2023

  • If you are offline are trying to translate - or its too slow or some other error - the error capturing right now isn't nice. We should try and do things to capture and quietly present errors
    • We should have an option to turn on or off these easygui message boxes. SOme people who have disabilities wont be able to close them (or alternatively, a way of it autoclosing the dialog after a set time

bb4c19a

Replace easygui with pyside6 | Messagebox has auto close feature (Default is 10 seconds) | Add Error handling when there is no existing settings.cfg and user click yes on launch the "Configure TranslateAndTTS" but discard the set-up. Normally, It will raise some error since no config file was found, but using this fix, a message box will notify the user that there is no config file created and the app will exit | Remove Pygame welcome message

acerjay added a commit to acerjay/TranslateAndTTS that referenced this issue Aug 28, 2023
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

No branches or pull requests

2 participants