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

[goecharger] Fix command sending #14494

Merged
merged 1 commit into from
Feb 26, 2023
Merged

[goecharger] Fix command sending #14494

merged 1 commit into from
Feb 26, 2023

Conversation

DeJe63
Copy link
Contributor

@DeJe63 DeJe63 commented Feb 24, 2023

There was missing "break" preventing commands to be sent correctly.

@wborn wborn changed the title Update GoEChargerV2Handler.java [goecharger] Fix command sending Feb 25, 2023
@wborn wborn added the bug An unexpected problem or unintended behavior of an add-on label Feb 25, 2023
@schogit
Copy link

schogit commented Feb 25, 2023

Hello together,

I just can confirm @DeJe63 Bugfix - I was debugging the binding and found the same issue.
Best regards

schogit

@lolodomo
Copy link
Contributor

lolodomo commented Feb 26, 2023

Change looks good to me.
Can you please sign properly your PR ?

Edit: considering the name of the branch, I assume you edited the file directly from Github.

@lolodomo
Copy link
Contributor

As this is only 2 lines added and before the fix is obvious, I am merging even if the PR is not properly signed off.

@lolodomo lolodomo merged commit 52d77ea into openhab:main Feb 26, 2023
@lolodomo lolodomo added this to the 4.0 milestone Feb 26, 2023
nemerdaud pushed a commit to nemerdaud/openhab-addons that referenced this pull request Feb 28, 2023
miloit pushed a commit to miloit/openhab-addons that referenced this pull request Mar 10, 2023
Signed-off-by: miloit <MichaelLoercher@web.de>
renescherer pushed a commit to renescherer/openhab-addons that referenced this pull request Mar 23, 2023
FordPrfkt pushed a commit to FordPrfkt/openhab-addons that referenced this pull request Apr 20, 2023
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.

4 participants