-
Notifications
You must be signed in to change notification settings - Fork 22
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
fix: disable if tts gets too long #1195
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you add some unit tests?
refactor: setup ttsChannel to be a getter, defaulting to true
…branch now and allows the build to compile
…late listens to changes in this stream
refactor: import main.dart to enable channel swapping via it's function updateChannelSubscription fix: queued text to speech not working correctly
…ted before going to the next item in the queue fix: fixed an error where it would sometimes delete the item that was next in queue feat: create a ttsqueueelement class to be used throughout our main queue class as needed fix: disabling of text to speech when messages are greater than 20
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice, code looks good. @SputNikPlop can you verify and test?
Co-authored-by: Kevin Wang <kevmo314@gmail.com>
Co-authored-by: Kevin Wang <kevmo314@gmail.com>
Co-authored-by: Kevin Wang <kevmo314@gmail.com>
Co-authored-by: Kevin Wang <kevmo314@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, just a few comments.
This disabled tts if the queue gets too long.