Page MenuHomePhabricator
Differential D13157

[lib] add passing DM operation ID when dispatching action
ClosedPublic
Actions

Authored by kamil on Aug 23 2024, 5:49 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Sep 27, 7:25 PM
Unknown Object (File)
Wed, Sep 25, 3:09 PM
Unknown Object (File)
Wed, Sep 25, 12:05 AM
Unknown Object (File)
Sep 18 2024, 3:24 AM
Unknown Object (File)
Sep 16 2024, 10:23 PM
Unknown Object (File)
Sep 9 2024, 10:24 AM
Unknown Object (File)
Sep 9 2024, 10:12 AM
Unknown Object (File)
Sep 8 2024, 11:29 AM
View All 28 Files
Subscribers
ashoat

Details

Reviewers
tomek
inka
Commits
rCOMM1e1af8136d22: [lib] add passing DM operation ID when dispatching action
Summary

ENG-8423.

dmOpID is needed to resolve promise returned from getDMOpsSendingPromise after queueing messages on Tunnelbroker.

Depends on D13156

Test Plan

Test that promise is resolved/rejected.

Note: This stack is tested end-to-end according to the process described here.

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

kamil created this revision.Aug 23 2024, 5:49 AM
kamil held this revision as a draft.
Herald added a subscriber: ashoat. · View Herald TranscriptAug 23 2024, 5:49 AM
kamil added a child revision: D13158: [lib][web][native] implement detecting DM messages as failed.Aug 23 2024, 5:52 AM
Harbormaster completed remote builds in B31241: Diff 43614.Aug 23 2024, 6:08 AM
kamil published this revision for review.Aug 23 2024, 7:27 AM
tomek accepted this revision.Aug 26 2024, 5:13 AM
tomek added inline comments.
lib/shared/dm-ops/process-dm-ops.js
82–91

It will be less confusing when expressed as an if-else because now it looks like metadata could be present for OUTBOUND messages.

This revision is now accepted and ready to land.Aug 26 2024, 5:13 AM
kamil updated this revision to Diff 43642.Aug 26 2024, 7:29 AM

address review

Harbormaster completed remote builds in B31267: Diff 43642.Aug 26 2024, 7:46 AM
Closed by commit rCOMM1e1af8136d22: [lib] add passing DM operation ID when dispatching action (authored by kamil). · Explain WhyAug 26 2024, 9:05 AM
This revision was automatically updated to reflect the committed changes.
kamil added a commit: rCOMM1e1af8136d22: [lib] add passing DM operation ID when dispatching action.

Revision Contents
Changeset List

PathSize
lib/
shared/
dm-ops/
29 lines

Diff 43614

View Options

lib/shared/dm-ops/process-dm-ops.js

Loading...