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

Update update.lst with alert for deprecated electroluxair binding #1681

Merged
merged 4 commits into from
Aug 25, 2024

Conversation

jannegpriv
Copy link
Contributor

The ElectroluxAir binding is removed since the Electrolux Delta API has been discontinued.

Signed-off-by: Jan Gustafsson <jannegpriv@gmail.com>
@jannegpriv jannegpriv requested a review from a team as a code owner August 24, 2024 21:16
Copy link
Member

@wborn wborn left a comment

Choose a reason for hiding this comment

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

It should be added to the [[MSG]] section (containing all the other ALERTs) and not this [[POST]] section.

Signed-off-by: Jan Gustafsson <jannegpriv@gmail.com>
@@ -150,6 +150,7 @@ ALERT;Transformations-MAP: "-" entry defined in a MAP file is no more used by si
[4.3.0]
ALERT;JavaScript Automation: The isJsInstanceOfJavaType method of the utils namespace has been removed. Use JavaScript's instanceof operator instead.
ALERT;Pentair Binding: EasyTouch thing has been renamed to more generic Controller and all channels have been organized into groups. You will need to reconfigure your setup to the new thing structure.
ALERT;ElectroluxAir binding: The binding has been removed since the Electrolux Delta API has been discontinued.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
ALERT;ElectroluxAir binding: The binding has been removed since the Electrolux Delta API has been discontinued.
ALERT;ElectroluxAir Binding: The binding has been removed since the Electrolux Delta API has been discontinued.

Can you also keep the list sorted by moving it before the JavaScript Automation ALERT? 🙂

Signed-off-by: Jan Gustafsson <jannegpriv@gmail.com>
Signed-off-by: Wouter Born <github@maindrain.net>
Copy link
Member

@wborn wborn left a comment

Choose a reason for hiding this comment

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

Thanks!

@wborn wborn merged commit 5478ef9 into openhab:main Aug 25, 2024
4 of 5 checks passed
@wborn wborn added this to the 4.3 milestone Aug 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants