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

Qrevo Curv Code Mappings #133846

Open
jhemak opened this issue Dec 22, 2024 · 2 comments
Open

Qrevo Curv Code Mappings #133846

jhemak opened this issue Dec 22, 2024 · 2 comments

Comments

@jhemak
Copy link

jhemak commented Dec 22, 2024

The problem

Hello! Thanks for creating and maintaining this integration!

Every one in a while I receive an error about a missing mapping. A few examples:

Missing RoborockMopModeS7 code: 304 - defaulting to 300
Missing RoborockDockTypeCode code: 17 - defaulting to 'unknown'
Missing RoborockFinishReason code: 2 - defaulting to 21

Is there something I can do to obtain all the codes for this model so they can be added to the mapping logic?

What version of Home Assistant Core has the issue?

core-2024.12.5

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant OS

Integration causing the issue

Roborock

Link to integration documentation on our website

https://www.home-assistant.io/integrations/roborock/

Diagnostics information

No response

Example YAML snippet

No response

Anything in the logs that might be useful for us?

No response

Additional information

No response

@home-assistant
Copy link

Hey there @Lash-L, mind taking a look at this issue as it has been labeled with an integration (roborock) you are listed as a code owner for? Thanks!

Code owner commands

Code owners of roborock can trigger bot actions by commenting:

  • @home-assistant close Closes the issue.
  • @home-assistant rename Awesome new title Renames the issue.
  • @home-assistant reopen Reopen the issue.
  • @home-assistant unassign roborock Removes the current integration label and assignees on the issue, add the integration domain after the command.
  • @home-assistant add-label needs-more-information Add a label (needs-more-information, problem in dependency, problem in custom component) to the issue.
  • @home-assistant remove-label needs-more-information Remove a label (needs-more-information, problem in dependency, problem in custom component) on the issue.

(message by CodeOwnersMention)


roborock documentation
roborock source
(message by IssueLinks)

@Lash-L
Copy link
Contributor

Lash-L commented Dec 25, 2024

Hi!

There is an outstanding pr for this in our base library

humbertogontijo/python-roborock#253

Once that is in this should be fixed

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

No branches or pull requests

2 participants