Closed Dunkelschunkel closed 1 year ago
The image isn't cropped to the device boundries.
So I went ahead and changed the ones you copied from (I didn't realise they weren't cropped properly). I will do yours for you as it's fair to say you were just going by whats already in the repo
@Sian-Lee-SA thank you
@Sian-Lee-SA should I add the other, smaller aqara switches for zigbee2mqtt as well? I do not own any other than the 3x2 one but I assume they are similar.
Just confirm with the zigbee2mqtt docs for the devices and what they have in their action section then I'll merge if all matches etc. Generally I would say no because I don't want the headaches of issues popping up about non working blueprints because it wasn't tested properly etc. But I'll let this one slide as they should be similar.
I'll check everything with the documentation at zigbee2mqtt and the blueprints from the deconz
Jan 26, 2023 10:56:46 AM Sian @.***>:
Just confirm with the zigbee2mqtt docs for the devices and what they have in their action sectio then I'll merge. Generally I would say no because I don't want the headaches of issues popping up about non working blueprints because it wasn't tested properly etc. But I'll let this one slide as they should be similar.
— Reply to this email directly, view it on GitHub[https://github.com/Sian-Lee-SA/Home-Assistant-Switch-Manager/pull/87#issuecomment-1404773783], or unsubscribe[https://github.com/notifications/unsubscribe-auth/AT5ZC5IBKWIRIAXP6POTFFTWUJC53ANCNFSM6AAAAAAUGYCFO4]. You are receiving this because you authored the thread.[Tracking image][https://github.com/notifications/beacon/AT5ZC5O2SZRQQKKLK3JYAZ3WUJC53A5CNFSM6AAAAAAUGYCFO6WGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTSTXMSZO.gif]
Also make sure you copy from the updated files in this repo as they're cropped and repositioned but not in a release yet
Tested locally with my on device @Sian-Lee-SA: I also checked the docs from zigbee2mqtt Aqara Opple 3x2 an it seems to work just fine.
Everything is based on the deconz blueprint for the same device as mentioned in issue 78
Blueprint Checklist