Skip to content

Added support for Shelly Plus 1 Mini#88

Open
sw-koenig wants to merge 2 commits into
alexryd:mainfrom
sw-koenig:plus-1-mini
Open

Added support for Shelly Plus 1 Mini#88
sw-koenig wants to merge 2 commits into
alexryd:mainfrom
sw-koenig:plus-1-mini

Conversation

@sw-koenig

@sw-koenig sw-koenig commented Oct 2, 2023

Copy link
Copy Markdown

Resolves #87

Dependency
As the model IDs for the devices are defined in node-shellies-ng, the following Pull-Request must be merged first:
alexryd/node-shellies-ng#7

Shelly has released "mini" versions for some of their new generation products. They basically have the same functions as the normal ones, but using different model IDs.

Change has been tested successfull in my local setup with the homebridge-shelly-ng plugin.
Best regards,
Swen

@lukyer

lukyer commented Oct 28, 2023

Copy link
Copy Markdown

Great job! Waiting for this to get my new Minis in homebridge. @alexryd would you find some time to merge this and deploy new version of shelly-ng? 🙏

@djcargo

djcargo commented Oct 31, 2023

Copy link
Copy Markdown

I will be also very happy to get Minis support :)

@jkdeleteme

Copy link
Copy Markdown

great! how to install still fork?

@holetse

holetse commented Dec 2, 2023

Copy link
Copy Markdown

Hmm, I am encountering an issue with this PR.

Specifically, the upstream changes to node-shellies-ng seems to have been made off of this commit (HEAD of main at fork time), and thus contains this commit which came after the 1.5.1 release. The change to DeviceDiscoverer is not compatible with homebridge-shelly-ng, giving a bunch of typescript errors around the change from an abstract class to an interface, e.g.:

src/platform.ts:39:45 - error TS2689: Cannot extend an interface 'DeviceDiscoverer'. Did you mean 'implements'?
export class ConfigDeviceDiscoverer extends DeviceDiscoverer {

It seems like this PR needs to address that change to the DeviceDiscoverer, or node-shellies-ng additions need to be made off of this commit.

@sw-koenig Thoughts?

@sw-koenig

Copy link
Copy Markdown
Author

Hi @holetse,
you're probably right. I'm not an expert for Typescript and thus did not looked into this in detail. I could only test the changes in my local homebridge setup, by directly modifying the .js files, which perfectly works for me. I couldn't managed to build the projects locally and use the local builds in homebridge. So maybe, there are additional changes needed, to be compatible with that change to DeviceDiscover in node-shellies-ng.

Best regards,
Swen

@jtowe1

jtowe1 commented Mar 1, 2024

Copy link
Copy Markdown

Yeah it looks like there are commits in node-shellies-ng to main after the v1.5.1 release (here https://github.com/alexryd/node-shellies-ng/releases/tag/v1.5.1 it says 6 commits to main since this release) and homebridge-shelly-ng uses v1.5.1. Seems like maybe a new release of node-shellies-ng needs to be cut and then a new version of homebridge-shelly-ng needs to be made using the updated node-shellies-ng.

@koszta5

koszta5 commented Dec 31, 2025

Copy link
Copy Markdown

If anybody (@jakubkasparek ) wants to enable support for this on their homebridge:

  • Install the vaniala homebridge-shelly-ng plugin
  • manually edited the following files: $HOMEBRIDGE_HOME/node_modules/homebridge-shelly-ng/node_modules/shellies-ng/dist/devices --> shelly-plus-1.js and shelly-plus-1-pm.js
  • base your edits on Added support for Shelly Plus 1 Mini node-shellies-ng#7 BUT in my case I bought (https://www.alza.cz/shelly-plus-1pm-mini-spinaci-modul-wifi-gen3-d8636605.htm) and to enable it I had to change "SNSW-001P8EU" to "S3SW-001P8EU" to get it recognised and working
  • additionally you need to edit $HOMEBRIDGE_HOME/node_modules/homebridge-shelly-ng/dist/device-delegates --> shelly-plus-1-pm.js and shelly-plus-1.js (edits like this PR suggests)
  • I got it all working :) all of my shellies show nicely in my home app :)

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.

[new device] Shelly Plus 1 Mini

8 participants