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
adjust TimeFreeze on reboot rather than dropping it #30
Conversation
Good can be better then None. There are times we'd rather have the best guess at the correct time rather than no time at all, so rather than dropping a persisted TimeFreeze on reboot, assume the current device time is consistent with the previous device time. https://jira.lyft.net/browse/NETWORK-1324
👮♀️ |
👀 @Reflejo |
I'll merge it once tests pass, thanks! |
🔒 @Reflejo I just realized that in order to make this all work in Lyft-iOS, I really need to just do the reset unconditionally. Because TimeFreeze is privately loaded in Clock's initialization. I'll update the PR. |
🔓 |
@Reflejo 25 minutes ago, I added another commit on craffert0/Kronos, but it's not showing up here. Should I close this and do a new PR? |
Ugh finally, github picked up the commit. |
🍔 |
🌍 |
Heads up, SQ doesn't work on public repos |
I've restarted the build from the travis UI for you |
🚀 I know it won't automatically work, though. |
Good can be better then None. We'd rather have the
best guess at the correct time rather than no time at all, so rather
than dropping a persisted TimeFreeze on reboot, assume the current device
time is consistent with the previous device time.