-
Notifications
You must be signed in to change notification settings - Fork 7
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
setting play time limit to not defined #76
Comments
Okay I found out, that every "wrong" value (e.g. 2:04:24) leads to not defined. |
Have to correct me, the app shows not defined but in really it's defined...
|
@JokerGermany yep, was going to say, the API will not allow higher than 6 hours. Will see if I can work on the outstanding issues today/tomorrow |
Next version will replace the time entity with a number entity that when set to -1 should set the play time limit to not defined... unfortunately the app seems to do some different things to what I was expecting so while the "support" will be there, it currently doesn't work as intended though. |
This has been implemented now and due to be released in 2024.5.0, apologies for the delay. |
Looks good, will implement it in my scripts in a couple of hours, thanks |
Checklist
Is your feature request related to a problem? Please describe.
When you want to delete the playtime limit, but want to keep the bedtime alarm, you need to set the playtime limit to not defined.
But It looks like you are not able in home assistant to set it to not defined
Describe the solution you'd like
If i am correct, the maximum time which you can set is 6 hours.
Perhaps an option would be if you set it to higher then 6 hours it is changed to "not defined"
Perhaps with a warning message?
Describe alternatives you've considered
A binary switch?
The text was updated successfully, but these errors were encountered: