Page MenuHomePhabricator

[lib] Make sure thick threads starts out with startReached
ClosedPublic

Authored by ashoat on Jul 20 2024, 9:43 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Sep 9, 8:43 PM
Unknown Object (File)
Mon, Sep 9, 8:43 PM
Unknown Object (File)
Mon, Sep 9, 8:43 PM
Unknown Object (File)
Mon, Sep 9, 8:43 PM
Unknown Object (File)
Thu, Sep 5, 10:34 PM
Unknown Object (File)
Thu, Sep 5, 10:08 PM
Unknown Object (File)
Thu, Sep 5, 2:16 AM
Unknown Object (File)
Wed, Sep 4, 6:25 PM
Subscribers
None

Details

Summary

In order to make sure that startReached starts out as true when a thread is created, we set truncationStatus to "exhaustive".

I thought about always using exhaustive, but this conflicts with our plans in the Limit number of displayed DMs project, where we'll want startReached to be false if there are messages in SQLite that are not represented in Redux.

Depends on D12827

Test Plan

I followed the test plan described in this Linear comment. Before this diff, the spinner was visible in the newly-created thick thread. After this diff, it was no longer visible

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage