search

skodaconnect / homeassistant-skodaconnect

Skoda Connect - A home assistant plugin to add integration with your car
Apache License 2.0
233 stars 27 forks source link

Missing entity value #123

Closed Depechie closed 2 years ago

Depechie commented 2 years ago

Describe the bug Not sure since when, but currently some entities are no longer showing values. Is this a known issue?

For example the Combined range now gives NaN. Last connected gives invalid time stamp.

gpuccio87 commented 2 years ago

Same problem for me. "Combined range" is unavailable

Farfar commented 2 years ago

I see that combined range is unavailable for me as well, will look into it . Last connected works fine here so should not be an issue with the integration I think. Maybe locale settings causing it?

Depechie commented 2 years ago

Could indeed be a locale thing, but used to work before ...

image

loeffelpan commented 2 years ago

Could indeed be a locale thing, but used to work before ...

image

This "invalid timestamp" issue isn't related to the other one you've mentioned. As I can see in your screenshot you have this issue on iOS. I would bet chromium based browsers aren't affected. See #113. I raised a pull request to fix this.

Depechie commented 2 years ago

Thx for the clarification @loeffelpan !!

gpuccio87 commented 2 years ago

Any news?

loeffelpan commented 2 years ago

@Farfar is on vacations. I took a look into it and it seens that the combined value is missing on the API response. I think we'll have to calculate the combined value from combustion and electric range.

I'll try to raise another PR during the weekend, so @Farfar can review it.

loeffelpan commented 2 years ago

Will be fixed as soon as @Farfar reviews my pull request (and acknowledge that it is ok).

Depechie commented 2 years ago

@loeffelpan seems something changed again? It was fixed, but now I got -1 again :/