Will_71
January 24, 2024, 7:38pm
21
Hi @Tlse-vins , have you tested the new image?
Hi @_Will_71 , I just tested and it’s fine, the buttons are working properly.
On, OFF, long press on both adjustment buttons.
Will_71
January 24, 2024, 9:17pm
23
And does the press duration appear?
So should I leave all the button states?
The press duration is not displayed. And I don’t see how to use it.
you can remove the states:
ON button released
OFF button released
Plus button held then released
Minus button held then released
Plus button released
Minus button released
What’s good is that we can use this remote to control other devices too; I tested it on outlets and it works.
Will_71
January 24, 2024, 10:12pm
25
Okay, I’ll take care of the actions tomorrow.
Regarding the timing, can you delete your Gladys remote, save it again? And retest?
That’s what I had done; I’ll do it again, but not before Sunday if I have time.
Will_71
January 26, 2024, 12:09pm
27
Ok no problem. That gives me time to remove the release actions.
Will_71
January 26, 2024, 11:17pm
28
@Tlse-vins , FYI the image is being updated with the removed actions
Everything works perfectly.
For now Gladys doesn’t handle the duration of a button press, so it’s not possible to dim a bulb gradually.
For the + button and the - button in scenes, I specified a value.
1 Like
Will_71
January 28, 2024, 6:59pm
30
Okay, I’ll make the pull request on GitHub.
Will_71
January 28, 2024, 10:16pm
31
@pierre-gilles , here is the PR
master ← William-De71:add-Philips-324131092621
ouvert 10:15PM - 28 Jan 24 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:
- [ ] 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.
- [ ] 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).
NOTE: these things are not required to open a PR and can be done afterwards / while the PR is open.
### Description of change
Add philips hue dimmer switch 324131092621
[Philips Hue Dimmer Switch](https://community.gladysassistant.com/t/zigbee2mqtt-ajout-telecommande-philips-hue-dimmer-switch/8675)
2 Likes
@_Will_71 Looks good to me
It’s merged and will be included in the next Gladys release
4 Likes