[lib][native][web] Update useSidebarInfos to include lastUpdatedAtLeastTime
ClosedPublic
Actions

Authored by ashoat on Tue, Nov 12, 12:42 PM.

Details

Reviewers

tomek

Commits

rCOMMed9e9cf2c268: [lib][native][web] Update useSidebarInfos to include lastUpdatedAtLeastTime

Summary

Before this diff, useSidebarInfos would return lastUpdatedAtLeastTime as lastUpdatedTime, as a temporary hack to help split up my work into smaller diffs.

In an earlier diff we updated lastUpdatedTime to be a Promise, and this allows us to avoid refactoring everything to handle the Promise.

In this diff, we address this temporary hack for sidebars, but for not for the ChatThreadItem itself (which will be handled later).

useSidebarInfos is used in places outside of chat-selectors.js, so this required some additional refactoring.

Depends on D13914

Test Plan

Tested in combination with the rest of the stack. See video in D13918

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

ashoat created this revision.Tue, Nov 12, 12:42 PM

ashoat added a child revision: D13916: [lib][web] Await lastUpdatedTime and update ChatThreadItems if different.

ashoat edited the test plan for this revision. (Show Details)Tue, Nov 12, 12:50 PM

Prettier

Harbormaster failed remote builds in B32626: Diff 45762!Tue, Nov 12, 1:10 PM

Harbormaster returned this revision to the author for changes because remote builds failed.Tue, Nov 12, 1:24 PM

Harbormaster failed remote builds in B32633: Diff 45769!

Prettier

Harbormaster completed remote builds in B32638: Diff 45776.Tue, Nov 12, 3:08 PM

ashoat requested review of this revision.Tue, Nov 12, 3:08 PM

tomek accepted this revision.Wed, Nov 13, 4:38 AM

tomek added inline comments.

lib/hooks/search-threads.js
145–159 ↗	(On Diff #45776)	Wondering whether we shouldn't just use the deep equal from lodash instead. If we have some checks for the `ID` change, should we protect ourselves against the change to some other fields?

This revision is now accepted and ready to land.Wed, Nov 13, 4:38 AM

ashoat added inline comments.Wed, Nov 13, 10:34 AM

lib/hooks/search-threads.js
145–159 ↗	(On Diff #45776)	In this case, we are comparing the results of `getAllFinalSidebarItems` against `getAllInitialSidebarItems` for the same input. Based on the implementation of those functions, the only field that might be different is `lastUpdatedTime`