MQTT device creation - semantic improvement

Feature requests
1

Hello

This is a purely aesthetic feature but it seems clearer to me for those who, like me, have a slow brain and aren’t very ‹ with it › :rofl:.

When creating MQTT devices I find it would be easier to understand to

2

Hello,

There are two requirements in one request (it’s better to create a request for your « copy » button) :face_exhaling:

I don’t see the added value of the first

3

Hello @spenceur,

Well, I find that it highlights and emphasizes the « Gladys to the outside » meaning more in the case I’m proposing! The first one, I find, seems more ambiguous regarding the direction of listening and publishing…

I admit I would do it, but I don’t really know how to do it :woozy_face: if you could give me some pointers for that I’d be grateful! :wink: It could help me for other cases too! :blush:

4
5

Thanks @spenceur, but what do I do next? I’m a beginner in this area on GitHub… :disappointed_relieved:

6

Look at this tutorial — you should be able to do it.

7

Thanks @Tlse-vins, I’ll take a look at that!

8

It’s impossible to use the copy/paste API on a non-HTTPS site. Possible on Gladys Plus but not locally, so this feature was removed (it existed in the past). We could put it back on Gladys Plus :slight_smile:

For the changes to the titles, the change isn’t obvious to me. The text below already explains well what it does, doesn’t it?

9

Too bad about the copy-paste :cry:

Yes, but I find the implicit meaning is missing, maybe add the meaning so that it’s instantly clearer?
MQTT topic to publish => MQTT topic to publish to Gladys
MQTT topic to listen to => MQTT topic to listen to from Gladys
As it stands, I often tend to copy the wrong topic :roll_eyes:
I don’t know if I’m the only one in this case… :slight_smile:

10

OK I understand, no problem making that more explicit

1 Like
11

@cce66 I’ve left feedback on your PR:

12

@cce66 You made quite a few PRs over the past few weeks, I left feedback on those PRs but without any response from you I don’t know if you saw them? :slight_smile:

13

Ah yes, I saw it then I had a solar panel failure just after and I zoned out… my memory is failing! Ohhhh dear :rofl: I’ll check now — I’m thinking about it this PM and I’ll look at it tonight with a clear head :+1:

14

@pierre-gilles I made the PRs and fixed one or two extra things — tell me if it’s OK or if I still have something to do (not comfortable with GitHub :upside_down_face:)

15

I’ve left feedback on this PR:

As for the others, which PR are you talking about? Can you put a link here?

16

Excuse me, no others for the moment, I was only talking about that one!