[native] Move DeleteThread logic for determining ThreadInfo to hook
ClosedPublic
Actions

Authored by ashoat on Feb 5 2023, 2:26 AM.

Details

Reviewers

atul

Commits

rCOMM2350c4ec3373: [native] Move DeleteThread logic for determining ThreadInfo to hook

Summary

This mirrors the same migration that we did for this logic in ThreadSettings.

Depends on D6594

Test Plan

Flow, code inspection / comparison to ThreadSettings code. Also tested in combination with following diffs

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

ashoat created this revision.Feb 5 2023, 2:26 AM

Herald added a subscriber: tomek. · View Herald TranscriptFeb 5 2023, 2:26 AM

ashoat added a child revision: D6596: [native] Fetch ENS names in ThreadSettings.Feb 5 2023, 2:27 AM

Harbormaster completed remote builds in B16116: Diff 22008.Feb 5 2023, 2:48 AM

ashoat requested review of this revision.Feb 5 2023, 2:48 AM

atul accepted this revision.Feb 5 2023, 7:26 PM

This revision is now accepted and ready to land.Feb 5 2023, 7:26 PM

Closed by commit rCOMM2350c4ec3373: [native] Move DeleteThread logic for determining ThreadInfo to hook. · Explain WhyFeb 5 2023, 8:34 PM

This revision was automatically updated to reflect the committed changes.

ashoat added a commit: rCOMM2350c4ec3373: [native] Move DeleteThread logic for determining ThreadInfo to hook.

Revision Contents
Changeset List

Path

Size

native/

chat/

settings/

delete-thread.react.js

34 lines

Diff 22080

View Options

native/chat/settings/delete-thread.react.js

[native] Move DeleteThread logic for determining ThreadInfo to hookClosedPublicActions