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

roborock: Fix waterflow setting for Q7 Max+ #1646

Merged
merged 1 commit into from
Jan 4, 2023
Merged

roborock: Fix waterflow setting for Q7 Max+ #1646

merged 1 commit into from
Jan 4, 2023

Conversation

nijel
Copy link
Contributor

@nijel nijel commented Jan 3, 2023

It has additional Custom setting (207) and distance_off provides more fine-grained setting in this case what is not gracefully handled by this PR, as I will create an issue to discuss this.

@codecov-commenter
Copy link

codecov-commenter commented Jan 3, 2023

Codecov Report

Merging #1646 (549971c) into master (329df73) will increase coverage by 0.04%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1646      +/-   ##
==========================================
+ Coverage   80.36%   80.40%   +0.04%     
==========================================
  Files         161      161              
  Lines       15988    16016      +28     
  Branches     3607     3612       +5     
==========================================
+ Hits        12848    12878      +30     
+ Misses       2877     2875       -2     
  Partials      263      263              
Impacted Files Coverage Δ
.../integrations/vacuum/roborock/tests/test_vacuum.py 98.75% <100.00%> (+0.12%) ⬆️
miio/integrations/vacuum/roborock/vacuum.py 64.09% <100.00%> (+0.78%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

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.

Please add a test for this behavior so that we won't break it accidentally in the future.

@nijel
Copy link
Contributor Author

nijel commented Jan 4, 2023

Good idea, I've added the tests.

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.

LGTM, thanks again!

@rytilahti rytilahti changed the title vacuum: Fix waterflow setting for Q7 Max+ roborock: Fix waterflow setting for Q7 Max+ Jan 4, 2023
@rytilahti rytilahti merged commit 29c663a into rytilahti:master Jan 4, 2023
@nijel nijel deleted the q7-waterflow branch January 5, 2023 08:55
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.

3 participants