-
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
Integration not working for me #22
Comments
Just a few notes for me Errors to resolve:
A device is returned from the API, however the logs don't currently give enough information as to where the list index error occurs, but I'm assuming as part of this function https://github.com/pantherale0/pynintendoparental/blob/main/pynintendoparental/device.py#L112 EDIT - Increased logging with #23 Need to first add more debug logs to the module which will help get to the bottom of the error (the console I have isn't used too often so could be something I've missed). |
Could you update to 0.1.4b1 and test again please? I'm expecting it to fail, but there should be a lot more logs available now to find out where the problem is. |
Hi, Thanks for the quick help, here it is : https://www.swisstransfer.com/d/8cba9b6d-f081-4af7-b8ac-8c047fe6060e |
Thank you 👍 That makes sense now, the module tries to get the daily summary for the current day (to calculate things like playing time today, disabled time, exceeded time, observations, important info and notices) however in your case it doesn't actually exist, and the latest entry is as follows:
So I know its directly caused by the API / Nintendo itself. What I now don't know is how your parental controls is configured for Nintendo to stop calculating daily summaries. Is there a chance you could provide the settings set in the "Settings" page of the mobile app please? Perhaps a setting there changes the behavior. |
Also, updating to 0.1.4b2 should now allow the component to finish init |
Here are the screenshot of settings of the app... I don't see anything special. I did the update, tested, same behaviour but less logs in error : https://www.swisstransfer.com/d/796436ac-ba86-4e9e-aa7d-22340d004386 |
thanks for the screenshots. I think its because there is no restrictions set on the device. I'll have a test later on. RE the icon, I did try adding the switch icon into the brands repo a while back but it wasn't accepted at the repo only accepts manufacturer logos/icons, not product logos/icons. I'm yet to source a valid Nintendo icon. |
So I set limits for the time played and content, I have delete and reconfigured. Still the same issue : https://www.swisstransfer.com/d/afe49719-ab0e-47ba-8b63-cad5da29ddae |
You would need to leave it set for Nintendo to finish calculating the daily summaries (they seem to do it on a daily basis, around midnight, but sometimes can be later, or sometimes earlier). |
Sorry, been tied up with work. There are some changes made upstream that will continue setup if the requested date doesn't exist, however time reporting won't work because that is the only place the data exists. I'll try and sort out a new version to allow platform setup to resume, and create an issue in Home Assistant instead (https://developers.home-assistant.io/docs/core/platform/repairs) that will disappear once the data is available. |
@Seexelas can you send another more recent log file please? I'm wondering if this might be timezone related. |
Hi @pantherale0, Thanks for your quick feedback. Here are the last logs : https://www.swisstransfer.com/d/acf28cb3-38dc-46c8-93c2-1b215c2cf93c |
|
Now I've just posted that, I saw you had "Switch 6" and "Switch 7" in the screenshots from the mobile app, perhaps its only one of the consoles that are impacted. I'll make changes tomorrow (had no internet connection) and get the updates released into beta asap. |
I have one switch which is on my account and is also the main account which had never been synchonized with Nintendo Parental. (which is switch 6 and 7, i've made two tests). I have two kids who's switch is added to my application also but they have their account and we are grouped in family. My timezone is europe/luxembourg. |
@Seexelas can you test the latest version I've just released please? An issue should be created instead as a way of alerting that there is a problem with the device, but as I don't have anything affected, I can't test properly myself :( |
I have indeed no more the date error, but a new error shows, below the last logs : https://www.swisstransfer.com/d/6fb27660-31cc-468f-a537-6de622f443e8 Thanks, |
Ok, I need the full logs from Home Assistant, could you provide and create a new issue please as this issue is separate from this one. |
Starting to look good now (despite the new error), we're slowly getting rid of the bugs, I know what needs to be done for that error, thanks for testing so quickly 👍 |
Ok, another update @Seexelas - can you test again please? Hopefully this one sorts it. |
can you send logs please? |
I forgot to past the link sorry : https://www.swisstransfer.com/d/7abbeff8-1082-40dd-a028-dde366875164 |
No worries :) |
ok, new update, can you test again please? |
With version 0.1.4b5, KO : https://www.swisstransfer.com/d/6c7a95cb-749b-435b-a7cc-71e75b3050ea |
ahh my fault there, didn't set the var in the module 👎 updated, try now. |
So yes, the package broke my HA (don't worry, it's not an issue) : https://www.swisstransfer.com/d/9b87a635-0ce2-4f26-880a-5552191d5064 The logs are here, if I remove the integration, it starts again. I will clean everything, reboot and retry, and i'll let you know the result. |
So i cleaned everything, new download of the last beta, restart. Configuration of the plugin, restart. It still not working : https://www.swisstransfer.com/d/456e3d92-650f-48ed-b3a7-e7e5cf2ce9c1 |
Ok, that shouldn't have happened, pulled that release and created a new one to remove the issue registry stuff for now to see if that helps. first logs didn't even show that the integration was installed. |
There is one more error in the logs, I don't know if it impact something : https://www.swisstransfer.com/d/62c79761-a485-46c3-8ff2-4d0e966e213e |
@Seexelas if the error is this one:
That can happen, basically the access token that Nintendo issues expires after 10 minutes, its purely cosmetic though and the next request to the API will refresh the token in the background. Also - because the API is not returning any of the daily information for your account, I would expect some of the values to be reported incorrectly. Kinda need someone who has the issue, can also intercept the HTTP requests that the mobile app makes and be able to debug the pynintendoparental module to be able to resolve that. Make sure debug logging is turned off for now though, otherwise it will spam the logs quite a bit ;) Will close this for now :) thank you for testing where I have been unable to. |
System Health details
System Information
Home Assistant Community Store
Home Assistant Cloud
Home Assistant Supervisor
Dashboards
Recorder
Checklist
Describe the issue
Hi,
I installed the integration plugin, configured my nintendo switch account, everything is reported as success.
But I have no device, no entities showing, and I found some errors in my logs.
Thanks for the help,
Reproduction steps
...
Debug logs
Diagnostics dump
I don't have one as no device is showing.
The text was updated successfully, but these errors were encountered: