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

[nikobus] open connection on refresh even if no modules exist #9603

Merged
merged 4 commits into from
Jan 4, 2021

Conversation

gitwouter
Copy link
Contributor

Signed-off-by: Wouter Denayer [email protected]

open connection on refresh even if no modules exist

currently, when no modules are defined the connection to nikobus is not opened and therefore the button presses are not detected.

@gitwouter gitwouter requested a review from crnjan as a code owner December 30, 2020 15:42
@gitwouter gitwouter force-pushed the nikobus_buttons_without_modules branch 2 times, most recently from 38d9c9e to 4cf0bed Compare December 30, 2020 16:06
@Hilbrand Hilbrand added the bug An unexpected problem or unintended behavior of an add-on label Dec 30, 2020
@gitwouter gitwouter force-pushed the nikobus_buttons_without_modules branch from 4cf0bed to 0baf8e5 Compare December 31, 2020 10:06
Signed-off-by: Wouter Denayer <[email protected]>
Signed-off-by: Wouter Denayer <[email protected]>
Copy link
Contributor

@crnjan crnjan left a comment

Choose a reason for hiding this comment

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

lgtm

@crnjan crnjan added rebuild Triggers Jenkins PR build and removed rebuild Triggers Jenkins PR build labels Jan 1, 2021
@fwolter
Copy link
Member

fwolter commented Jan 3, 2021

You need to sign-off your commits. See https://www.openhab.org/docs/developer/contributing.html#sign-your-work The necessary commands are listed when you click on "Details" at the DCO check below.

@gitwouter gitwouter force-pushed the nikobus_buttons_without_modules branch from 720409b to 8021ce1 Compare January 3, 2021 18:32
@fwolter fwolter merged commit 1f3b0e2 into openhab:main Jan 4, 2021
@fwolter fwolter added this to the 3.1 milestone Jan 4, 2021
lucacalcaterra pushed a commit to lucacalcaterra/openhab-addons that referenced this pull request Feb 2, 2021
…b#9603)

* open connection on refresh even if no modules exist

Signed-off-by: Wouter Denayer <[email protected]>

* ran mvn spotless:apply

Signed-off-by: Wouter Denayer <[email protected]>

* removed unnecessary file

Signed-off-by: Wouter Denayer <[email protected]>

* returning when the connection is offline since following code won't work
either.

Signed-off-by: Wouter Denayer <[email protected]>

Co-authored-by: Wouter Denayer <[email protected]>
Signed-off-by: Luca Calcaterra <[email protected]>
thinkingstone pushed a commit to thinkingstone/openhab-addons that referenced this pull request Nov 7, 2021
…b#9603)

* open connection on refresh even if no modules exist

Signed-off-by: Wouter Denayer <[email protected]>

* ran mvn spotless:apply

Signed-off-by: Wouter Denayer <[email protected]>

* removed unnecessary file

Signed-off-by: Wouter Denayer <[email protected]>

* returning when the connection is offline since following code won't work
either.

Signed-off-by: Wouter Denayer <[email protected]>

Co-authored-by: Wouter Denayer <[email protected]>
marcfischerboschio pushed a commit to bosch-io/openhab-addons that referenced this pull request May 5, 2022
…b#9603)

* open connection on refresh even if no modules exist

Signed-off-by: Wouter Denayer <[email protected]>

* ran mvn spotless:apply

Signed-off-by: Wouter Denayer <[email protected]>

* removed unnecessary file

Signed-off-by: Wouter Denayer <[email protected]>

* returning when the connection is offline since following code won't work
either.

Signed-off-by: Wouter Denayer <[email protected]>

Co-authored-by: Wouter Denayer <[email protected]>
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.

5 participants