Page MenuHomePhabricator

[native] Use connectivity instead of keyserver connection status in Calendar
ClosedPublic

Authored by tomek on Feb 29 2024, 2:50 AM.
Tags
None
Referenced Files
F3396311: D11195.diff
Sun, Dec 1, 11:41 AM
Unknown Object (File)
Fri, Nov 29, 9:43 AM
Unknown Object (File)
Tue, Nov 12, 2:25 PM
Unknown Object (File)
Tue, Nov 12, 12:55 PM
Unknown Object (File)
Tue, Nov 12, 10:51 AM
Unknown Object (File)
Tue, Nov 12, 6:39 AM
Unknown Object (File)
Oct 31 2024, 7:08 AM
Unknown Object (File)
Oct 10 2024, 11:33 AM
Subscribers

Details

Summary

Each keyserver can have different connection status and modifying this logic to take this into account could be complicated. Instead, we can simply check if we're connected to network at all and try to load more entries. If, for some reason, some keyservers won't receive this update, CalendarQueryHandler should fix it later, based on currentCalendarQuery which is updated on updateCalendarQueryActionTypes.started.

https://linear.app/comm/issue/ENG-5772/update-calendar-component

Depends on D11187

Test Plan

Run the app, scroll the calendar - it should update, disconnect Wi-Fi, scroll again - shouldn't update, connect - should update. This doesn't work reliably on a simulator - NetInfo doesn't always fire correct events, but the logic of Calendar works correctly according to the emitted events.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable