-
-
Notifications
You must be signed in to change notification settings - Fork 242
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
[webaudio] Allow stopping play #2031
Conversation
Job #1240: Bundle Size — 15.75MiB (-0.08%).Important Bundle introduced 1 and removed 1 duplicate package – View changed duplicate packages Warning Bundle introduced 13 new packages: @jsep-plugin/regex, @jsep-plugin/arrow, @jsep-plugin/object and 10 more – View changed packages Bundle metrics
Bundle size by type
View job #1240 report View dalgwen:webaudio_stop branch activity |
393fe6a
to
c384a21
Compare
c384a21
to
bd356a7
Compare
Use of a dictionary to store audio sources currently playing and stop them all when asked to. A corresponding core (webaudio sink) modification is following. Signed-off-by: Gwendal Roulleau <[email protected]>
bd356a7
to
954355d
Compare
Thanks @florian-h05 for removing the "awaiting other PR" tag, as the core pull request was indeed merged. I rebased the code (as the "play" part moves to another file), and it is now ready for review. |
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.
LGTM, thanks!
Use of a dictionary to store audio sources currently playing and stop them all when asked to.
Refs openhab/openhab-core#3766.
Signed-off-by: Gwendal Roulleau [email protected]