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

setting play time limit to not defined #76

Closed
3 of 4 tasks
JokerGermany opened this issue Jan 27, 2024 · 6 comments
Closed
3 of 4 tasks

setting play time limit to not defined #76

JokerGermany opened this issue Jan 27, 2024 · 6 comments
Assignees
Labels
enhancement New feature or request upstream Issues with REST API

Comments

@JokerGermany
Copy link

JokerGermany commented Jan 27, 2024

Checklist

  • I have filled out the template to the best of my ability.
  • This only contains 1 feature request (if you have multiple feature requests, open one feature request for each feature request).
  • This issue is not a duplicate feature request of previous feature requests.
  • This issue is a feature request to test other Nintendo Parental Controls mobile app configurations

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?

@pantherale0 pantherale0 self-assigned this Jan 28, 2024
@pantherale0 pantherale0 added enhancement New feature or request upstream Issues with REST API labels Jan 28, 2024
@JokerGermany
Copy link
Author

JokerGermany commented Feb 14, 2024

Okay I found out, that every "wrong" value (e.g. 2:04:24) leads to not defined.
Which is a little bit problematic for my automations, but good for using it to set not defined XD

@JokerGermany
Copy link
Author

JokerGermany commented Feb 18, 2024

Okay I found out, that every "wrong" value (e.g. 2:04:24) leads to not defined. Which is a little bit problematic for my automations, but good for using it to set not defined XD

Have to correct me, the app shows not defined but in really it's defined...

@pantherale0 Can you give me a Hint what code i have to change to be able to set more than 6 hours?
Looks like the API don't accept values higher than 6 hours :(

@pantherale0
Copy link
Owner

pantherale0 commented Feb 18, 2024

@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

@pantherale0
Copy link
Owner

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.

@pantherale0
Copy link
Owner

This has been implemented now and due to be released in 2024.5.0, apologies for the delay.

@JokerGermany
Copy link
Author

Looks good, will implement it in my scripts in a couple of hours, thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request upstream Issues with REST API
Projects
None yet
Development

No branches or pull requests

2 participants