Page MenuHomePhabricator
Differential D6054

[lib/native] move useCanCreateReactionFromMessage hook from native to lib
ClosedPublic
Actions

Authored by ginsu on Dec 27 2022, 7:13 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Oct 1, 3:01 PM
Unknown Object (File)
Sat, Sep 28, 10:31 AM
Unknown Object (File)
Sat, Sep 28, 10:31 AM
Unknown Object (File)
Sat, Sep 28, 10:31 AM
Unknown Object (File)
Sat, Sep 28, 10:23 AM
Unknown Object (File)
Wed, Sep 18, 8:14 AM
Unknown Object (File)
Sep 3 2024, 11:43 PM
Unknown Object (File)
Aug 31 2024, 9:17 PM
View All 93 Files
Subscribers
ashoat
atul
tomek

Details

Reviewers
tomek
atul
Commits
rCOMM2c8502862fe5: [lib/native] move useCanCreateReactionFromMessage hook from native to lib
Summary

Move useCanCreateReactionFromMessage hook into reaction-utils in lib so that both web and native can use this hook

Depends on D6005

Linear Task: ENG-2464

Test Plan

flow also ran app locally on native and nothing crashes

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

ginsu created this revision.Dec 27 2022, 7:13 AM
Herald added subscribers: atul, tomek, ashoat. · View Herald TranscriptDec 27 2022, 7:13 AM
ginsu edited the summary of this revision. (Show Details)Dec 27 2022, 7:14 AM
ginsu added reviewers: atul, tomek.
ginsu added a parent revision: D6005: [lib/keyserver/native] move all the reaction message types into its own file.
ginsu added a child revision: D6055: [web] introduce useOnClickReact hook in reaction message utils.Dec 27 2022, 7:25 AM
Harbormaster completed remote builds in B14799: Diff 20184.Dec 27 2022, 7:27 AM
ginsu requested review of this revision.Dec 27 2022, 7:27 AM
tomek accepted this revision.Dec 29 2022, 5:05 AM
atul resigned from this revision.Dec 29 2022, 12:48 PM

Resigning to unblock since this was reviewed by @tomek.

This revision is now accepted and ready to land.Dec 29 2022, 12:48 PM
Closed by commit rCOMM2c8502862fe5: [lib/native] move useCanCreateReactionFromMessage hook from native to lib (authored by ginsu). · Explain WhyDec 29 2022, 1:03 PM
This revision was automatically updated to reflect the committed changes.
ginsu added a commit: rCOMM2c8502862fe5: [lib/native] move useCanCreateReactionFromMessage hook from native to lib.

Revision Contents
Changeset List

Diff 20390

View Options

lib/shared/reaction-utils.js

Loading...
View Options

native/chat/multimedia-message.react.js

Loading...
View Options

native/chat/reaction-message-utils.js

Loading...
View Options

native/chat/robotext-message.react.js

Loading...
View Options

native/chat/text-message.react.js

Loading...