[native] Convert ComposeThreadButton to a hook
ClosedPublic
Actions

Authored by ashoat on Feb 17 2023, 6:57 AM.

Details

Reviewers

Commits

rCOMMd7663dac467c: [native] Convert ComposeThreadButton to a hook

Summary

This diff should be a no-op. In a later diff, I need to update the createPendingThread call here. I figured the component made more sense as a hook and it was a quick refactor.

Depends on D6756

Test Plan

Flow, and playing around with thread composer on native

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

ashoat created this revision.Feb 17 2023, 6:57 AM

Herald added a subscriber: tomek. · View Herald TranscriptFeb 17 2023, 6:57 AM

ashoat added a child revision: D6758: [lib][native][web] Include current user as member when calling createPendingThread from baseCreatePendingSidebar.Feb 17 2023, 6:58 AM

Harbormaster completed remote builds in B16574: Diff 22675.Feb 17 2023, 7:15 AM

ashoat requested review of this revision.Feb 17 2023, 7:15 AM

atul accepted this revision.Feb 20 2023, 1:16 PM

This revision is now accepted and ready to land.Feb 20 2023, 1:16 PM

Closed by commit rCOMMd7663dac467c: [native] Convert ComposeThreadButton to a hook (authored by ashoat). · Explain WhyFeb 20 2023, 3:13 PM

This revision was automatically updated to reflect the committed changes.

ashoat added a commit: rCOMMd7663dac467c: [native] Convert ComposeThreadButton to a hook.

Revision Contents
Changeset List

Path

Size

native/

chat/

compose-thread-button.react.js

83 lines

Diff 22675

View Options

native/chat/compose-thread-button.react.js

[native] Convert ComposeThreadButton to a hookClosedPublicActions