Page MenuHomePhabricator

[web-db] implement SQL queries for report store
ClosedPublic

Authored by kamil on May 25 2023, 10:28 AM.
Tags
None
Referenced Files
Unknown Object (File)
Nov 10 2024, 6:58 AM
Unknown Object (File)
Oct 4 2024, 2:04 AM
Unknown Object (File)
Sep 29 2024, 1:18 AM
Unknown Object (File)
Sep 29 2024, 1:18 AM
Unknown Object (File)
Sep 29 2024, 1:18 AM
Unknown Object (File)
Sep 29 2024, 1:17 AM
Unknown Object (File)
Sep 29 2024, 1:10 AM
Unknown Object (File)
Sep 13 2024, 11:29 PM
Subscribers

Details

Summary

Mapping native structure and logic

Depends on D7982

Test Plan

Run tests

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kamil held this revision as a draft.
kamil published this revision for review.May 25 2023, 1:46 PM
michal added inline comments.
web/database/queries/report-queries.test.js
25 ↗(On Diff #27068)

Shouldn't we setupSQLiteDB in the beforeAll?

76 ↗(On Diff #27068)

I think you missed the name

This revision is now accepted and ready to land.May 30 2023, 8:26 AM

add missing name

web/database/queries/report-queries.test.js
25 ↗(On Diff #27068)

because tests are concentrated on the reports table (created via createReportTable()) it doesn't really make any difference, I'll remove this

76 ↗(On Diff #27068)

oversight, fixing