Page MenuHomePhabricator
Differential D13829

[lib] Hide membership robotext from MessagePreview in thin threads
AcceptedPublic
Actions

Authored by ashoat on Oct 30 2024, 11:48 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Dec 19, 5:33 AM
Unknown Object (File)
Sun, Dec 15, 2:38 AM
Unknown Object (File)
Wed, Dec 4, 12:19 AM
Unknown Object (File)
Sun, Dec 1, 3:37 PM
Unknown Object (File)
Tue, Nov 26, 8:28 AM
Unknown Object (File)
Nov 18 2024, 4:50 PM
Unknown Object (File)
Nov 13 2024, 8:40 AM
Unknown Object (File)
Nov 10 2024, 9:55 PM
View All 27 Files
Subscribers
None

Details

Reviewers
tomek
Summary

Part of addressing ENG-9556.

This diff makes it so that in thin threads, we don't show robotext about users joining/leaving/being added/being removed in MessagePreview on both web and native.

Depends on D13828

Test Plan

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
No Lint Coverage
Unit
No Test Coverage

Event Timeline

ashoat created this revision.Oct 30 2024, 11:48 AM
ashoat added a child revision: D13830: [lib][native][web] Ignore showInMessagePreview if no messages found.
ashoat added inline comments.Oct 30 2024, 11:58 AM
lib/shared/messages/add-members-message-spec.js
233 ↗(On Diff #45479)

Flow didn't like when I tried checking params.threadInfo.thick instead

Harbormaster completed remote builds in B32446: Diff 45479.Oct 30 2024, 1:05 PM
ashoat requested review of this revision.Oct 30 2024, 1:05 PM
tomek accepted this revision.Oct 31 2024, 6:06 AM
tomek added inline comments.
lib/shared/messages/add-members-message-spec.js
233 ↗(On Diff #45479)

That's because only RawThreadInfo has this property. ThreadInfo doesn't have it.

This revision is now accepted and ready to land.Oct 31 2024, 6:06 AM
ashoat removed a child revision: D13830: [lib][native][web] Ignore showInMessagePreview if no messages found.Oct 31 2024, 11:08 AM
ashoat added a child revision: D13910: [lib] Inline getMostRecentMessageInfo.Nov 12 2024, 12:39 PM
ashoat mentioned this in D13919: [lib] Don't bump thin thread timestamp for membership robotext.Nov 12 2024, 12:47 PM
ashoat removed a child revision: D13910: [lib] Inline getMostRecentMessageInfo.Wed, Dec 11, 3:30 PM
ashoat added a child revision: D13918: [lib] Reactions only bump thread timestamps when reacting to viewer.Wed, Dec 11, 3:54 PM
ashoat updated this revision to Diff 46364.Wed, Dec 11, 3:57 PM

Rebase

Harbormaster completed remote builds in B33046: Diff 46364.Wed, Dec 11, 5:09 PM

Revision Contents
Changeset List

Diff 46364

View Options

lib/shared/messages/add-members-message-spec.js

Loading...
View Options

lib/shared/messages/join-thread-message-spec.js

Loading...
View Options

lib/shared/messages/leave-thread-message-spec.js

Loading...
View Options

lib/shared/messages/remove-members-message-spec.js

Loading...