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

[knx] Remove workaround for DPT 14 #14503

Merged
merged 1 commit into from
Feb 26, 2023

Conversation

holgerfriedrich
Copy link
Member

Fixed in Calimero 2.4.

Carryover from smarthomej/addons#104.

Also-by: Jan N. Klug github@klug.nrw

Fixed in Calimero 2.4.

Carryover from smarthomej/addons#104.

Also-by: Jan N. Klug <github@klug.nrw>
Signed-off-by: Holger Friedrich <mail@holger-friedrich.de>
@holgerfriedrich
Copy link
Member Author

holgerfriedrich commented Feb 25, 2023

@J-N-K can I start the carryover for the knx binding? Is the commit message / credit ok for you? I added you to the sign-off using an Also-by statement. Not sure if this is the intended way.
(may not be relevant for this PR where we basically remove code, but for the following ones....)

Refs: #14502

Copy link
Contributor

@lolodomo lolodomo left a comment

Choose a reason for hiding this comment

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

I assume you know what you do :)
LGTM

@lolodomo lolodomo merged commit 984914f into openhab:main Feb 26, 2023
@lolodomo lolodomo added the enhancement An enhancement or new feature for an existing add-on label Feb 26, 2023
@lolodomo lolodomo added this to the 4.0 milestone Feb 26, 2023
@holgerfriedrich
Copy link
Member Author

@J-N-K seems that the credits in the log message were removed.
When I submitted, it looked like this:

[knx] Remove workaround for DPT 14

Fixed in Calimero 2.4.

Carryover from smarthomej/addons#104.

Also-by: Jan N. Klug <github@klug.nrw>
Signed-off-by: Holger Friedrich <mail@holger-friedrich.de>

@J-N-K
Copy link
Member

J-N-K commented Feb 26, 2023

I guess that is a wrong edit during merging.

@lolodomo
Copy link
Contributor

I guess that is a wrong edit during merging.

I changed nothing when merging, I just take and keep unchanged what GitHub proposed. As there was at least one "Signed-off-by", I did not ask myself more questions.
Look at the other PR, I provided a screenshot.

@holgerfriedrich holgerfriedrich deleted the pr-knx-smarthomej104 branch February 27, 2023 21:01
nemerdaud pushed a commit to nemerdaud/openhab-addons that referenced this pull request Feb 28, 2023
Signed-off-by: Holger Friedrich <mail@holger-friedrich.de>
miloit pushed a commit to miloit/openhab-addons that referenced this pull request Mar 10, 2023
Signed-off-by: Holger Friedrich <mail@holger-friedrich.de>
Signed-off-by: miloit <MichaelLoercher@web.de>
renescherer pushed a commit to renescherer/openhab-addons that referenced this pull request Mar 23, 2023
Signed-off-by: Holger Friedrich <mail@holger-friedrich.de>
FordPrfkt pushed a commit to FordPrfkt/openhab-addons that referenced this pull request Apr 20, 2023
Signed-off-by: Holger Friedrich <mail@holger-friedrich.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement An enhancement or new feature for an existing add-on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants