Page MenuHomePhabricator

[lib] refactor Report Store Operations to use generic type
ClosedPublic

Authored by kamil on Jul 25 2023, 6:01 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Nov 23, 6:56 PM
Unknown Object (File)
Sat, Nov 23, 6:56 PM
Unknown Object (File)
Sat, Nov 23, 4:21 PM
Unknown Object (File)
Sat, Nov 23, 4:21 PM
Unknown Object (File)
Sat, Nov 23, 4:21 PM
Unknown Object (File)
Wed, Nov 20, 5:41 PM
Unknown Object (File)
Wed, Nov 20, 5:41 PM
Unknown Object (File)
Wed, Nov 20, 5:41 PM
Subscribers

Details

Summary

Make logic part of the general ops interface

Depends on D8619

Test Plan

Flow

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kamil held this revision as a draft.
kamil retitled this revision from [lib] reactor Report Store Operations to use generic type to [lib] refactor Report Store Operations to use generic type.Jul 25 2023, 6:27 AM
kamil published this revision for review.Jul 25 2023, 6:35 AM
atul added inline comments.
lib/ops/report-store-ops.test.js
58 ↗(On Diff #29018)

Still think we should call this reportStoreOps since the operations are related to how reports are stored and we're not "operating" on the reports themselves?

This revision is now accepted and ready to land.Jul 26 2023, 12:07 AM

rename

lib/ops/report-store-ops.test.js
58 ↗(On Diff #29018)

I updated this because reportStoreOps was defined in the upper scope, but since it'll be updated I can keep old name