Differential D15158

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

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

Tags

None

Referenced Files

None

Subscribers

Details

Reviewers

Commits

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

Summary

Depends on D15157

Test Plan

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

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

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 ↗	(On Diff #49250)	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 ↗	(On Diff #49250)	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 49274

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

Loading...