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

TypeError: Cannot read properties of undefined (reading 'timeOfDay') #132

Closed
drobati opened this issue Feb 22, 2023 · 4 comments
Closed
Labels
bug Something isn't working
Milestone

Comments

@drobati
Copy link

drobati commented Feb 22, 2023

Describe the bug
"A JavaScript error occurred in the main process" error window, when trying to open app.

TypeError: Cannot read properties of undefined (reading 'timeOfDay')

To Reproduce
Double-click on the Aerial app.

Expected behavior
Not to do this.

Screenshots
Error window
Windows Version

Aerial Version
1.1.0
Windows 11.

Additional context
Might be related to location permissions? Not sure why you're not able to read the time of day.

@drobati drobati added the bug Something isn't working label Feb 22, 2023
@drobati
Copy link
Author

drobati commented Feb 22, 2023

I turned off Prioritize videos for current time of day, which might be related to that. If I don't update, that probably solved it, and I'll catch the fix later.

@drobati
Copy link
Author

drobati commented Feb 22, 2023

@drobati
Copy link
Author

drobati commented Feb 22, 2023

I'm noticing a second issue, where when I log in with my fingerprint while on the screensaver, it ends the screensaver and logs in successfully, and within the second locks and starts the screensaver a second time.

After logging in the second time it goes away.

OrangeJedi added a commit that referenced this issue Feb 25, 2023
@OrangeJedi
Copy link
Owner

Thanks for letting me know! I took a look at the first error and couldn't get it to reproduce, but I did add some code that should take care of it in the next release. Disabling the Prioritize videos for current time of day will definitely stop the error for now, and tbh, not very many videos have day/night tags right now (see #102)

As for the second issue, that is a known problem that I'll hopefully fix soon.

@OrangeJedi OrangeJedi added this to the v1.2.0 milestone Feb 25, 2023
OrangeJedi added a commit that referenced this issue Mar 6, 2023
-Closes all windows on system lock, unlock, and suspend (see discussion in #132)
-Added option to not run Aerial on battery power
@OrangeJedi OrangeJedi mentioned this issue Mar 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: Released
Development

No branches or pull requests

2 participants