Small move/rename diff to convert this hook into a general function. This "hook" was not using any react features so this really should just be a function
Depends on D8600
Paths
| Differential D8609 Authored by ginsu on Jul 24 2023, 4:22 PM.
Details Summary Small move/rename diff to convert this hook into a general function. This "hook" was not using any react features so this really should just be a function Depends on D8600 Test Plan flow and confirmed that native and web apps work fine
Diff Detail
Event TimelineHerald added subscribers: tomek, ashoat. · View Herald TranscriptJul 24 2023, 4:22 PM2023-07-24 16:22:08 (UTC-7) ginsu edited the summary of this revision. (Show Details)Jul 24 2023, 4:22 PM2023-07-24 16:22:37 (UTC-7) ginsu added a parent revision: D8600: [native] show which reactions the viewer has already selected in the inline engagement. Harbormaster completed remote builds in B21158: Diff 28980.Jul 24 2023, 4:39 PM2023-07-24 16:39:34 (UTC-7) ginsu added a child revision: D8610: [native] introduce DummyInlineEngagementNode.Jul 24 2023, 7:44 PM2023-07-24 19:44:53 (UTC-7) This revision is now accepted and ready to land.Jul 24 2023, 11:11 PM2023-07-24 23:11:42 (UTC-7) This revision was landed with ongoing or failed builds.Jul 31 2023, 1:19 PM2023-07-31 13:19:27 (UTC-7) Closed by commit rCOMM9c5f41a61b9b: [lib/native/web] convert useInlineEngagementText to… (authored by ginsu). · Explain Why This revision was automatically updated to reflect the committed changes. Harbormaster completed remote builds in B21397: Diff 29309.Jul 31 2023, 2:33 PM2023-07-31 14:33:42 (UTC-7)
Revision Contents
Diff 29309 lib/hooks/inline-engagement-text.react.js
lib/shared/inline-engagement-utils.js
native/chat/inline-engagement.react.js
web/chat/inline-engagement.react.js
|