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

Possible Variable issue - just FYI #60

Closed
regularguy01 opened this issue Mar 30, 2023 · 5 comments
Closed

Possible Variable issue - just FYI #60

regularguy01 opened this issue Mar 30, 2023 · 5 comments

Comments

@regularguy01
Copy link

Looks like they changed the way the Var integration works?
enkama/hass-variables#35

Will this effect the Alerts history function?

@regularguy01
Copy link
Author

I see you already posed about it.

@finity69x2
Copy link
Owner

unfortunately, yes.

As you saw I'll lock my variables integration at the last version that uses the variables domain.

I may try out the newer variables version on my test setup so I can update the package for the newer version for those that have updated. But I can't give a timeframe for when I might get to it.

@regularguy01
Copy link
Author

unfortunately, yes.

As you saw I'll lock my variables integration at the last version that uses the variables domain.

I may try out the newer variables version on my test setup so I can update the package for the newer version for those that have updated. But I can't give a timeframe for when I might get to it.

If you like I can try and update and test it. Based on my reading they call it a sensor now instead of variable. You also might just try using input_text helpers instead.

@finity69x2
Copy link
Owner

Sorry, I've been pretty busy lately and haven't had a chance to even think about this issue yet.

but yes, I guess I should update the package to use the non-variable domain options.

I've always liked the variable integration for it's ability to have custom attributes that you could set via a service call. the new so-called variables (but not really now since it's just a new custom sensor) can still do that too so it's not a total loss.

I just need to find the time to decide which direction to go and then re-code the package to that standard.

@regularguy01
Copy link
Author

No worries.

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