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

Authored by ginsu on Dec 27 2022, 3:13 PM.

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, 3:13 PM

Herald added subscribers: atul, tomek, ashoat. · View Herald TranscriptDec 27 2022, 3:13 PM

ginsu edited the summary of this revision. (Show Details)Dec 27 2022, 3:14 PM

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, 3:25 PM

Harbormaster completed remote builds in B14799: Diff 20184.Dec 27 2022, 3:27 PM

ginsu requested review of this revision.Dec 27 2022, 3:27 PM

tomek accepted this revision.Dec 29 2022, 1:05 PM

Resigning to unblock since this was reviewed by @tomek.

This revision is now accepted and ready to land.Dec 29 2022, 8:48 PM

Closed by commit rCOMM2c8502862fe5: [lib/native] move useCanCreateReactionFromMessage hook from native to lib. · Explain WhyDec 29 2022, 9: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

Path

Size

lib/

shared/

reaction-utils.js

39 lines

native/

chat/

multimedia-message.react.js

2 lines

reaction-message-utils.js

39 lines

robotext-message.react.js

2 lines

text-message.react.js

2 lines

Diff 20390

View Options

lib/shared/reaction-utils.js

View Options

native/chat/multimedia-message.react.js

View Options

native/chat/reaction-message-utils.js

View Options

native/chat/robotext-message.react.js

View Options

native/chat/text-message.react.js

[lib/native] move useCanCreateReactionFromMessage hook from native to libClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 20390

lib/shared/reaction-utils.js

native/chat/multimedia-message.react.js

native/chat/reaction-message-utils.js

native/chat/robotext-message.react.js

native/chat/text-message.react.js

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

Revision Contents
Changeset List