Skip to content

Add HIGH_SPEED_WHEEL to color options#26

Merged
mdz merged 1 commit intomdz:mainfrom
swbradshaw:main
Mar 17, 2022
Merged

Add HIGH_SPEED_WHEEL to color options#26
mdz merged 1 commit intomdz:mainfrom
swbradshaw:main

Conversation

@swbradshaw
Copy link
Copy Markdown
Contributor

@swbradshaw swbradshaw commented Mar 16, 2022

When using my Sundance Chelsee hot tub on the multi-color option, I get an error in the HA logs KeyError: 'HIGH_SPEED_WHEEL'

Running this command python -m smarttub -u yyyy -p xxxx -vv info --lights show the following results (followed by the same KeyError.

DEBUG:smarttub.api:GET spas/#########/lights successful: {'lights': [{'zone': 1, 'mode': 'HIGH_SPEED_WHEEL', 'cycleSpeed': 2, 'color': {'red': 255, 'blue': 0, 'green': 0, 'white': 0}, 'intensity': 50, 'lastUpdated': '2022-03-16T03:23:29.121Z', 'createdBy': 'SYSTEM'}]}

@mdz mdz enabled auto-merge (squash) March 16, 2022 04:04
@mdz mdz disabled auto-merge March 17, 2022 15:37
@mdz mdz merged commit 3c41c9d into mdz:main Mar 17, 2022
@swbradshaw
Copy link
Copy Markdown
Contributor Author

Thanks for merging @mdz . What is the process to get HA fixed? Publish a new version here, and then do a PR on HA repo to point to the new version?

@mdz
Copy link
Copy Markdown
Owner

mdz commented Mar 18, 2022

Thanks for merging @mdz . What is the process to get HA fixed? Publish a new version here, and then do a PR on HA repo to point to the new version?

That's right.

@swbradshaw
Copy link
Copy Markdown
Contributor Author

@mdz Thanks. I created the PR there. home-assistant/core#68356

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