Page MenuHomePhabricator
Differential D13324

[lib] update `processDMOpsActionType` to handle composable DM messages
ClosedPublic
Actions

Authored by kamil on Sep 13 2024, 3:40 AM.
Tags
None
Referenced Files
F3200932: D13324.id44206.diff
Sat, Nov 9, 2:59 PM
Unknown Object (File)
Sun, Nov 3, 6:46 AM
Unknown Object (File)
Mon, Oct 21, 5:43 AM
Unknown Object (File)
Sun, Oct 20, 2:58 PM
Unknown Object (File)
Sun, Oct 20, 2:58 PM
Unknown Object (File)
Sun, Oct 20, 2:58 PM
Unknown Object (File)
Sun, Oct 20, 2:57 PM
Unknown Object (File)
Mon, Oct 14, 2:29 PM
View All 38 Files
Subscribers
ashoat

Details

Reviewers
tomek
inka
Commits
rCOMM5095574803af: [lib] update `processDMOpsActionType` to handle composable DM messages
Summary

ENG-8424.

Previously it was done using sendDMActionTypes.started - but this code will be removed later in the stack.

Depends on D13323

Test Plan

Tested, that only when sending text message outboundP2PMessageIDs is set, and failed messages are persisted.

Diff Detail

Repository
rCOMM Comm
Branch
publish
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

kamil created this revision.Sep 13 2024, 3:40 AM
kamil held this revision as a draft.
Herald added a subscriber: ashoat. · View Herald TranscriptSep 13 2024, 3:40 AM
kamil added a child revision: D13325: [lib] update `processOutboundP2PMessages` to return failed message IDs.Sep 13 2024, 3:43 AM
Harbormaster completed remote builds in B31592: Diff 44137.Sep 13 2024, 3:57 AM
kamil published this revision for review.Sep 13 2024, 4:34 AM
kamil added inline comments.
lib/reducers/message-reducer.js
1926
tomek accepted this revision.Sep 13 2024, 6:14 AM
tomek added inline comments.
lib/reducers/message-reducer.js
1908–1909

Do we still need to keep these two conditions?

This revision is now accepted and ready to land.Sep 13 2024, 6:14 AM
kamil added inline comments.Sep 16 2024, 3:00 AM
lib/reducers/message-reducer.js
1908–1909

yeah, otherwise Flow complains

kamil updated this revision to Diff 44206.Sep 16 2024, 3:00 AM

fix comment

Harbormaster completed remote builds in B31634: Diff 44206.Sep 16 2024, 3:16 AM
Closed by commit rCOMM5095574803af: [lib] update `processDMOpsActionType` to handle composable DM 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: rCOMM5095574803af: [lib] update `processDMOpsActionType` to handle composable DM messages.

Revision Contents
Changeset List

PathSize
lib/
reducers/
21 lines

Diff 44137

View Options

lib/reducers/message-reducer.js

Loading...