SteffeyDev / homebridge-hyundai-bluelink

Homebridge plugin for Hyundai BlueLink
Apache License 2.0
26 stars 13 forks source link

Disable status change via config, but enable only status info #15

Closed Hari-Seldon-nl closed 3 years ago

Hari-Seldon-nl commented 3 years ago

Is your feature request related to a problem? Please describe: No

Describe the solution you'd like: I would be good to have status and status changes separated. I'm very happy with the sensor info like doors closed or open. But would like to disable the possibility to change the status ('homekit closing the door') via the config.

Describe alternatives you've considered: None, only alternative is not using the plug-in, which is a shame because I realy like the connection with HomeKit

Additional context: I have a Tucson 2021 EU Model, and Bluelink plugin is working fine.

athal7 commented 3 years ago

@Hari-Seldon-nl are you thinking the HomeKit accessories would look the same, just interacting with them would do nothing? This is a pretty unfamiliar pattern as far as homebridge and HomeKit are concerned, so I am cautious about not creating a frustrating user experience.

Maybe you could tell me about the problem you're experiencing that is behind this request?

Hari-Seldon-nl commented 3 years ago

Hi Athal7, No, that is not what I meant. I would like to see a status icon when the doors are open or closed (just like with window/door sensors). The same for engine on or off. I could then use this in a Siri shortcut. Besides that (if so configured) separate icons to open the door or to start or stop the engine (just like with lamps). The reason is that I find the security of only userid and password very poor to make these kind of functions available in HomeKit, which every family member has access to. That is not safe within the family and certainly not externally (because of hacking). This way, you get many more satisfied users who can choose for themselves how far they want to go.

athal7 commented 3 years ago

Re status icons - this is already supported:

Re separate tiles, that is an option already available:

Re Siri, that's on the roadmap, thanks for the request!

Not sure I followed the part about how username/password are related here...

athal7 commented 3 years ago

I just checked, and was able to unlock the car using Siri. Going to close this issue as resolved, but happy to reopen if there's anything I've misunderstood.

Hari-Seldon-nl commented 3 years ago

Thanks for listening and providing the answer! Much appreciated!