Page MenuHomePhabricator

Implement notificationsCreationData for send-reaction-message-spec
AbandonedPublic

Authored by marcin on Aug 8 2024, 4:04 AM.
Tags
None
Referenced Files
F3543574: D13026.diff
Thu, Dec 26, 11:25 AM
Unknown Object (File)
Wed, Dec 18, 3:43 PM
Unknown Object (File)
Wed, Dec 18, 3:43 PM
Unknown Object (File)
Wed, Dec 18, 3:43 PM
Unknown Object (File)
Nov 20 2024, 10:59 PM
Unknown Object (File)
Nov 10 2024, 3:07 PM
Unknown Object (File)
Nov 10 2024, 2:56 PM
Unknown Object (File)
Nov 10 2024, 5:54 AM
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