[web] avoid calling worker with empty store ops
ClosedPublic
Actions

Authored by kamil on Dec 12 2023, 12:53 AM.

Tags

None

Referenced Files

	F2894010: D10300.diff
	Fri, Oct 4, 2:15 PM

	F2887489: D10300.diff
	Fri, Oct 4, 2:37 AM

	F2887126: D10300.id.diff
	Fri, Oct 4, 1:37 AM

	F2886559: D10300.id34706.diff
	Fri, Oct 4, 12:15 AM

	F2886550: D10300.id34509.diff
	Fri, Oct 4, 12:09 AM

	Unknown Object (File)
	Fri, Sep 6, 12:46 AM

	Unknown Object (File)
	Fri, Sep 6, 12:46 AM

	Unknown Object (File)
	Fri, Sep 6, 12:46 AM

View All 43 Files

Subscribers

ashoat

Details

Reviewers

tomek
michal

Commits

rCOMM221149dec99e: [web] avoid calling worker with empty store ops

Summary

That was previously the way it worked, I removed it when refactoring which was not so good, so bringing it back.

Test Plan

Thread, Draft and report ops worker.
Other are not scheduled on worker.

Diff Detail

Repository

rCOMM Comm

Branch

not-init-fixed

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

kamil created this revision.Dec 12 2023, 12:53 AM

kamil held this revision as a draft.

Herald added a subscriber: ashoat. · View Herald TranscriptDec 12 2023, 12:53 AM

Harbormaster completed remote builds in B24971: Diff 34509.Dec 12 2023, 1:21 AM

kamil published this revision for review.Dec 13 2023, 2:03 AM

tomek accepted this revision.Dec 13 2023, 4:52 AM

This revision is now accepted and ready to land.Dec 13 2023, 4:52 AM

Closed by commit rCOMM221149dec99e: [web] avoid calling worker with empty store ops (authored by kamil). · Explain WhyDec 15 2023, 5:37 AM

This revision was automatically updated to reflect the committed changes.

kamil added a commit: rCOMM221149dec99e: [web] avoid calling worker with empty store ops.

Revision Contents
Changeset List

Path

Size

web/

database/

utils/

store.js

8 lines

Diff 34509

View Options

web/database/utils/store.js

[web] avoid calling worker with empty store opsClosedPublicActions