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

[yamahamusiccast] Fix VolumeDB channel resets on UDP event #12671

Merged
merged 3 commits into from May 1, 2022

Conversation

florian-h05
Copy link
Contributor

This PR fixes a issue that was introduced by my PR #12519.
The recently added volume in dB channel resetted on UDP events that do not contain volume information due to a bug.

Fixes #12656.

@jlaur Maybe you could review?

Signed-off-by: Florian Hotze florianh_dev@icloud.com

Fixes [yamahamusiccast] VolumeDB channel resets to -80 on player channel change openhab#12656.

Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
@florian-h05 florian-h05 requested a review from coop-git as a code owner May 1, 2022 09:25
Copy link
Contributor

@jlaur jlaur left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, glad to see this -90 dB magic value gone. :) A few minor comments added.

@jlaur jlaur added the bug An unexpected problem or unintended behavior of an add-on label May 1, 2022
Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
@florian-h05 florian-h05 requested a review from jlaur May 1, 2022 10:46
@florian-h05
Copy link
Contributor Author

@jlaur
Ready for re-review.

Copy link
Contributor

@jlaur jlaur left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - thanks!

@jlaur jlaur merged commit 69c3b0f into openhab:main May 1, 2022
@jlaur jlaur added this to the 3.3 milestone May 1, 2022
@florian-h05 florian-h05 deleted the yamahamusiccast-fix12656 branch May 1, 2022 10:50
andan67 pushed a commit to andan67/openhab-addons that referenced this pull request Nov 6, 2022
…2671)

* [yamahamusiccast] Fix VolumeDB resets on UDP event
* [yamahamusiccast] Update channel description & docs

Fixes [yamahamusiccast] VolumeDB channel resets to -80 on player channel change openhab#12656.

Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
andrasU pushed a commit to andrasU/openhab-addons that referenced this pull request Nov 12, 2022
…2671)

* [yamahamusiccast] Fix VolumeDB resets on UDP event
* [yamahamusiccast] Update channel description & docs

Fixes [yamahamusiccast] VolumeDB channel resets to -80 on player channel change openhab#12656.

Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
Signed-off-by: Andras Uhrin <andras.uhrin@gmail.com>
psmedley pushed a commit to psmedley/openhab-addons that referenced this pull request Feb 23, 2023
…2671)

* [yamahamusiccast] Fix VolumeDB resets on UDP event
* [yamahamusiccast] Update channel description & docs

Fixes [yamahamusiccast] VolumeDB channel resets to -80 on player channel change openhab#12656.

Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug An unexpected problem or unintended behavior of an add-on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[yamahamusiccast] VolumeDB channel resets to -80 on player channel change
2 participants