Skip to content

Add an extra line for Shelly Plus 2PM (new model name)#376

Merged
thecode merged 2 commits intohome-assistant-libs:mainfrom
emilbratt:new-models
Apr 18, 2023
Merged

Add an extra line for Shelly Plus 2PM (new model name)#376
thecode merged 2 commits intohome-assistant-libs:mainfrom
emilbratt:new-models

Conversation

@emilbratt
Copy link
Copy Markdown
Contributor

@emilbratt emilbratt commented Apr 13, 2023

My new Shelly Plus 2PM`s model name is not recognised.
I found that there exist an entry for my model, but with model name SNSW-002P16EU.
My new shelly device has the model name SNSW-102P16EU.
Im short. I added a 2nd entry to const.py with the new model name and a trailing comment.

Everything else works as expected. The shelly status is printed.

Below is some of the output from running example.py before.

** shelly_alias - Unknown (SNSW-102P16EU)  @ 192.168.1.10 **

Below is some of the output from running example.py after.

** shelly_alias - Shelly Plus 2PM  @ 192.168.1.10 **

Comment thread aioshelly/const.py Outdated
Comment thread aioshelly/const.py Outdated
Co-authored-by: Shay Levy <levyshay1@gmail.com>
Copy link
Copy Markdown
Collaborator

@bieniu bieniu 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 @emilbratt

Copy link
Copy Markdown
Contributor

@thecode thecode left a comment

Choose a reason for hiding this comment

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

Thanks!

@thecode thecode merged commit e9883a2 into home-assistant-libs:main Apr 18, 2023
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.

3 participants