Added Requirement.createdBy property (#446)

final-hill · Nov 5, 2024 · eaf1875 · eaf1875
1 parent b349e47
commit eaf1875
Show file tree

Hide file tree

Showing 26 changed files with 102 additions and 8 deletions.
diff --git a/domain/requirements/Requirement.ts b/domain/requirements/Requirement.ts
@@ -21,6 +21,7 @@ export abstract class Requirement extends BaseEntity {
         this.lastModified = props.lastModified;
         this.modifiedBy = props.modifiedBy;
         this.isSilence = props.isSilence;
+        this.createdBy = props.createdBy;
     }
 
     // This fixes the issue with em.create not honoring the constructor signature
@@ -73,11 +74,15 @@ export abstract class Requirement extends BaseEntity {
     /**
      * The user who last modified the requirement
      */
-    // System Admin is the default user for the initial migration
-    // This can be removed in v0.14.0 or later
-    @ManyToOne({ entity: () => AppUser, default: 'ac594919-50e3-438a-b9bc-efb8a8654243' })
+    @ManyToOne({ entity: () => AppUser })
     modifiedBy!: AppUser;
 
+    /**
+     * The user who created the requirement
+     */
+    @ManyToOne({ entity: () => AppUser })
+    createdBy!: AppUser;
+
     /**
      * Whether the requirement is a silence requirement.
      * (i.e. a requirement that is not included in the solution)

diff --git a/migrations/.snapshot-cathedral.json b/migrations/.snapshot-cathedral.json
@@ -274,7 +274,15 @@
           "autoincrement": false,
           "primary": false,
           "nullable": false,
-          "default": "'ac594919-50e3-438a-b9bc-efb8a8654243'",
+          "mappedType": "uuid"
+        },
+        "created_by_id": {
+          "name": "created_by_id",
+          "type": "uuid",
+          "unsigned": false,
+          "autoincrement": false,
+          "primary": false,
+          "nullable": false,
           "mappedType": "uuid"
         },
         "is_silence": {
@@ -514,6 +522,18 @@
           "referencedTableName": "public.app_user",
           "updateRule": "cascade"
         },
+        "requirement_created_by_id_foreign": {
+          "constraintName": "requirement_created_by_id_foreign",
+          "columnNames": [
+            "created_by_id"
+          ],
+          "localTableName": "public.requirement",
+          "referencedColumnNames": [
+            "id"
+          ],
+          "referencedTableName": "public.app_user",
+          "updateRule": "cascade"
+        },
         "requirement_primary_actor_id_foreign": {
           "constraintName": "requirement_primary_actor_id_foreign",
           "columnNames": [

diff --git a/migrations/Migration20241105190126.ts b/migrations/Migration20241105190126.ts
@@ -0,0 +1,23 @@
+import { Migration } from '@mikro-orm/migrations';
+
+export class Migration20241105190126 extends Migration {
+    override async up(): Promise<void> {
+        this.addSql(`alter table "requirement" add column "created_by_id" uuid not null default '00000000-0000-0000-0000-000000000000';`);
+        this.addSql(`update "requirement" set "created_by_id" = "modified_by_id";`);
+        this.addSql(`alter table "requirement" alter column "created_by_id" drop default;`);
+        this.addSql(`alter table "requirement" alter column "modified_by_id" drop default;`);
+        this.addSql(`alter table "requirement" alter column "modified_by_id" drop default;`);
+        this.addSql(`alter table "requirement" alter column "modified_by_id" type uuid using ("modified_by_id"::text::uuid);`);
+        this.addSql(`alter table "requirement" add constraint "requirement_created_by_id_foreign" foreign key ("created_by_id") references "app_user" ("id") on update cascade;`);
+    }
+
+    override async down(): Promise<void> {
+        this.addSql(`alter table "requirement" drop constraint "requirement_created_by_id_foreign";`);
+
+        this.addSql(`alter table "requirement" drop column "created_by_id";`);
+
+        this.addSql(`alter table "requirement" alter column "modified_by_id" drop default;`);
+        this.addSql(`alter table "requirement" alter column "modified_by_id" type uuid using ("modified_by_id"::text::uuid);`);
+        this.addSql(`alter table "requirement" alter column "modified_by_id" set default 'ac594919-50e3-438a-b9bc-efb8a8654243';`);
+    }
+}
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
     "name": "@final-hill/cathedral",
-    "version": "0.18.1",
+    "version": "0.19.0",
     "description": "Requirements management system",
     "keywords": [],
     "private": true,
@@ -77,4 +77,4 @@
         "vitest": "^2.1.4",
         "vue-tsc": "^2.1.8"
     }
-}
+}
diff --git a/server/api/assumption/index.post.ts b/server/api/assumption/index.post.ts
@@ -23,6 +23,7 @@ export default defineEventHandler(async (event) => {
             reqId: await getNextReqId(assumptionReqIdPrefix, em, solution) as Assumption['reqId'],
             name,
             description,
+            createdBy: sessionUser,
             modifiedBy: sessionUser,
             lastModified: new Date(),
             isSilence

diff --git a/server/api/constraint/index.post.ts b/server/api/constraint/index.post.ts
@@ -25,6 +25,7 @@ export default defineEventHandler(async (event) => {
             name,
             description,
             category,
+            createdBy: sessionUser,
             lastModified: new Date(),
             modifiedBy: sessionUser,
             isSilence

diff --git a/server/api/effect/index.post.ts b/server/api/effect/index.post.ts
@@ -23,6 +23,7 @@ export default defineEventHandler(async (event) => {
             reqId: await getNextReqId('E.5.', em, solution) as Effect['reqId'],
             name,
             description,
+            createdBy: sessionUser,
             modifiedBy: sessionUser,
             lastModified: new Date(),
             isSilence

diff --git a/server/api/environment-component/index.post.ts b/server/api/environment-component/index.post.ts
@@ -25,6 +25,7 @@ export default defineEventHandler(async (event) => {
             name,
             description,
             lastModified: new Date(),
+            createdBy: sessionUser,
             modifiedBy: sessionUser,
             isSilence
         })

diff --git a/server/api/epic/index.post.ts b/server/api/epic/index.post.ts
@@ -24,6 +24,7 @@ export default defineEventHandler(async (event) => {
             reqId: await getNextReqId('G.5.', em, solution) as Epic['reqId'],
             name,
             description,
+            createdBy: sessionUser,
             modifiedBy: sessionUser,
             lastModified: new Date(),
             priority,

diff --git a/server/api/functional-behavior/index.post.ts b/server/api/functional-behavior/index.post.ts
@@ -26,6 +26,7 @@ export default defineEventHandler(async (event) => {
             description,
             priority,
             lastModified: new Date(),
+            createdBy: sessionUser,
             modifiedBy: sessionUser,
             isSilence
         })

diff --git a/server/api/glossary-term/index.post.ts b/server/api/glossary-term/index.post.ts
@@ -25,6 +25,7 @@ export default defineEventHandler(async (event) => {
             name,
             description,
             lastModified: new Date(),
+            createdBy: sessionUser,
             modifiedBy: sessionUser,
             isSilence
         })

diff --git a/server/api/invariant/index.post.ts b/server/api/invariant/index.post.ts
@@ -24,6 +24,7 @@ export default defineEventHandler(async (event) => {
             name,
             description,
             modifiedBy: sessionUser,
+            createdBy: sessionUser,
             lastModified: new Date(),
             isSilence
         })

diff --git a/server/api/justification/index.post.ts b/server/api/justification/index.post.ts
@@ -22,6 +22,7 @@ export default defineEventHandler(async (event) => {
         reqId: undefined, // Should there be a Documentation category (D)?
         name,
         description,
+        createdBy: sessionUser,
         modifiedBy: sessionUser,
         lastModified: new Date(),
         isSilence

diff --git a/server/api/limit/index.post.ts b/server/api/limit/index.post.ts
@@ -24,6 +24,7 @@ export default defineEventHandler(async (event) => {
             name,
             description,
             modifiedBy: sessionUser,
+            createdBy: sessionUser,
             lastModified: new Date(),
             isSilence
         })

diff --git a/server/api/non-functional-behavior/index.post.ts b/server/api/non-functional-behavior/index.post.ts
@@ -27,6 +27,7 @@ export default defineEventHandler(async (event) => {
             priority,
             lastModified: new Date(),
             modifiedBy: sessionUser,
+            createdBy: sessionUser,
             isSilence
         })
 

diff --git a/server/api/obstacle/index.post.ts b/server/api/obstacle/index.post.ts
@@ -24,6 +24,7 @@ export default defineEventHandler(async (event) => {
             name,
             description,
             modifiedBy: sessionUser,
+            createdBy: sessionUser,
             lastModified: new Date(),
             isSilence
         })

diff --git a/server/api/organization/index.post.ts b/server/api/organization/index.post.ts
@@ -25,6 +25,7 @@ export default defineEventHandler(async (event) => {
         description,
         slug: slugify(name),
         lastModified: new Date(),
+        createdBy: sessionUser,
         modifiedBy: sessionUser,
         isSilence: false
     })

diff --git a/server/api/outcome/index.post.ts b/server/api/outcome/index.post.ts
@@ -24,6 +24,7 @@ export default defineEventHandler(async (event) => {
             name,
             description,
             modifiedBy: sessionUser,
+            createdBy: sessionUser,
             lastModified: new Date(),
             isSilence
         })