Differential D15158

[web] implement QueuedDMOperations ops processing on shared worker
ClosedPublic
Actions

Authored by kamil on Jul 29 2025, 12:21 PM.

Details

Reviewers

tomek

Commits

rCOMM9656146099b8: [web] implement QueuedDMOperations ops processing on shared worker

Summary

ENG-10711.

Depends on D15157

Test Plan

Test processing ops
Read entire store and check if dm ops are there

Diff Detail

Repository

rCOMM Comm

Branch

publish-que

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

kamil created this revision.Jul 29 2025, 12:21 PM

kamil held this revision as a draft.

Herald added a subscriber: ashoat. · View Herald TranscriptJul 29 2025, 12:21 PM

kamil added a child revision: D15159: [lib] read QueuedDMOperations on app start and populate redux.Jul 29 2025, 12:24 PM

Harbormaster completed remote builds in B35028: Diff 49250.Jul 29 2025, 12:39 PM

kamil published this revision for review.Jul 29 2025, 1:17 PM

tomek accepted this revision.Jul 30 2025, 2:00 PM

tomek added inline comments.

web/shared-worker/worker/process-operations.js
669	Just to make sure - the whole payload is an operation and we don't need to extract it from a field, right?

This revision is now accepted and ready to land.Jul 30 2025, 2:00 PM

kamil added inline comments.Jul 30 2025, 3:34 PM

web/shared-worker/worker/process-operations.js
669	Yes, it is defined in D15155

Closed by commit rCOMM9656146099b8: [web] implement QueuedDMOperations ops processing on shared worker. · Explain WhyJul 30 2025, 4:20 PM

This revision was automatically updated to reflect the committed changes.

kamil added a commit: rCOMM9656146099b8: [web] implement QueuedDMOperations ops processing on shared worker.

Revision Contents
Changeset List

Path

Size

web/

shared-worker/

worker/

process-operations.js

8 lines

Diff 49250

View Options

web/shared-worker/worker/process-operations.js

[web] implement QueuedDMOperations ops processing on shared workerClosedPublicActions