fiLLLip / namron-homey

Athom Homey app repository to add support for Namron Z-wave devices
GNU General Public License v3.0
8 stars 7 forks source link

Support for LED Dimmer Z-Wave 200W #8

Closed chrmat closed 4 years ago

chrmat commented 4 years ago

Is it possible to add support for https://www.elektroimportoren.no/led-dimmer-z-wave-200w/1402756/Product.html

fiLLLip commented 4 years ago

Short answer; yes, it should be possible. Long answer; Since I don't have a Homey myself, it's hard for me to verify any new devices working. I would need screenshots of the device id once added as a basic device, like in the other issues reported here. You would also need to the testing for me, as I'm unable to. If that is ok, I think we should be able to fix it. It's harder with this device, as the upstream project has not added a similar device.

fiLLLip commented 4 years ago

Also, if you or someone who own this device (and a Homey) know how to code, pull requests are more than welcome!

storstad commented 4 years ago

Can’t help on the coding, but having installed 5 dimmers of the kind and using Homey, I’m happy to help & test.

storstad commented 4 years ago

0A0111A4-8453-4C07-8223-ADA74F0558CE B3F1546F-19CA-4E33-9EF8-D928B293E5D0

aasmundbo commented 4 years ago

I just installed my first of these LED dimmers and I can help out testing as well. I am interested in contributing, but I'm not a JS programmer nor familiar with how to develop/debug Homey apps, so figuring out that must come first. I'm probably a better tester than developer πŸ˜… .

fiLLLip commented 4 years ago

If you are able, please check out the feature branch: https://github.com/fiLLLip/namron-homey/tree/feat/rotary-dimmer Try to pull it down, have node and npm installed, run npm run run to test it, or something. At least that is the theory! Difficult to check when I don't own a Homey

fiLLLip commented 4 years ago

If you're unable, I can deploy it as a test version you can install in Homey. Just tell me :)

aasmundbo commented 4 years ago

I can try that branch as soon as possible. I had a stab at this myself and got it working just fine locally a few hours ago so we’re probably doing double work on this. Oh well πŸ˜…

aasmundbo commented 4 years ago

Basic functionality is working fine (adding to Homey, using it in flows etc.) πŸ˜„ Configuring the device isn't properly working (dimming speed, default brightness etc.). More details in the code review of the pull request.

I am trying to contribute with (in my mind) improved SVG files of the rotary dimmer, but I'm unable to push to the repo (Permission to fiLLLip/namron-homey.git denied). Either my git setup is wrong, or I don't have the permission, hoping the latter πŸ˜…

fiLLLip commented 4 years ago

Basic functionality is working fine (adding to Homey, using it in flows etc.) πŸ˜„ Configuring the device isn't properly working (dimming speed, default brightness etc.). More details in the code review of the pull request.

Nice! Will check it out :)

I am trying to contribute with (in my mind) improved SVG files of the rotary dimmer, but I'm unable to push to the repo (Permission to fiLLLip/namron-homey.git denied). Either my git setup is wrong, or I don't have the permission, hoping the latter πŸ˜…

Yeah, you don't have permission to the repo. If you fork it, you can create a PR to it.

aasmundbo commented 4 years ago

Yeah, you don't have permission to the repo. If you fork it, you can create a PR to it.

I have to fork the repo, make a PR there? If I'd like to contribute to your Namron repo -- is that the way to do it? No way to get access to this repo? Sorry if it's a dumb question, I'm not used to working the "github way" (if that's a thing :smile:)

fiLLLip commented 4 years ago

Yeah, you don't have permission to the repo. If you fork it, you can create a PR to it.

I have to fork the repo, make a PR there? If I'd like to contribute to your Namron repo -- is that the way to do it? No way to get access to this repo? Sorry if it's a dumb question, I'm not used to working the "github way" (if that's a thing πŸ˜„)

Of course I can add you as a contributer :)

fiLLLip commented 4 years ago

@chrmat @storstad @aasmundbo Please test it before I push it to production :) https://homey.app/no-no/app/net.filllip-namron/Namron/test/

aasmundbo commented 4 years ago

I’ll try ti test it tonight and get back to you.

storstad commented 4 years ago

In the mountains today, can test later tonight!

aasmundbo commented 4 years ago

I tested it briefly and it works fine.

I recommend to push it to production!

storstad commented 4 years ago

Tested yesterday, and can confirm that 4 / 5 dimmers paired just fine, and seems smooth operating πŸ‘

However, the 5th (the one initially added as a generic Z-wave-device to add the images in this thread) doesn't pair. Successfully removed from Homey several times, but every time it insist on not being recognised as a Namron dimmer, only to be added as a generic Z-wave πŸ€”

I guess that is a problem related to the dimmer or Homey, but could it also have something todo with how the Namron app adds/removes devices?

aasmundbo commented 4 years ago

@storstad have you tried factory resetting the failing dimmer? How to do it is described in the user manual, but I think it involves pressing and holding the reset button for 5 or 10 seconds.