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

[shelly] Fix BLU Discovery when Shelly Cloud Bluetooth Gateway is enabled #17167

Merged
merged 1 commit into from
Jul 30, 2024

Conversation

markus7017
Copy link
Contributor

The binding's gateway script could not get access to the BLE observer in case the Shelly Cloud Bluetooth Gateway is enabled. Therefore no discovery results will be passed to openHAB.

The binding now checks the device settings and adjust them if nessecary.

this case the binding's gateway script could not get access to the BLE
observer).

Signed-off-by: Markus Michels <markus7017@gmail.com>
@markus7017 markus7017 self-assigned this Jul 27, 2024
@markus7017 markus7017 added enhancement An enhancement or new feature for an existing add-on awaiting feedback Awaiting feedback from the pull request author labels Jul 27, 2024
@jlaur
Copy link
Contributor

jlaur commented Jul 28, 2024

@markus7017 - the label "awaiting feedback" is set when awaiting feedback from the contributor. When set, it signals low-priority and likely won't be picked up for a review. If you are waiting for user feedback, you could instead use the label "additional testing preferred". And to block the PR from being merged, you can mark it as a draft (and optionally add label "work in progress" for better visibility).

@markus7017 markus7017 removed the awaiting feedback Awaiting feedback from the pull request author label Jul 28, 2024
@markus7017
Copy link
Contributor Author

ok, good to know
a user confirmed that it works, so ready to merge

@markus7017
Copy link
Contributor Author

@jlaur it's ready to merge

@markus7017 markus7017 changed the title [shelly][WIP] Fixes BLU Discovery when Shelly Cloud Bluetooth Gateway is enabled [shelly] Fixes BLU Discovery when Shelly Cloud Bluetooth Gateway is enabled Jul 28, 2024
@markus7017
Copy link
Contributor Author

Please consider to also merge into 4.2

@jlaur jlaur added bug An unexpected problem or unintended behavior of an add-on and removed enhancement An enhancement or new feature for an existing add-on labels Jul 28, 2024
@jlaur jlaur changed the title [shelly] Fixes BLU Discovery when Shelly Cloud Bluetooth Gateway is enabled [shelly] Fix BLU Discovery when Shelly Cloud Bluetooth Gateway is enabled Jul 28, 2024
@markus7017 markus7017 requested a review from jlaur July 29, 2024 19:56
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 for the fix!

@jlaur jlaur merged commit 077cea0 into openhab:main Jul 30, 2024
5 checks passed
@jlaur jlaur added this to the 4.3 milestone Jul 30, 2024
jlaur pushed a commit that referenced this pull request Jul 30, 2024
#17167)

this case the binding's gateway script could not get access to the BLE
observer).

Signed-off-by: Markus Michels <markus7017@gmail.com>
@jlaur jlaur added the patch A PR that has been cherry-picked to a patch release branch label Jul 30, 2024
@openhab-bot
Copy link
Collaborator

This pull request has been mentioned on openHAB Community. There might be relevant details there:

https://community.openhab.org/t/shelly-binding/56862/4003

digitaldan pushed a commit to digitaldan/openhab-addons that referenced this pull request Aug 29, 2024
openhab#17167)

this case the binding's gateway script could not get access to the BLE
observer).

Signed-off-by: Markus Michels <markus7017@gmail.com>
pgfeller pushed a commit to pgfeller/openhab-addons that referenced this pull request Sep 29, 2024
openhab#17167)

this case the binding's gateway script could not get access to the BLE
observer).

Signed-off-by: Markus Michels <markus7017@gmail.com>
Signed-off-by: Patrik Gfeller <patrik.gfeller@proton.me>
joni1993 pushed a commit to joni1993/openhab-addons that referenced this pull request Oct 15, 2024
openhab#17167)

this case the binding's gateway script could not get access to the BLE
observer).

Signed-off-by: Markus Michels <markus7017@gmail.com>
matchews pushed a commit to matchews/openhab-addons that referenced this pull request Oct 18, 2024
openhab#17167)

this case the binding's gateway script could not get access to the BLE
observer).

Signed-off-by: Markus Michels <markus7017@gmail.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 patch A PR that has been cherry-picked to a patch release branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants