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

Add repeat param to Roborock segment clean #1771

Merged
merged 1 commit into from
May 6, 2023

Conversation

MrBartusek
Copy link
Contributor

Hey! This is actuall fix for #1034, for some reason, this issue was closed with sketchy workarounds.

I've added repeat prop for segment_clean function in RoborockVacuum. This represents the count of repeats that the vacuum does in specified segments:

image

This is done as specified in XiaomiRobotVacuumProtocol. I've tested this with my Roborock S5 Max and it works well.

Copy link
Owner

@rytilahti rytilahti 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 PR, I think this is good to go (the linting checks are unrelated)!

The earlier format might be what earlier firmware versions used, but I think this is fine to merge. If someone is using this and we get bug reports about incorrect behavior, we can adjust the payload based on the version information.

@rytilahti rytilahti merged commit e15252c into rytilahti:master May 6, 2023
@MrBartusek
Copy link
Contributor Author

Thanks for the merge! I will probably be also adding support for that in Home Assistant after the next release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants