Page MenuHomePhabricator

[lib][web] Introduce useCreateChatThreadItem
ClosedPublic

Authored by ashoat on Wed, Oct 30, 11:41 AM.
Tags
None
Referenced Files
F3356297: D13823.id45538.diff
Sat, Nov 23, 6:04 PM
F3355315: D13823.diff
Sat, Nov 23, 2:15 PM
F3351670: D13823.diff
Sat, Nov 23, 3:04 AM
Unknown Object (File)
Thu, Nov 21, 1:09 PM
Unknown Object (File)
Mon, Nov 18, 3:19 PM
Unknown Object (File)
Fri, Nov 15, 10:14 AM
Unknown Object (File)
Thu, Nov 14, 7:04 PM
Unknown Object (File)
Thu, Nov 14, 3:02 PM
Subscribers
None

Details

Summary

Factoring this out into a hook because in a later diff, I update it to depend on another hook.

Depends on D13822

Test Plan

Flow, and tested in combination with the rest of the stack:

  1. Make sure membership operations (user joining / leaving) don't appear in MessagePreview
  2. Make sure reactions to the viewer's messages still appear in MessagePreview
  3. Make sure reactions to other user's messages don't appear in MessagePreview
  4. Test fresh login to thread with only one message in the last 14 days, which is a reaction to a non-viewer message. Make sure the reaction initially appears in MessagePreview, but then is replaced after more messages are fetched by the client

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable