-
Notifications
You must be signed in to change notification settings - Fork 714
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
Add door open/close state #3208
Conversation
add : Driver Front state Driver Rear state Passenger Front state Passenger Rear state
I believe what you have here should be sufficient for MQTT. But not tested. :-) Also see my #3087 where I added a field to MQTT, and tested it. |
If I am reading this correctly, this would publish the value as true or false. But I think it might be better to publish it as "open" or "closed" instead. |
I must admit that I do not know the language at all. but currently on MQTT there is a doors_open state which sends true or false if at least one of the 4 doors is open which is cool ! But i am developing a lovelace custom card for HA and i would like to retrieve the state for each door individually. I would be grateful to see these 4 states in teslamate natively I'm not sure I have the skills in this language to do it correctly |
Yes, it has been a annoyance for me too. But maybe not so annoying that I have created a PR to fix it. :-) On another note, I have a growing dislike the approach taken by teslamate of publishing each and every piece of data as a seperate mqtt topic. I guess it makes things simpler for systems like node-red, and maybe home assistant. But I tend to prefer if it json encoded data, and send it related data in groups. This means I don't have to listen to so many topics. For a good example of what I mean here, have a look at the way latitude and longitude are sent as two different topics. It doesn't make a lot of sense, because anything that will want to consume one value will also want to consume the other value. It could get confusing if you accidentally use the old latitude with the new longitude or vice versa. Another good example here would be to combine the door open/window/frunk/trunk open into one json encoded topic. Because typically it is unlikely you would only want one. This is something else I haven't got around to creating a PR for :-( and could be a different solution to this PR and my PR. But reluctant to much development with teslamate right now until I get some feedback on my PR. |
That is just the most used MQTT convention. Have topics with simple values. I do agree having just a publish of the actual raw JSON response as-is can be very useful! |
I think my modification will not be integrated into the original branch ? |
That really depends on how you deploy. Personally what I do is:
And then I tell my deploy to use |
May I have a suggestion? Your PR publishes topics like |
please update your branch |
I recently switched from Teslemetry, which used to provide these states in HA, but Teslamate doesn't yet. @tagcashdev are you still working on this? |
@SaswatPadhi Do feel free to take over this request, if you think you can help. |
Hi @brianmay, Since I don't have edit access to (@tagcashdev's branch to) update this PR, I instead opened a new PR. Please see #3962. |
solved by #3962 |
add :
Driver Front state
Driver Rear state
Passenger Front state
Passenger Rear state
PS : I don't know if my modifications are sufficient to be able to retrieve the status via MQTT ex. "teslamate/cars/1/passenger_front_state but that's the goal