Skip to content

Update cli.py / device_manager.py to use get_home_data_v3 and discovered Roborock Q10 devices #418

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

lboue
Copy link

@lboue lboue commented Jul 17, 2025

Use get_home_data_v3 to discovered Roborock Q10 devices

I was able to discover my Q10 device after cloning this repo (36f5f2b) and changing to v3 :

(venv) user@vm:~/roborock/python-roborock/python-roborock$ python -m roborock.cli discover
Discovered devices Roborock Q10 S5+
(venv) user@vm:~/roborock/python-roborock/python-roborock$ python -m roborock.cli list-devices
Known devices Roborock Q10 S5+: 1simZp7ZguuihAICo*****

@lboue lboue changed the title Update cli.py to use get_home_data_v3 anddiscovered Roborock Q10 devices Update cli.py to use get_home_data_v3 and discovered Roborock Q10 devices Jul 17, 2025
@lboue lboue changed the title Update cli.py to use get_home_data_v3 and discovered Roborock Q10 devices Update cli.py / device_manager.py to use get_home_data_v3 and discovered Roborock Q10 devices Jul 17, 2025
@lboue lboue marked this pull request as ready for review July 17, 2025 16:04
@lboue lboue mentioned this pull request Jul 17, 2025
Copy link
Contributor

@allenporter allenporter left a comment

Choose a reason for hiding this comment

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

Two problems:
(1) Commit style is not matching the expected commit style. Please amend the last commit with feat: XXX and force push
(2) The tests are not passing with this change

@lboue
Copy link
Author

lboue commented Jul 18, 2025

Two problems:
(1) Commit style is not matching the expected commit style. Please amend the last commit with feat: XXX and force push

Thanks. I will check that.

(2) The tests are not passing with this change

Perhaps the version needs to be adapted to the device model.

@allenporter
Copy link
Contributor

(2) The tests are not passing with this change

Perhaps the version needs to be adapted to the device model.

It's because the test mocks out the get home data call and the call changed to be a different call. The test just needs to be updated to match the other call.

Use get_home_data_v3 to discovered Roborock Q10 devices
Update device_manager.py to use get_home_data_v3
@lboue lboue force-pushed the get_home_data_v3 branch from e017988 to 07f1530 Compare July 30, 2025 19:13
@lboue
Copy link
Author

lboue commented Jul 30, 2025

Two problems:

  • (1) Commit style is not matching the expected commit style. Please amend the last commit with feat: XXX and force push

Fixed now

(2) The tests are not passing with this change

I updated the tests.

@lboue lboue requested a review from allenporter July 30, 2025 19:44
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