HomePhabricator
Diffusion Comm 4375848efb55

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

Description

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

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.

Reviewers: atul, inka, kamil

Reviewed By: kamil

Subscribers: ashoat

Differential Revision: https://phab.comm.dev/D11195

Details

Provenance
tomekAuthored on Feb 29 2024, 1:30 AM
Reviewer
kamil
Differential Revision
D11195: [native] Use connectivity instead of keyserver connection status in Calendar
Parents
rCOMM1ee876eefe7e: [lib] introduce addCommunityActionType
Branches
Unknown
Tags
Unknown