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

Add Shelly Plus PM Mini #848

Merged
merged 4 commits into from
Sep 25, 2023

Conversation

D1gitaldad
Copy link
Contributor

Adding the Shelly Plus PM Mini with its measured values

Add addPM1ToGen2Device for ShellyPlusPMMini
Add Shelly Plus PM Mini
@JuniperChris929
Copy link
Contributor

It's NOT shelly pm mini but shelly 1 pm mini - so this code needs to be fixed (see my original pull request)

@D1gitaldad
Copy link
Contributor Author

Hi , are you sure? I'm only talking about the Shelly Plus PM Mini, which according to the Shelly Knowledge Base does not have the switch or input components. The device name in the web interface is clearly shellypmmini-xxxxxxxxxxxx.

Attached is the link to KB: https://shelly-api-docs.shelly.cloud/gen2/Devices/ShellyPlusPMMini

@ntfrnd
Copy link

ntfrnd commented Sep 23, 2023

Hi, I would also like to see the new "Shelly Plus PM Mini" integrated into iobroker.
It is a very small device (gen2) with MQTT for current measurement.
It is super suitable for PV systems and for home automation.

See: https://kb.shelly.cloud/knowledge-base/shelly-plus-pm-mini

@JuniperChris929
Copy link
Contributor

You are right - there are 3 new shelly minis:
Plus PM Mini (just power meter, nothing else)
Plus 1 Mini (2000W Switch)
Plus 1 PM Mini (2000W Switch with integrated power meter)

Am I right, that you just integrated the power meter (mini) here?
If so, that would be golden! I created a pull request for just the Plus1mini and Plus1PMMini.
In theory, once the devs approve the pull requests we will have all mini-series devices supported :)

@D1gitaldad
Copy link
Contributor Author

That's exactly what it looks like it's about the power meter. As far as Matthias writes my code nicer incl. the changes suggested by you all new Mini devices would be included :)

For me, the code has been working very reliably for a week now and I get the values from the Shelly Plus PM Mini displayed free of charge in ioBroker.

@GermanBluefox
Copy link
Member

Thank you!

@GermanBluefox GermanBluefox merged commit 241f608 into iobroker-community-adapters:master Sep 25, 2023
1 of 8 checks passed
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.

4 participants