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

HA Strava fails to startup with HA 2022.5 #45

Open
jouster1974 opened this issue May 5, 2022 · 12 comments · May be fixed by #46
Open

HA Strava fails to startup with HA 2022.5 #45

jouster1974 opened this issue May 5, 2022 · 12 comments · May be fixed by #46

Comments

@jouster1974
Copy link

HA Strava was all working fine on 2022.4.7....testing on 2022.5 and getting the following error on start

Setup failed for custom integration ha_strava: Unable to import component: cannot import name 'EVENT_TIME_CHANGED' from 'homeassistant.const' (/usr/src/homeassistant/homeassistant/const.py)

Any idea what might be causing the issue...have had a look about and cant see anyone else reporting

@jouster1974 jouster1974 changed the title HA Strava fails to sytartup with HA 2022.5 HA Strava fails to startup with HA 2022.5 May 5, 2022
@jartill
Copy link

jartill commented May 5, 2022

Yep. This is because there is a breaking change of the time_changed event being removed. Unfortunately the fix is beyond my limited skills

https://developers.home-assistant.io/blog/2022/04/20/saying-goodbye-to-event_time_changed/

@jouster1974
Copy link
Author

thanks for the update and for taking the time to comment...beyond mine too

@BertrumUK
Copy link

Same for me. I also had to change the manifest file to use voluptuous 0.13.1 to get to this error otherwise it threw a voluptuous error instead.

@BertrumUK
Copy link

I’ve got it working again for me by hashing out the offending line. I do get a camera error on a HA restart, but as I do not use the camera, it’s not an issue for me.

The line is I. The init_py file

CONF_WEBHOOK_ID, EVENT_COMPONENT_LOADED, EVENT_CORE_CONFIG_UPDATE, EVENT_HOMEASSISTANT_START, #EVENT_TIME_CHANGED,

@BertrumUK
Copy link

EF42BB6F-6AE3-4DB3-B4DC-818242A3FB89

@jouster1974
Copy link
Author

@BertrumUK you superstar...I did look through a couple of files for the EVENT_TIME_CHANGED but didn't check this one...now hashed out and working perfectly again..I also don't use images

@BertrumUK
Copy link

Hashing out line 20 in camera.py clears the error on a restart

) from hashlib import md5 #from homeassistant.const import EVENT_TIME_CHANGED

@jfmfreire
Copy link

jfmfreire commented May 6, 2022

i commented a line 28 in init.py, line 20 in camera.py and works for me.
image

@woolf0007
Copy link

i am wondering if this strava is so cumbersome, i set up the connection between strava and fitbit and fitbit is basic integration in HA... What do you think?

@bjornsivertsen
Copy link

i am wondering if this strava is so cumbersome, i set up the connection between strava and fitbit and fitbit is basic integration in HA... What do you think?

Does any level of information get lost in transit?

@youdroid youdroid linked a pull request May 9, 2022 that will close this issue
@youdroid
Copy link

youdroid commented May 9, 2022

I opened a PR (#46) to remove these two constants from both files.
Once accepted, everything will work fine again. 👍

@craibo
Copy link

craibo commented May 25, 2022

I have a working forked version of this here: https://github.com/craibo/ha_strava

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

8 participants