Page MenuHomePhabricator
Differential D13186

[lib][native] update `useEditMessage` return type
ClosedPublic
Actions

Authored by kamil on Aug 28 2024, 2:53 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Nov 23, 10:42 PM
Unknown Object (File)
Sat, Nov 23, 10:12 PM
Unknown Object (File)
Wed, Nov 6, 10:04 PM
Unknown Object (File)
Wed, Nov 6, 10:04 PM
Unknown Object (File)
Wed, Nov 6, 10:04 PM
Unknown Object (File)
Wed, Nov 6, 10:04 PM
Unknown Object (File)
Wed, Nov 6, 9:41 PM
Unknown Object (File)
Oct 6 2024, 8:29 PM
View All 48 Files
Subscribers
ashoat

Details

Reviewers
tomek
inka
Commits
rCOMM6cc6e90afda0: [lib][native] update `useEditMessage` return type
Summary

ENG-8976;

SendEditMessageResult is used only in dispatchActionPromise, other than that we only care about success/failure.

With that change, we're able to use this hook also for DMs where SendEditMessageResult is not existing.

Test Plan

Editing still works

Diff Detail

Repository
rCOMM Comm
Branch
logic-3
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

kamil created this revision.Aug 28 2024, 2:53 AM
kamil held this revision as a draft.
Herald added a subscriber: ashoat. · View Herald TranscriptAug 28 2024, 2:53 AM
kamil added a child revision: D13187: [lib] add editing DM messages.Aug 28 2024, 2:56 AM
Harbormaster completed remote builds in B31314: Diff 43722.Aug 28 2024, 3:12 AM
kamil published this revision for review.Aug 28 2024, 3:59 AM
tomek accepted this revision.Aug 28 2024, 5:34 AM
tomek added inline comments.
lib/shared/edit-messages-utils.js
28–48

We can simplify the syntax

This revision is now accepted and ready to land.Aug 28 2024, 5:34 AM
kamil added a parent revision: D13193: [lib] make it possible to send DM ops to all thread members.Aug 28 2024, 6:24 AM
kamil updated this revision to Diff 43744.Aug 28 2024, 6:37 AM

simplify syntax

Harbormaster completed remote builds in B31334: Diff 43744.Aug 28 2024, 6:56 AM
Closed by commit rCOMM6cc6e90afda0: [lib][native] update `useEditMessage` return type (authored by kamil). · Explain WhyAug 28 2024, 8:33 AM
This revision was automatically updated to reflect the committed changes.
kamil added a commit: rCOMM6cc6e90afda0: [lib][native] update `useEditMessage` return type.

Revision Contents
Changeset List

PathSize
lib/
shared/
7 lines
native/
chat/
10 lines

Diff 43722

View Options

lib/shared/edit-messages-utils.js

Loading...
View Options

native/chat/chat-input-bar.react.js

Loading...