Yes that will be fine with the Devices widget ![]()
[quote=« _Will_71, post:20, topic:7315 »]
@pierre-gilles, did you plan to apply this dev to the Room Device widget
Yes that will be fine with the Devices widget ![]()
[quote=« _Will_71, post:20, topic:7315 »]
@pierre-gilles, did you plan to apply this dev to the Room Device widget
For me you can deprecate it, the Devices widget does the same job with more features.
And there’s double work in case of modification
Indeed, if the device widget provides the same thing as the room device — and with added features — then you might as well deprecate the room device, which becomes redundant and confusing!
No problem as far as I’m concerned, but… Will the migration from one to the other be able to proceed without a hitch?
I’m in favor of deprecating it too.
Cool, what I’m going to do then is deploy this new Devices widget in the next version of Gladys, while keeping for now the « Room Devices » widget.
Depending on feedback, as soon as everyone is convinced that the « Devices » widget works as expected, I’ll do another release to migrate everyone from one to the other ![]()
Well, however I’m on summer vacation starting tonight so it won’t be deployed today otherwise there will be no one left for bug fixes, and it’s imperative that Gladys remains stable over the summer.
Yes, without any problems ![]()
Following feedback from @AlexTrovato and @Lokkye on GitHub, I pushed fixes and made a new Docker build ![]()
Still on the same tag:
\r\ngladysassistant/gladys:device-widget-rename-reorder
Open to feedback, and if it’s good for everyone, I’ll deploy this in the next Gladys release ![]()
Hi everyone, have any of you been able to test the Docker image or the PR?
@Lokkye @AlexTrovato FYI I had addressed your feedback before the holidays!
This feature is now available in Gladys Assistant 4.27!