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

restore evu date attribute correctly to solve the TypeError: 'datetime.time' #283

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

s-kurz
Copy link
Contributor

@s-kurz s-kurz commented Oct 9, 2024

restore the evu dates attribute and solve the TypeError: 'datetime.time' issue
fixes #249
fixes #281

@Gifford47
Copy link

Can this PR already be merged?

@s-kurz
Copy link
Contributor Author

s-kurz commented Oct 31, 2024

added min firmware version for the current power consumption sensor in alignment with the energy sensors min firmware
fixes #277

@s-kurz s-kurz changed the title restore evu date attribute correctly restore evu date attribute correctly to solve the TypeError: 'datetime.time' Oct 31, 2024
@Gifford47
Copy link

In my case the attribute EVU_DAYS isn't set correctly. It's always "" (empty string). Does it works on your own?@s-kurz

@s-kurz
Copy link
Contributor Author

s-kurz commented Oct 31, 2024

@Gifford47 Its similar for me. I just fixed the issue that came from the attribute restoration. To test this part I manually change the attributes and then did a restart, which does restored them correctly. I never checked the logic of how the dates are actually set, nor am I sure if they should be set for my setup

@Gifford47
Copy link

Ok, thanks for the info!
I'll test it the next days

@Gifford47
Copy link

@s-kurz can you have a closer look why the attribute "SA.EVU_DAYS" isn't set correctly? I didn't find the bug yet...

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 this pull request may close these issues.

Update 2024-10-05-beta multiple sensors disappearing
2 participants