Hi everyone!
I talked with @_Will_71 and @Brisou , both want to contribute to a potential Z-Wave JS UI integration, so that’s super positive
That said, the first step to develop an integration (setting up the whole project, UI, tests, etc…) is a big step, so I offered my help by doing a live stream like I did for Sonos. Then I’ll pass the baton to whoever wants to add devices little by little, a bit like we do on Zigbee2mqtt!
So I propose, this Sunday, December 17 at 10:00 , a YouTube live to code together in one day a first MVP of a Z-Wave JS UI integration!
Follow on YouTube here:
VIDEO
On my end, I equipped myself with hardware this week with this duo:
The idea will of course be to make a minimal integration; I won’t be too greedy so as not to repeat the mistakes of that integration
Then, after this development, I’ll launch the integration with a single device (this Fibaro sensor), and afterward the work will be to add support for all compatibilities.
Thanks to everyone who will follow the live — I’d welcome your help in the comments because I’m not very familiar with the Z-Wave protocol nor with Z-Wave JS UI.
9 Likes
guim31
December 15, 2023, 4:50pm
22
I think you’ve got something really interesting with those « first step » live streams.
2 Likes
A three-person livestream?
1 Like
For sure, I didn’t expect that! All the better if it helps to « kick off development »
No, just me!
1 Like
He tried but I’m not available this weekend and I’m not at the same level as @pierre-gilles
3 Likes
cce66
December 15, 2023, 8:49pm
26
Great, Sunday 10am… oh well I’m not a fan of mass so I’ll be listening in and since it’s Sunday I’ll bring the croissants!
3 Likes
If we can get on the same page, I’m happy to get my hands dirty again
4 Likes
Thanks to everyone who came to watch this live
The bet was met again, with a functional integration, complete PR with tests, translations, assets and Docker image, in 7h30 !!
The list of integrations with Z-Wave :
The configuration page that allows adding an external MQTT broker :
The « Discovery » page that lets you see detected Z-Wave devices :
The « Devices » page that lets you see Z-Wave devices added to Gladys :
The dashboard with a Z-Wave sensor added
The replay of the live stream is available here :
VIDEO
The PR is on GitHub (would appreciate your review!):
master ← zwavejs-ui-integration
ouvert 04:06PM - 17 Dec 23 UTC
### Pull Request check-list
To ensure your Pull Request can be accepted as fa… st as possible, make sure to review and check all of these items:
- [x] If your changes affects code, did your write the tests?
- [x] Are tests passing? (`npm test` on both front/server)
- [x] Is the linter passing? (`npm run eslint` on both front/server)
- [x] Did you run prettier? (`npm run prettier` on both front/server)
- [x] If you are adding a new features/services, did you run integration comparator? (`npm run compare-translations` on front)
- [x] Did you test this pull request in real life? With real devices? If this development is a big feature or a new service, we recommend that you provide a Docker image to the community ([french forum](https://community.gladysassistant.com/)/[english forum](https://en-community.gladysassistant.com/)) for testing before merging.
- [x] If your changes modify the API (REST or Node.js), did you modify the API documentation? (Documentation is based on comments in code)
- [ ] If you are adding a new features/services which needs explanation, did you modify the user documentation? See [the GitHub repo](https://github.com/GladysAssistant/v4-website) and the [website](https://gladysassistant.com).
- [ ] Did you add fake requests data for the demo mode (`front/src/config/demo.js`) so that the demo website is working without a backend? (if needed) See [https://demo.gladysassistant.com](https://demo.gladysassistant.com).
### Description of change
Add a new Z-Wave JS UI integration.
A test Docker image is being built with the tag :
gladysassistant/gladys:zwavejs-ui-integration
(Available in about 30 minutes, around 6:20 PM or so)
Thanks to everyone who was there, I hope it was interesting and that it gave you motivation to contribute to Gladys!
6 Likes
pierre-gilles:
at 7:30 !!
That’s definitely the time to remember ^^@pierre-gilles , couldn’t follow everything but that will make a good dynamic and sound wallpaper for my next week
[quote=« pierre-gilles, post:28, topic:8573 »]
Correction: « […] re-pair the device in Gladys. » or « […] pair the device again in Gladys. »
I didn’t catch the live stream, but well done.
Is this integration intended to be merged as-is?
Thanks for the feedback @Terdious I’ll fix it!
Thanks @_Will_71
Yes! I’d be happy to get your review
The information saved in the MQTT configuration is not displayed when you return to the configuration page.
My devices don’t have a « location » but are still discovered
Otherwise I can’t go any further, I don’t have any compatible devices at the moment
Oh really? I can’t reproduce it!
Ah! I’ll change that
Yet I do have devices in the « Discovery » tab.
Ah, I think I understand, I was wrong — I didn’t scope the variables by service in the front-end!! my bad, I’ll fix it!
cce66
December 18, 2023, 8:43am
39
localisation rather than location
No, as said in the live stream, the term on the Z-Wave JS UI side is « location » and not localisation, so I prefer to keep their jargon so that the user understands where it comes from
1 Like
