Skip to content
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

All activity elapsed/moving time now showing in seconds #189

Open
jouster1974 opened this issue Dec 14, 2024 · 5 comments
Open

All activity elapsed/moving time now showing in seconds #189

jouster1974 opened this issue Dec 14, 2024 · 5 comments
Assignees
Labels
bug Something isn't working ha requirement Home Assistant core change forced requirement

Comments

@jouster1974
Copy link

jouster1974 commented Dec 14, 2024

Not 100% sure when this changed and things its been ok since 2024.12.1 but not something I appear to have the ability to change at the moment without possibly invoking some sort of template

image

image

not sure if that has anything to do with issue #188 and any Strava API change, especially with all their upcoming 3rd party changes

For clarity, just did the recent 2024.12.3 update and still the same

image

for clarity, the total move distances hasn't changed either and not added to that issue

@craibo
Copy link
Owner

craibo commented Dec 14, 2024

This would be a different issue to the API issue.

It will have been introduced by a core change.
The time has always been stored in seconds and HA was displaying this in a human readable format automagically.

I will see if there is a change I can make to force the human readable formatting again on that sensor

@craibo craibo self-assigned this Dec 18, 2024
@craibo craibo added bug Something isn't working ha requirement Home Assistant core change forced requirement labels Dec 18, 2024
@jouster1974
Copy link
Author

I'm sure you already have seen this but still the same in 2024.12.4

@craibo
Copy link
Owner

craibo commented Dec 19, 2024

Hi @jouster1974
I have opened a bug with HA core project home-assistant/frontend#23338 as the entities we have in ha_strava are configured correctly, according to the documentation, to show up in human readable format on the HA frontend.

@craibo
Copy link
Owner

craibo commented Dec 20, 2024

Hi @jouster1974

This change was made by @piitaya here home-assistant/frontend#23025. To help get this change rolled back or improved to use the human formatting by default, please add your comments to this PR and the bug home-assistant/frontend#23338.

This change has been made in a way in core that I cannot revert in the integration.

Thank you

@jouster1974
Copy link
Author

Hi @jouster1974

This change was made by @piitaya here home-assistant/frontend#23025. To help get this change rolled back or improved to use the human formatting by default, please add your comments to this PR and the bug home-assistant/frontend#23338.

This change has been made in a way in core that I cannot revert in the integration.

Thank you

this has been logged as requested.....for info, my YTD run and ride distance seems to be going backwards and now shows as almost zero for both and ALL distances are as before far away from where they should be

image

I assume this is still a Strava API issue (related to their API changes) but thought I would mention as I saw a new update today

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working ha requirement Home Assistant core change forced requirement
Projects
None yet
Development

No branches or pull requests

2 participants