-
Notifications
You must be signed in to change notification settings - Fork 85
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[FEATURE] Move External BLE Sensor Selection to Settings #609
Comments
Some good suggestions, I'll look in to it. |
I'll remove the whole Faikin-auto from main page if a password is set. |
OK try next beta, I think I have addressed your concerns. |
Yeh we don't do a secondary poll for a name so has to be in the basic advertisement. Else MAC |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Daikin hardware
Faikin-S3-MINI-N4-R2: 9f65e5b 2024-11-30T08:32:55 S21
Is your feature request related to a problem? Please describe.
The web pages to control Faikin is exposed on my network, and also my guest network so users without access to home assistant can control the HVAC units. The BLE sensor selection is available on the homepage of the Faikin interface, not protected by the settings password. This allows any user on my network to choose the wrong temperature sensor on any HVAC page, i.e. the office HVAC could be pointed at the bedroom external sensor by a non admin user, resulting in the wrong temperatures being passed to the HVAC unit.
I struggled with this setting this up this initially - reading the docs was clear until the end where suddenly it just stops. I turned on BLE then was digging through settings trying to find where "The device will appear with its MAC address as its name." appeared. I assumed "BLE sensor ID" was supposed to change to a drop down. I wondered what ID meant in this context and ultimately didn't put the mac address in there.
I found the actual drop down by accident later on the homepage - then realised that this fills in the mac address in the ID field in settings.
Describe the solution you'd like
autob
toExternal BLE Sensor MAC Address (Without : delimiters)
autob
next toble
in settingsautob
with the mac address of the BLE deviceDescribe alternatives you've considered
Leave the BLE device dropdown on the homepage but make it read only. Make
autob
field the only way to set the value.Additional context
As an aside, the process of updating the LYWSD03MMC devices is considerably more convoluted since the recent default firmware update which only accepts signed OTA updates MI Home. It's still very doable but might want to tweak your summary slightly?
Also, the custom firmware only offers "BTHome2" in terms of BTHome. Might want to add that you support BTHome and BTHome2 in the docs (Everything seems to be working on BTHome2, but as the docs didn't say it I spent time testing)
The text was updated successfully, but these errors were encountered: