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

[MQTT.Homeassistant] Make the mqtt.vacuum compliant #11562

Merged
merged 1 commit into from
Nov 20, 2021

Conversation

antroids
Copy link
Contributor

[MQTT.Homeassistant] make the mqtt.vacuum compliant with the Specification

The mqtt.vacuum HA component has a Specification at https://www.home-assistant.io/integrations/vacuum.mqtt/
The goal of this PR is to make OpenHAB implementation compliant with the Specification, to be sure that any vacuum can be supported by Homeassistant integration out of the box.

I added support for legacy schema and all missing configuration attributes.
Unfortunately, time and usage status attributes are out of specification and they are specific for old valetudo versions only. So, I removed them from implementation. The new valetudo firmware should be supported by Homie binding. Any firmware specific mqtt messages can also be handled by generic mqtt binding.

…th the specification

Signed-off-by: Anton Kharuzhy <publicantroids@gmail.com>
@cweitkamp cweitkamp added the enhancement An enhancement or new feature for an existing add-on label Nov 12, 2021
@fwolter fwolter merged commit 4a3a9d5 into openhab:main Nov 20, 2021
@fwolter
Copy link
Member

fwolter commented Nov 20, 2021

You could set yourself as the code owner in the CODEOWNERS file.

@fwolter fwolter added this to the 3.2 milestone Nov 20, 2021
@antroids
Copy link
Contributor Author

antroids commented Nov 20, 2021

You could set yourself as the code owner in the CODEOWNERS file.

Do I need to create a PR for this?

@fwolter
Copy link
Member

fwolter commented Nov 20, 2021

Yes.

@antroids antroids deleted the mqtt_vacuum_new branch November 20, 2021 11:56
@wborn wborn changed the title [MQTT.Homeassistant] make the mqtt.vacuum compliant [MQTT.Homeassistant] Make the mqtt.vacuum compliant Dec 18, 2021
NickWaterton pushed a commit to NickWaterton/openhab-addons that referenced this pull request Dec 30, 2021
…th the specification (openhab#11562)

Signed-off-by: Anton Kharuzhy <publicantroids@gmail.com>
Signed-off-by: Nick Waterton <n.waterton@outlook.com>
mischmidt83 pushed a commit to mischmidt83/openhab-addons that referenced this pull request Jan 9, 2022
…th the specification (openhab#11562)

Signed-off-by: Anton Kharuzhy <publicantroids@gmail.com>
Signed-off-by: Michael Schmidt <mi.schmidt.83@gmail.com>
nemerdaud pushed a commit to nemerdaud/openhab-addons that referenced this pull request Jan 28, 2022
…th the specification (openhab#11562)

Signed-off-by: Anton Kharuzhy <publicantroids@gmail.com>
marcfischerboschio pushed a commit to bosch-io/openhab-addons that referenced this pull request May 5, 2022
…th the specification (openhab#11562)

Signed-off-by: Anton Kharuzhy <publicantroids@gmail.com>
andan67 pushed a commit to andan67/openhab-addons that referenced this pull request Nov 6, 2022
…th the specification (openhab#11562)

Signed-off-by: Anton Kharuzhy <publicantroids@gmail.com>
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