Page MenuHomePhabricator

[lib] assign local id to report while creating
ClosedPublic

Authored by kamil on May 22 2023, 6:46 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Nov 10, 6:58 AM
Unknown Object (File)
Mon, Nov 4, 10:43 AM
Unknown Object (File)
Wed, Oct 23, 12:35 AM
Unknown Object (File)
Mon, Oct 21, 10:45 AM
Unknown Object (File)
Oct 15 2024, 7:50 AM
Unknown Object (File)
Sep 28 2024, 10:51 PM
Unknown Object (File)
Sep 28 2024, 10:51 PM
Unknown Object (File)
Sep 28 2024, 10:51 PM
Subscribers

Details

Summary

Adding id while adding new report to store.

Depends on D7912

Test Plan

Make sure each report type will have an assigned id.

After this, I made id field mandatory and checked for flow errors which should make sure that it was added in all places

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 23 2023, 1:40 AM
kamil edited the test plan for this revision. (Show Details)

It might be a good idea to create an intermediate function that generates the id. At this point that function would just return getUUID(), but if we ever decide to change id scheme, having this function would be really helpful.

This revision is now accepted and ready to land.May 23 2023, 3:15 AM
kamil edited the test plan for this revision. (Show Details)

add intermediate function