Page MenuHomePhabricator
Differential D13329

[web][native] update `InputStateContainer` logic for sending text messages
ClosedPublic
Actions

Authored by kamil on Sep 13 2024, 4:12 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Nov 14, 1:02 PM
Unknown Object (File)
Thu, Nov 14, 12:59 PM
Unknown Object (File)
Tue, Nov 12, 2:36 PM
Unknown Object (File)
Sat, Nov 9, 8:06 AM
Unknown Object (File)
Wed, Oct 30, 3:06 AM
Unknown Object (File)
Tue, Oct 22, 8:15 PM
Unknown Object (File)
Mon, Oct 21, 1:43 AM
Unknown Object (File)
Oct 20 2024, 3:54 PM
View All 45 Files
Subscribers
ashoat

Details

Reviewers
tomek
inka
Commits
rCOMM303c9c123089: [web][native] update `InputStateContainer` logic for sending text messages
Summary

ENG-8424.

This makes updates code to use one hook on both platforms which handles both thick and thin threads, and avoids hacking logic on both InputStateContainer implementations.

Depends on D13328

Test Plan

Tested end-to-end according to the process described here.

Additionally, tested creating threads and sidebars.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kamil created this revision.Sep 13 2024, 4:12 AM
kamil held this revision as a draft.
Herald added a subscriber: ashoat. · View Herald TranscriptSep 13 2024, 4:12 AM
Harbormaster failed remote builds in B31598: Diff 44143!Sep 13 2024, 4:16 AM
kamil added a child revision: D13330: [web][native][lib] deprecate `sendDMActionTypes` and simplify hooks.Sep 13 2024, 4:17 AM
kamil edited the test plan for this revision. (Show Details)Sep 13 2024, 4:22 AM
Harbormaster completed remote builds in B31598: Diff 44143.Sep 13 2024, 4:37 AM
kamil published this revision for review.Sep 13 2024, 4:40 AM
kamil edited the summary of this revision. (Show Details)
tomek accepted this revision.Sep 13 2024, 6:57 AM
This revision is now accepted and ready to land.Sep 13 2024, 6:57 AM
kamil updated this revision to Diff 44210.Sep 16 2024, 3:08 AM

rebase

Harbormaster completed remote builds in B31638: Diff 44210.Sep 16 2024, 3:28 AM
Closed by commit rCOMM303c9c123089: [web][native] update `InputStateContainer` logic for sending text messages (authored by kamil). · Explain WhySep 16 2024, 4:05 AM
This revision was automatically updated to reflect the committed changes.
kamil added a commit: rCOMM303c9c123089: [web][native] update `InputStateContainer` logic for sending text messages.
kamil mentioned this in D13334: Call textMessageCreationSideEffectsFunc after sending DM text message.Sep 17 2024, 12:50 AM

Revision Contents
Changeset List

PathSize
native/
input/
131 lines
web/
input/
132 lines

Diff 44224

View Options

native/input/input-state-container.react.js

Loading...
View Options

web/input/input-state-container.react.js

Loading...