[lib] add report store ops on JS side
ClosedPublic
Actions

Authored by kamil on May 22 2023, 10:46 AM.

Details

Reviewers

tomek
michal

Commits

rCOMM281a00b7b442: [lib] add report store ops on JS side

Summary

All operation types and function with processing those.

Depends on D7919

Test Plan

Run tests

Diff Detail

Repository

rCOMM Comm

Branch

reports-native

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

kamil created this revision.May 22 2023, 10:46 AM

kamil held this revision as a draft.

Herald added subscribers: atul, ashoat. · View Herald TranscriptMay 22 2023, 10:46 AM

kamil added a child revision: D7921: [lib] refactor types to allow returning report store operations.May 22 2023, 10:54 AM

Harbormaster completed remote builds in B19609: Diff 26802.May 22 2023, 11:04 AM

kamil published this revision for review.May 23 2023, 1:49 AM

tomek accepted this revision.May 24 2023, 2:06 AM

tomek added inline comments.

lib/ops/report-store-ops.js
7	Should this be readonly?

This revision is now accepted and ready to land.May 24 2023, 2:06 AM

make field read-only

lib/ops/report-store-ops.js
7	yes, fixing

Harbormaster completed remote builds in B19812: Diff 27131.May 29 2023, 4:00 AM

Closed by commit rCOMM281a00b7b442: [lib] add report store ops on JS side (authored by kamil). · Explain WhyMay 29 2023, 4:18 AM

This revision was automatically updated to reflect the committed changes.

kamil added a commit: rCOMM281a00b7b442: [lib] add report store ops on JS side.

Revision Contents
Changeset List

Path

Size

lib/

ops/

report-store-ops.js

49 lines

report-store-ops.test.js

180 lines

Diff 26802

View Options

lib/ops/report-store-ops.js