Paths

Table of Contentst

[lib] send confirmation for ignored DM ops
ClosedPublic
Actions

Authored by kamil on Oct 9 2024, 7:41 AM.

Tags

None

Referenced Files

	F3362107: D13663.id45017.diff
	Sun, Nov 24, 9:27 PM

	F3362103: D13663.id45140.diff
	Sun, Nov 24, 9:26 PM

	F3361868: D13663.diff
	Sun, Nov 24, 7:53 PM

	Unknown Object (File)
	Sat, Nov 23, 5:56 AM

	Unknown Object (File)
	Wed, Nov 20, 9:13 AM

	Unknown Object (File)
	Mon, Nov 11, 1:18 AM

	Unknown Object (File)
	Sun, Nov 10, 11:25 PM

	Unknown Object (File)
	Sun, Nov 10, 4:45 PM

View All 58 Files

Subscribers

ashoat

Details

Reviewers

tomek

Commits

rCOMMeed6217c4364: [lib] send confirmation for ignored DM ops

Summary

ENG-9456.

Avoiding keeping this operation forever when it's invalid.

Depends on D13662

Test Plan

Created DM Op which will be ignored and make sure it is confirmed.

Diff Detail

Repository

rCOMM Comm

Branch

fix-count

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

kamil created this revision.Oct 9 2024, 7:41 AM

kamil held this revision as a draft.

Herald added a subscriber: ashoat. · View Herald TranscriptOct 9 2024, 7:41 AM

kamil published this revision for review.Oct 9 2024, 7:52 AM

Harbormaster completed remote builds in B32145: Diff 45017.Oct 9 2024, 7:59 AM

Why do we ignore all the promises? What is the downside of awaiting them?

This revision is now accepted and ready to land.Oct 10 2024, 5:29 AM

use await

Harbormaster completed remote builds in B32223: Diff 45140.Oct 14 2024, 3:51 AM

Closed by commit rCOMMeed6217c4364: [lib] send confirmation for ignored DM ops (authored by kamil). · Explain WhyOct 14 2024, 4:07 AM

This revision was automatically updated to reflect the committed changes.

kamil added a commit: rCOMMeed6217c4364: [lib] send confirmation for ignored DM ops.

Revision Contents
Changeset List

Path

Size

lib/

shared/

dm-ops/

process-dm-ops.js

6 lines

Diff 45017

View Options

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

[lib] send confirmation for ignored DM opsClosedPublicActions