Page MenuHomePhabricator

[lib] Introduce MessageSpec.showInMessagePreview
Needs ReviewPublic

Authored by ashoat on Wed, Oct 30, 11:46 AM.
Tags
None
Referenced Files
F3105318: D13827.id45477.diff
Wed, Oct 30, 10:19 PM
F3105119: D13827.diff
Wed, Oct 30, 8:22 PM
F3104449: D13827.id45477.diff
Wed, Oct 30, 5:18 PM
F3104421: D13827.id.diff
Wed, Oct 30, 5:17 PM
F3104411: D13827.diff
Wed, Oct 30, 5:16 PM
Subscribers
None

Details

Reviewers
tomek
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

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.Wed, Oct 30, 12:45 PM
Harbormaster failed remote builds in B32444: Diff 45477!