[Need you] A new dashboard display of device names for complex devices?

I tried it on the Dashboard and the problem is partially resolved. Actually it all depends on the configuration.

For my part I have a SONOFF dual relay flashed with Tasmota.

On it I have my water heater and a light. So I configured an outlet and a light.

In that case the defined names are not taken into account.

In the Dashboard I have the device name for both contacts

But if I change it and, instead of the lamp, define an outlet, then the Dashboard display is correct.

Normally only identical features are considered.

Mmm indeed, because the code only applies this rule to « complex » devices that have multiple features of the same type, whereas here you change the type and so the display considers them as two different features, therefore distinguishable in the UI!

How to do it? :thinking: If anyone has an idea, I have no answer at all.

Ok at first I didn’t understand why it worked for @Psoy and not for me.
And it was when I saw the configuration of my relay that I understood. Hence my message to point it out

After that I absolutely don’t mind configuring the lighting as an outlet!

Should we take only the type and not the category?

[quote=« _Will_71, post:44, topic:7559 »]
Besides, it absolutely doesn’t bother me to set the lighting as an

[quote="lmilcent

I’ve made the change — we can check if anyone has any objections.
I’ve also tried to fix the charts.

@pierre-gilles the ball is in your court.

@AlexTrovato The PR doesn’t seem to be passing:

@AlexTrovato I added an automatic build for PRs on Cloudflare Pages, so from now on each PR will have a comment with a Gladys Plus test link :slight_smile:

Don’t hesitate to share the link here as soon as you have a working version!

3 Likes

That’s going to be really handy!

1 Like

Indeed, I hadn’t taken the thermostat’s new box into account.
I’ve just fixed it (rebased everything from master).

@pierre-gilles I can’t find the comment with the link :confused:
It shouldn’t be retroactive on PRs that are already open.

@AlexTrovato Indeed, I think it’s because it’s a PR from a fork — they don’t run PRs so that someone could see the secrets or use my Cloudflare builds on my account

Alright, I’ll run the build myself and post the URL here!

The build is available here: https://7b22b8fd.gladys-plus.pages.dev/

Hello there,

Regarding the graphs, they look fine on my end:

And everything else is still working.

Thanks @AlexTrovato and @pierre-gilles

(I don’t have the screenshot but it’s also all good when clicking on the graph)

1 Like

Thanks for the quick response so early in the morning @Terdious :folded_hands:

And thanks for the PR @AlexTrovato!

I’ll wait until tomorrow to see the other feedback, and if it’s good it’ll go to production

2 Likes

Works for me, you can merge.

1 Like

Hello,
The curves are fine with me as well

1 Like

Ok, same here.

2 Likes