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

Bonus time is not correctly added to play time limit #81

Closed
5 of 7 tasks
MrIcemanLE opened this issue Feb 6, 2024 · 2 comments
Closed
5 of 7 tasks

Bonus time is not correctly added to play time limit #81

MrIcemanLE opened this issue Feb 6, 2024 · 2 comments

Comments

@MrIcemanLE
Copy link

System Health details

System Information

version core-2024.1.5
installation_type Home Assistant OS
dev false
hassio true
docker true
user root
virtualenv false
python_version 3.11.6
os_name Linux
os_version 6.1.63-haos-raspi
arch aarch64
timezone Europe/Berlin
config_dir /config
Home Assistant Community Store
GitHub API ok
GitHub Content ok
GitHub Web ok
GitHub API Calls Remaining 5000
Installed Version 1.34.0
Stage running
Available Repositories 1390
Downloaded Repositories 34
Home Assistant Cloud
logged_in true
subscription_expiration 9. Juni 2024 um 02:00
relayer_connected true
relayer_region eu-central-1
remote_enabled true
remote_connected true
alexa_enabled false
google_enabled false
remote_server eu-central-1-6.ui.nabu.casa
certificate_status ready
instance_id cae2a01e424f4ababacf9365cc72b7ae
can_reach_cert_server ok
can_reach_cloud_auth ok
can_reach_cloud ok
Home Assistant Supervisor
host_os Home Assistant OS 11.4
update_channel stable
supervisor_version supervisor-2024.01.1
agent_version 1.6.0
docker_version 24.0.7
disk_total 234.0 GB
disk_used 92.7 GB
healthy true
supported true
board rpi4-64
supervisor_api ok
version_api ok
installed_addons Mosquitto broker (6.4.0), Studio Code Server (5.15.0), Node-RED (17.0.4), Terminal & SSH (9.8.1), Samba share (12.2.0), TasmoAdmin (0.29.1), PS5 MQTT (1.3.1), ESPHome (2023.12.9), Zigbee2MQTT (1.35.2-1), File editor (5.7.0), Vcontrol add-on (1.10.1)
Husqvarna Automower
can_reach_server ok
Dashboards
dashboards 2
resources 12
views 22
mode storage
Recorder
oldest_recorder_run 29. Januar 2024 um 09:34
current_recorder_run 5. Februar 2024 um 14:52
estimated_db_size 2051.04 MiB
database_engine sqlite
database_version 3.41.2

Checklist

  • I have enabled debug logging for my installation.
  • I have filled out the issue template to the best of my ability.
  • This issue only contains 1 issue (if you have multiple issues, open one issue for each issue).
  • This issue is not a duplicate issue of currently previous issues..

Nintendo Parental Controls mobile app

  • I have checked the configuration of the mobile app and the switch device is registered.
  • I experience this issue if I test again with the tested configuration in the readme.
  • This issue does not happen when tested with the tested configuration in the readme.

Describe the issue

When i set the bonus time to 1h (1:00) the play time limit gets added 2h of playtime. After reducing the bonus time back to zero the playtime changes to a different value (sometimes the prior defined bonus time is added to the play time).

Reproduction steps

  1. restart of home assistant (play time limit 3:15), bonus time (0:00) and bedtime alarm (unknown) --> corresponding to the app
  2. change the bonus time by typing a 30 in the minutes field
  3. play time is set to 4:15

Debug logs

2024-02-06 10:35:22.731 DEBUG (MainThread) [custom_components.nintendo_parental] Finished fetching nintendo_parental data in 17.392 seconds (success: True)
2024-02-06 10:35:35.101 DEBUG (MainThread) [custom_components.nintendo_parental] Finished fetching nintendo_parental data in 1.695 seconds (success: True)
2024-02-06 10:36:35.715 DEBUG (MainThread) [custom_components.nintendo_parental] Finished fetching nintendo_parental data in 1.166 seconds (success: True)
2024-02-06 10:37:36.561 DEBUG (MainThread) [custom_components.nintendo_parental] Finished fetching nintendo_parental data in 1.012 seconds (success: True)
2024-02-06 10:37:43.549 DEBUG (MainThread) [custom_components.nintendo_parental.time] Got request to add bonus time for device 85fba71148c5fe93 to 00:30:00
2024-02-06 10:37:45.157 DEBUG (MainThread) [custom_components.nintendo_parental] Finished fetching nintendo_parental data in 0.912 seconds (success: True)
2024-02-06 10:38:45.688 DEBUG (MainThread) [custom_components.nintendo_parental] Finished fetching nintendo_parental data in 1.139 seconds (success: True)

Diagnostics dump

No response

@pantherale0
Copy link
Owner

Hi

That behaviour is odd, bonus time only goes in one direction currently to replicate some of the functionality found in Family Link. But when you restart it shouldn't be jumping from 3:15 to 4:15, unless you have set bonus time to either 60 minutes, or 30 minutes (twice).

Its not native functionality available via Nintendo so I expect the value to be lost during a restart.

Will take a look when I've got some time.

Thanks

@pantherale0
Copy link
Owner

Bonus time is being removed for now.

You are probably best building automations/scripts in Home Assistant for this. The complexity is too high to implement something that Nintendo doesn't even support directly anyway.

@pantherale0 pantherale0 closed this as not planned Won't fix, can't repro, duplicate, stale Mar 4, 2024
pantherale0 added a commit that referenced this issue Mar 7, 2024
Fix exception for user screen time sensor #87 
Fix authentication issues - #86 
Replace time entities with number entities - #84  
Remove bonus time - #81
Update readme - #89 
---------

Co-authored-by: Rob <robert.mclellan+github@gmail.com>
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