Allow changing the display order of devices in a box

Yes that will be fine with the Devices widget :slight_smile:

[quote=« _Will_71, post:20, topic:7315 »]
@pierre-gilles, did you plan to apply this dev to the Room Device widget

2 Likes

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

1 Like

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!

1 Like

No problem as far as I’m concerned, but… Will the migration from one to the other be able to proceed without a hitch?

1 Like

I’m in favor of deprecating it too.

2 Likes

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 :slight_smile:

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 :slight_smile:

4 Likes

Following feedback from @AlexTrovato and @Lokkye on GitHub, I pushed fixes and made a new Docker build :slight_smile:

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 :slight_smile:

2 Likes

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!