-
Notifications
You must be signed in to change notification settings - Fork 80
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
Several assorted improvements #857
Several assorted improvements #857
Conversation
…d it Maybe helps with debugging stuck notifications
d3c1186
to
61043ee
Compare
Smoke test is fine. This code has been active on my daily driver for 6 days with no visible issues. Nightly backups have been succeeding. Would you like me to test the functionality, e.g. user CA cert? |
would be nice, but not essential. I had tested this myself. Maybe it makes a difference with an AOSP build? |
I'm going to accept the "not essential" aspect of this and go ahead and approve! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Passes smoke test for the last few weeks, seems fine.
Closes #825 #822