@things-factory/dataset 7.0.1-rc.9 → 7.0.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@things-factory/dataset",
-  "version": "7.0.1-rc.9",
+  "version": "7.0.2",
   "main": "dist-server/index.js",
   "browser": "dist-client/index.js",
   "things-factory": true,
@@ -27,34 +27,34 @@
     "migration:create": "node ../../node_modules/typeorm/cli.js migration:create -d ./server/migrations"
   },
   "dependencies": {
-    "@operato/app": "^7.0.0-rc",
-    "@operato/data-grist": "^7.0.0-rc",
-    "@operato/dataset": "^7.0.0-rc",
-    "@operato/ghost-print": "^7.0.0-rc",
-    "@operato/graphql": "^7.0.0-rc",
-    "@operato/grist-editor": "^7.0.0-rc",
-    "@operato/i18n": "^7.0.0-rc",
-    "@operato/layout": "^7.0.0-rc",
-    "@operato/moment-timezone-es": "^7.0.0-rc",
-    "@operato/p13n": "^7.0.0-rc",
-    "@operato/shell": "^7.0.0-rc",
-    "@operato/styles": "^7.0.0-rc",
-    "@operato/utils": "^7.0.0-rc",
-    "@things-factory/auth-base": "^7.0.1-rc.9",
-    "@things-factory/aws-base": "^7.0.1-rc.9",
-    "@things-factory/board-service": "^7.0.1-rc.9",
-    "@things-factory/env": "^7.0.1-rc.8",
-    "@things-factory/integration-base": "^7.0.1-rc.9",
-    "@things-factory/organization": "^7.0.1-rc.9",
-    "@things-factory/personalization": "^7.0.1-rc.9",
-    "@things-factory/scheduler-client": "^7.0.1-rc.9",
-    "@things-factory/shell": "^7.0.1-rc.8",
-    "@things-factory/work-shift": "^7.0.1-rc.9",
-    "@things-factory/worklist": "^7.0.1-rc.9",
+    "@operato/app": "^7.0.0",
+    "@operato/data-grist": "^7.0.0",
+    "@operato/dataset": "^7.0.0",
+    "@operato/ghost-print": "^7.0.0",
+    "@operato/graphql": "^7.0.0",
+    "@operato/grist-editor": "^7.0.0",
+    "@operato/i18n": "^7.0.0",
+    "@operato/layout": "^7.0.0",
+    "@operato/moment-timezone-es": "^7.0.0",
+    "@operato/p13n": "^7.0.0",
+    "@operato/shell": "^7.0.0",
+    "@operato/styles": "^7.0.0",
+    "@operato/utils": "^7.0.0",
+    "@things-factory/auth-base": "^7.0.2",
+    "@things-factory/aws-base": "^7.0.2",
+    "@things-factory/board-service": "^7.0.2",
+    "@things-factory/env": "^7.0.0",
+    "@things-factory/integration-base": "^7.0.2",
+    "@things-factory/organization": "^7.0.2",
+    "@things-factory/personalization": "^7.0.2",
+    "@things-factory/scheduler-client": "^7.0.2",
+    "@things-factory/shell": "^7.0.2",
+    "@things-factory/work-shift": "^7.0.2",
+    "@things-factory/worklist": "^7.0.2",
     "cron-parser": "^4.3.0",
     "moment-timezone": "^0.5.45",
     "simple-statistics": "^7.8.3",
     "statistics": "^3.3.0"
   },
-  "gitHead": "70c4b8737f2ccb632699f31a92173401624ee9f7"
+  "gitHead": "19f0e3097a5b583831ae530d8ff4138ca50d1619"
 }

package/server/activities/activity-ooc-review.ts

@@ -1,3 +1,4 @@
+import { In } from 'typeorm'
 import { Activity, ActivityInstance, ActivityInstanceStatus, UpdateActivityInstanceStateAddendum } from '@things-factory/worklist'
 import { issue } from '@things-factory/worklist/dist-server/controllers/activity-instance/issue'
 
@@ -10,7 +11,7 @@ async function callback(activityInstance: ActivityInstance, addendum: UpdateActi
 
   if (state == ActivityInstanceStatus.Ended) {
     const activity = (await tx.getRepository(Activity).findOneBy({
-      domain: { id: domain.id },
+      domain: domain.parentId ? In([domain.id, domain.parentId]) : { id: domain.id },
       name: 'OOC Resolve'
     })) as Activity
 

package/server/controllers/create-data-sample.ts

@@ -228,26 +228,31 @@ export async function createDataSample(newDataSample: NewDataSample, context: Re
       })
 
       if (scenario) {
-        scenario.start(scenario.name + ':' + dataSet.name + ':' + dataSample.id, {
-          dataOocId: dataOoc.id,
-          dataSampleId: dataSample.id,
-          dataSet: dataSet.id,
-          data,
-          ooc,
-          oos,
-          judgment,
-          collectedAt,
-          workDate,
-          workShift,
-          domain: {
-            id: domain.id,
-            subdomain: domain.subdomain,
-            name: domain.name
-          },
-          updator: {
-            id: user.id,
-            email: user.email,
-            name: user.name
+        scenario.start({
+          instanceName: scenario.name + ':' + dataSet.name + ':' + dataSample.id,
+          domain,
+          user,
+          variables: {
+            dataOocId: dataOoc.id,
+            dataSampleId: dataSample.id,
+            dataSet: dataSet.id,
+            data,
+            ooc,
+            oos,
+            judgment,
+            collectedAt,
+            workDate,
+            workShift,
+            domain: {
+              id: domain.id,
+              subdomain: domain.subdomain,
+              name: domain.name
+            },
+            updator: {
+              id: user.id,
+              email: user.email,
+              name: user.name
+            }
           }
         })
       } else {
@@ -263,7 +268,7 @@ export async function createDataSample(newDataSample: NewDataSample, context: Re
     if (activity) {
       const assignee =
         dataSet.supervisoryRoleId &&
-        (await tx.getRepository(Role).findOneBy({ domain: { id: domain.id }, id: dataSet.supervisoryRoleId }))
+        (await tx.getRepository(Role).findOneBy({ domain: { id: In([domain.id, domain.parentId].filter(Boolean)) }, id: dataSet.supervisoryRoleId }))
 
       const assignees = dataSet.supervisoryRoleId ? [{ type: 'Role', value: dataSet.supervisoryRoleId, assignee }] : []
 
@@ -289,7 +294,7 @@ export async function createDataSample(newDataSample: NewDataSample, context: Re
         )
       }
     } else {
-      console.error('OOC Review Activity not installed.')
+      console.warn('OOC Review Activity not installed.')
     }
 
     try {
@@ -315,32 +320,37 @@ export async function createDataSample(newDataSample: NewDataSample, context: Re
     if (normalScenarioId) {
       const scenario = await tx.getRepository(Scenario).findOne({
         where: {
-          domain: { id: In([domain.id, domain.parentId]) },
+          domain: { id: In([domain.id, domain.parentId].filter(Boolean)) },
           id: normalScenarioId
         },
         relations: ['domain', 'steps', 'updater']
       })
 
       if (scenario) {
-        scenario.start(scenario.name + ':' + dataSet.name + ':' + dataSample.id, {
-          dataSampleId: dataSample.id,
-          dataSet: dataSet.id,
-          data,
-          ooc,
-          oos,
-          judgment,
-          collectedAt,
-          workDate,
-          workShift,
-          domain: {
-            id: domain.id,
-            subdomain: domain.subdomain,
-            name: domain.name
-          },
-          updator: {
-            id: user.id,
-            email: user.email,
-            name: user.name
+        scenario.start({
+          instanceName: scenario.name + ':' + dataSet.name + ':' + dataSample.id, 
+          domain,
+          user,
+          variables: {
+            dataSampleId: dataSample.id,
+            dataSet: dataSet.id,
+            data,
+            ooc,
+            oos,
+            judgment,
+            collectedAt,
+            workDate,
+            workShift,
+            domain: {
+              id: domain.id,
+              subdomain: domain.subdomain,
+              name: domain.name
+            },
+            updator: {
+              id: user.id,
+              email: user.email,
+              name: user.name
+            }
           }
         })
       } else {
@@ -350,14 +360,14 @@ export async function createDataSample(newDataSample: NewDataSample, context: Re
 
     if (dataSet.requiresReview) {
       const activity = (await tx.getRepository(Activity).findOneBy({
-        domain: { id: domain.id },
+        domain: { id: In([domain.id, domain.parentId].filter(Boolean)) },
         name: 'Data Review'
       })) as Activity
 
       if (activity) {
         const assignee =
           dataSet.supervisoryRoleId &&
-          (await tx.getRepository(Role).findOneBy({ domain: { id: domain.id }, id: dataSet.supervisoryRoleId }))
+          (await tx.getRepository(Role).findOneBy({ domain: { id: In([domain.id, domain.parentId].filter(Boolean)) }, id: dataSet.supervisoryRoleId }))
 
         const assignees = dataSet.supervisoryRoleId
           ? [{ type: 'Role', value: dataSet.supervisoryRoleId, assignee }]

package/server/controllers/issue-collect-data.ts

@@ -27,7 +27,7 @@ export async function issueCollectData(
     })
 
     const activity = (await tx.getRepository(Activity).findOneBy({
-      domain: { id: domain.id },
+      domain: { id: In([domain.id, domain.parentId].filter(Boolean)) },
       name: 'Collect Data'
     })) as Activity
 

package/server/service/data-set/data-set-query.ts

@@ -69,7 +69,8 @@ export class DataSetQuery {
       where: { id: user.id },
       relations: ['roles']
     })
-    const roles = user.roles.filter(role => role.domainId === domain.id).map(role => role.id)
+
+    const roles = user.roles.filter(role => role.domainId === domain.id || (domain.parentId && role.domainId === domain.parentId)).map(role => role.id)
 
     if (!roles.length) {
       return { items: [], total: 0 }