Remove MQTT entity names that start with device name#140
Remove MQTT entity names that start with device name#140NewsGuyTor wants to merge 3 commits intodevWaves:mainfrom
Conversation
This should make it function better in Home Assistant
Capusjon
left a comment
There was a problem hiding this comment.
you forgot to change the version on line 565.
static const String versionNum = "v7.1.5";
thats what I would suggest, it makes it so that we can see in the webinterface that the right version is uploaded.
|
After implementing said changes almost all the warnings are gone. Only one that remains for me is:
changing line 3371 to
should fix the warning, only issue I have with this fix is that the data produced by the Switchbot for Illuminance isn't in lux. So technically it's not the best solution, might need to use an other topic instead of illuminance. But for now this suits me and the warnings are gone. @NewsGuyTor thanx for your part of the fix. If you could add in my previous comment that would be great, for this comment do with it what you want.. |
|
@Capusjon : Fixed |
This should make it function better in Home Assistant