-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
Conversation
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 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. Refs: #14502 |
There was a problem hiding this 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
@J-N-K seems that the credits in the log message were removed.
|
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. |
Signed-off-by: Holger Friedrich <mail@holger-friedrich.de>
Signed-off-by: Holger Friedrich <mail@holger-friedrich.de> Signed-off-by: miloit <MichaelLoercher@web.de>
Signed-off-by: Holger Friedrich <mail@holger-friedrich.de>
Signed-off-by: Holger Friedrich <mail@holger-friedrich.de>
Fixed in Calimero 2.4.
Carryover from smarthomej/addons#104.
Also-by: Jan N. Klug github@klug.nrw