[lib] Resolve the promise after processing is done
AbandonedPublicDraft
Actions

Authored by tomek on Mar 21 2024, 8:31 AM.

Tags

None

Referenced Files

	F3354246: D11370.diff
	Sat, Nov 23, 12:31 PM

	Unknown Object (File)
	Tue, Nov 12, 10:55 PM

	Unknown Object (File)
	Tue, Nov 12, 10:55 PM

	Unknown Object (File)
	Tue, Nov 12, 9:39 PM

	Unknown Object (File)
	Fri, Nov 1, 5:40 PM

	Unknown Object (File)
	Oct 11 2024, 10:34 AM

	Unknown Object (File)
	Oct 11 2024, 10:34 AM

	Unknown Object (File)
	Oct 11 2024, 9:15 AM

View All 41 Files

Subscribers

ashoat

Details

Reviewers

kamil
inka
will

Summary

Maintain the mapping between actions and promises and resolve after the processing is done.

https://linear.app/comm/issue/ENG-7108/resolve-the-promise-after-ops-processing

Depends on D11369

Test Plan

Modify one dispatch, which dispatches draft update action. Add a sleep to DB processing code. Check if the promise is resolved after the timeout.

Diff Detail

Repository

rCOMM Comm

Branch

ops

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

tomek created this revision.Mar 21 2024, 8:31 AM

Herald added a subscriber: ashoat. · View Herald TranscriptMar 21 2024, 8:31 AM

Harbormaster returned this revision to the author for changes because remote builds failed.Mar 21 2024, 8:42 AM

Harbormaster failed remote builds in B27651: Diff 38272!

In the new approach, we no longer need this.

Revision Contents
Changeset List

Path

Size

lib/

components/

ops-context-provider.react.js

34 lines

Diff 38272

View Options

lib/components/ops-context-provider.react.js

[lib] Resolve the promise after processing is doneAbandonedPublicDraftActions