Page MenuHomePhabricator
Differential D13827

[lib] Introduce MessageSpec.showInMessagePreview
ClosedPublic
Actions

Authored by ashoat on Oct 30 2024, 11:46 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Dec 26, 7:11 PM
Unknown Object (File)
Thu, Dec 26, 7:11 PM
Unknown Object (File)
Thu, Dec 26, 7:11 PM
Unknown Object (File)
Thu, Dec 26, 11:37 AM
Unknown Object (File)
Thu, Dec 26, 5:50 AM
Unknown Object (File)
Nov 28 2024, 6:04 AM
Unknown Object (File)
Nov 26 2024, 5:28 AM
Unknown Object (File)
Nov 23 2024, 4:40 AM
View All 33 Files
Subscribers
None

Details

Reviewers
tomek
Commits
rCOMM726d17a73e0f: [lib] Introduce MessageSpec.showInMessagePreview
Summary

This diff introduces a generic way to support ENG-9556.

Depends on D13826

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:46 AM
ashoat added a child revision: D13828: [lib] Only show reactions to viewer's messages in MessagePreview.
ashoat added inline comments.Oct 30 2024, 11:56 AM
lib/shared/messages/multimedia-message-spec.js
373–374

See here for an explanation on this change

Harbormaster returned this revision to the author for changes because remote builds failed.Oct 30 2024, 12:45 PM
Harbormaster failed remote builds in B32444: Diff 45477!
Harbormaster completed remote builds in B32444: Diff 45477.Oct 30 2024, 1:40 PM
ashoat requested review of this revision.Oct 30 2024, 1:40 PM
tomek accepted this revision.Oct 31 2024, 5:56 AM
This revision is now accepted and ready to land.Oct 31 2024, 5:56 AM
ashoat mentioned this in D13826: [lib] Fetch message to preview from MessagePreview.Oct 31 2024, 6:09 AM
ashoat removed a child revision: D13828: [lib] Only show reactions to viewer's messages in MessagePreview.Oct 31 2024, 11:09 AM
ashoat added a child revision: D13830: [lib][native][web] Ignore showInMessagePreview if no messages found.
Closed by commit rCOMM726d17a73e0f: [lib] Introduce MessageSpec.showInMessagePreview (authored by ashoat). · Explain WhyOct 31 2024, 11:38 AM
This revision was automatically updated to reflect the committed changes.
ashoat added a commit: rCOMM726d17a73e0f: [lib] Introduce MessageSpec.showInMessagePreview.

Revision Contents
Changeset List

PathSize
lib/
hooks/
27 lines
shared/
messages/
10 lines
3 lines

Diff 45477

View Options

lib/hooks/message-hooks.js

Loading...
View Options

lib/shared/messages/message-spec.js

Loading...
View Options

lib/shared/messages/multimedia-message-spec.js

Loading...