Page MenuHomePhabricator
Differential D13322

[lib] handle failed P2P messages when sending composable DM message
ClosedPublic
Actions

Authored by kamil on Sep 13 2024, 3:25 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Oct 20, 2:57 PM
Unknown Object (File)
Sun, Oct 20, 2:57 PM
Unknown Object (File)
Sun, Oct 20, 2:57 PM
Unknown Object (File)
Mon, Oct 14, 2:29 PM
Unknown Object (File)
Oct 10 2024, 8:01 AM
Unknown Object (File)
Oct 3 2024, 6:09 PM
Unknown Object (File)
Sep 30 2024, 9:37 PM
Unknown Object (File)
Sep 27 2024, 4:38 AM
View All 28 Files
Subscribers
ashoat

Details

Reviewers
tomek
inka
Commits
rCOMM122966e5c4ac: [lib] handle failed P2P messages when sending composable DM message
Summary

ENG-8424.

Previously, it was implemented on sendDMActionTypes.success, but this was overcomplicated and will be removed.

Depends on D13321

Test Plan

Test dispatching action and make sure field is properly set.

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:25 AM
kamil held this revision as a draft.
Herald added a subscriber: ashoat. · View Herald TranscriptSep 13 2024, 3:25 AM
kamil added inline comments.Sep 13 2024, 3:29 AM
lib/types/redux-types.js
727

Here @ashoat suggested a different name, but I think it's readable enough because we use Outbound P2P messages only for thick thread DMs, but I can rename it, not feeling strongly

kamil added a child revision: D13323: [lib] rename `messageIDWithoutAutoRetry` -> `composableMessageID`.Sep 13 2024, 3:33 AM
Harbormaster completed remote builds in B31590: Diff 44128.Sep 13 2024, 3:42 AM
kamil published this revision for review.Sep 13 2024, 4:33 AM
tomek accepted this revision.Sep 13 2024, 6:02 AM
This revision is now accepted and ready to land.Sep 13 2024, 6:02 AM
ashoat added inline comments.Sep 14 2024, 9:14 AM
lib/types/redux-types.js
727

I think adding "P2P" here is sufficient to avoid confusion :)

Closed by commit rCOMM122966e5c4ac: [lib] handle failed P2P messages when sending composable DM message (authored by kamil). · Explain WhySep 16 2024, 4:05 AM
This revision was automatically updated to reflect the committed changes.
kamil added a commit: rCOMM122966e5c4ac: [lib] handle failed P2P messages when sending composable DM message.

Revision Contents
Changeset List

PathSize
lib/
reducers/
18 lines
types/
2 lines

Diff 44128

View Options

lib/reducers/message-reducer.js

Loading...
View Options

lib/types/redux-types.js

Loading...