Skip to content

[ecovacs] Fix device definition for T50 PRO OMNI#20017

Merged
jlaur merged 1 commit intoopenhab:mainfrom
maniac103:ecovacs-t50-fix
Jan 11, 2026
Merged

[ecovacs] Fix device definition for T50 PRO OMNI#20017
jlaur merged 1 commit intoopenhab:mainfrom
maniac103:ecovacs-t50-fix

Conversation

@maniac103
Copy link
Copy Markdown
Contributor

Unlike the T30, this device uses the custom water amount feature. It also was misdetected as T30S.

Fixes #19482

Unlike the T30, this device uses the custom water amount feature. It
also was misdetected as T30S.

Fixes openhab#19482

Signed-off-by: Danny Baumann <dannybaumann@web.de>
@maniac103
Copy link
Copy Markdown
Contributor Author

Once accepted, please also backport to 5.1.x.

I'm actually not sure about the state of all the other T30S variants listed in the device list (whether they use custom water amount or not). I took the list largely from here, but that library doesn't have support for custom water amount in either case, and given it's JS based, it likely won't choke on the lack of water amount in the response either.

Copy link
Copy Markdown
Contributor

@jlaur jlaur left a comment

Choose a reason for hiding this comment

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

Thanks for the fix, LGTM.

@jlaur jlaur added the bug An unexpected problem or unintended behavior of an add-on label Jan 11, 2026
@jlaur jlaur merged commit 973e556 into openhab:main Jan 11, 2026
3 checks passed
jlaur pushed a commit that referenced this pull request Jan 11, 2026
Unlike the T30, this device uses the custom water amount feature. It
also was misdetected as T30S.

Fixes #19482

Signed-off-by: Danny Baumann <dannybaumann@web.de>
@jlaur jlaur added the backported A PR that has been cherry-picked to a patch release branch label Jan 11, 2026
@jlaur jlaur added this to the 5.2 milestone Mar 1, 2026
Merlin10437 pushed a commit to Merlin10437/openhab-addons that referenced this pull request Mar 24, 2026
Unlike the T30, this device uses the custom water amount feature. It
also was misdetected as T30S.

Fixes openhab#19482

Signed-off-by: Danny Baumann <dannybaumann@web.de>
Signed-off-by: Merlin10437 <152161717+Merlin10437@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backported A PR that has been cherry-picked to a patch release branch bug An unexpected problem or unintended behavior of an add-on

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[ecovacs] Support for DEEBOT T50 Pro Omni Gen2

2 participants