Page MenuHomePhabricator

Implement notificationsCreationData for send-reaction-message-spec
AbandonedPublic

Authored by marcin on Aug 8 2024, 4:04 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Nov 10, 3:07 PM
Unknown Object (File)
Sun, Nov 10, 2:56 PM
Unknown Object (File)
Sun, Nov 10, 5:54 AM
Unknown Object (File)
Oct 15 2024, 1:21 AM
Unknown Object (File)
Oct 10 2024, 12:49 PM
Unknown Object (File)
Oct 10 2024, 12:49 PM
Unknown Object (File)
Oct 10 2024, 11:47 AM
Unknown Object (File)
Sep 22 2024, 6:28 PM
Subscribers

Details

Reviewers
kamil
tomek
ashoat
Summary

This differential implements notificationsCreationData for reaction-message-spec

Test Plan
  1. Apply this small patch: https://gist.github.com/marcinwasowicz/d8a0f08110384e3f740b7cf367136b37
  2. Send text message
  3. Ensure that recipient is seeing notification informing about heart reaction

Diff Detail

Repository
rCOMM Comm
Branch
marcin/eng-8237
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

marcin retitled this revision from Implement notificationsCreationData for send-reaction=message-spec to Implement notificationsCreationData for send-reaction-message-spec.
Harbormaster returned this revision to the author for changes because remote builds failed.Aug 8 2024, 4:27 AM
Harbormaster failed remote builds in B31005: Diff 43254!
marcin requested review of this revision.Aug 8 2024, 5:05 AM
lib/shared/dm-ops/send-reaction-message-spec.js
87–97

Code duplication – let's see if we can reduce it. See comments in parent diffs

ashoat requested changes to this revision.Aug 8 2024, 7:26 AM
This revision now requires changes to proceed.Aug 8 2024, 7:26 AM