better_thermostat icon indicating copy to clipboard operation
better_thermostat copied to clipboard

Allow to change the real thermostat (climate entity)

Open janvda opened this issue 1 year ago • 11 comments

What is the feature?

Using the configure window you can change most thermostat settings but you can't change the real thermostat (climate entity). Here below a screenshot of the configure window:

image

So it would be nice to add also the possibility to change the real thermostat as you can do when creating a new better thermostat device (see screenshot below)

image

janvda avatar Feb 11 '24 21:02 janvda

#1087

wtom avatar Feb 11 '24 23:02 wtom

closed as it is indeed a duplicate of https://github.com/KartoffelToby/better_thermostat/discussions/1087

janvda avatar Feb 12 '24 07:02 janvda

reopened as https://github.com/KartoffelToby/better_thermostat/discussions/1087 is not a ticket but a discussion.

janvda avatar Feb 12 '24 07:02 janvda

Yes, this featuture (or bugfix?) would be great! What I today discovered: I have two TRVs from different brands, one needs the auto/heat swapped, the other not. Setting this separately is currently not possible when combined in one BT. I guess this may be closely related to this feature?

countrr avatar Feb 19 '24 21:02 countrr

The thing is, the entity is the heart of the configuration. If it changes, you can create a new one straight away.

Its not a bug. It how the config flow works

KartoffelToby avatar Feb 19 '24 22:02 KartoffelToby

@KartoffelToby What would you think of adding some text in the initial and edit dialogs? I think there could be some room for a short explanation, even if just to say "you cannot edit this later."

I can make a PR for it, shouldn't be too difficult :)

bassie1995 avatar Feb 20 '24 07:02 bassie1995

Understood, it’s just a pity that any surrounding automations also need to be updated, as the new BT will have a different name. But obviously this is not a life threatening problem :) About the swapping of auto/heat: could that be made separate for each TRV?

countrr avatar Feb 20 '24 07:02 countrr

@bassie1995 It's also easy to add the option for replacing the entity, but as I said, the question is what's the point if a completely new TVR can be created using the options anyway?

But if it's what many people want, I won't stand in the way. Feel free to make a PR :)

KartoffelToby avatar Feb 20 '24 07:02 KartoffelToby

@countrr yes, that is possible, I would be surprised if it wasn't already queried for each individual?

KartoffelToby avatar Feb 20 '24 07:02 KartoffelToby

@countrr yes, that is possible, I would be surprised if it wasn't already queried for each individual?

Sorry, yes, you are right, it can be set separately. I had not recognized that the same setting screen appears twice. I did not recognize the device ID shown in the top, as it is not the friendly name.

countrr avatar Feb 20 '24 13:02 countrr

Understood, it’s just a pity that any surrounding automations also need to be updated, as the new BT will have a different name. But obviously this is not a life threatening problem :)

If you set the same name as before, then BT will have the same name and the automations should work fine. I didn't it a couple of times. Also scheduler card works after recreation of the BT integration normally.

wtom avatar Feb 20 '24 17:02 wtom