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

Component breaks all HA sensors #5

Closed
danodemano opened this issue Apr 26, 2019 · 6 comments
Closed

Component breaks all HA sensors #5

danodemano opened this issue Apr 26, 2019 · 6 comments

Comments

@danodemano
Copy link

danodemano commented Apr 26, 2019

Tried to install and get this in the logs:

  File "/root/.homeassistant/custom_components/email/sensor.py", line 60
    _LOGGER.error(f'IMAPClient setup_platform error: {err}')
                                                          ^
SyntaxError: invalid syntax

Which not only broke itself but ALL my sensors in HA.

Running HA 0.91.4

@ljmerza
Copy link
Owner

ljmerza commented Apr 26, 2019

that's weird. f-strings has been supported since python 3.6 do you know what version you are on?

@danodemano
Copy link
Author

I'm on 3.5.3 which is still supported by HA. Guess we found the "problem" :)

@ljmerza
Copy link
Owner

ljmerza commented Apr 26, 2019

Ah ok i pushed the change to format()

@danodemano
Copy link
Author

Thanks! Another error though now:

  File "/root/.homeassistant/custom_components/email/sensor.py", line 120
    return f'email_{self.email_address}'
                                       ^
SyntaxError: invalid syntax

@ljmerza
Copy link
Owner

ljmerza commented Apr 26, 2019

ok fixed

@danodemano
Copy link
Author

Looks like that got it, thanks!

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

No branches or pull requests

2 participants