Merge pull request #52 from remap-keys/support-organization-for-create-keyboard-statistics-command

Add a logic to check whether the user belongs to the organization which is the target keyboard

Author: yoichiro

functions/src/keyboards/create-keyboard-statistics-command.ts

@@ -132,12 +132,27 @@ export class CreateKeyboardStatisticsCommand extends AbstractCommand<ICreateKeyb
       };
     }
     const keyboardDefinition = keyboardDefinitionSnapshot.data()!;
-    if (keyboardDefinition.author_uid !== uid) {
-      return {
-        success: false,
-        errorCode: ERROR_KEYBOARD_DEFINITION_NOT_FOUND,
-        errorMessage: `The user is not an owner of the keyboard definition ${keyboardDefinitionId}.`,
-      };
+    if (keyboardDefinition.author_type === 'organization') {
+      const organizationId = keyboardDefinition.organization_id;
+      const result = await this.checkUserIsOrganizationMember(
+        uid,
+        organizationId
+      );
+      if (!result) {
+        return {
+          success: false,
+          errorCode: ERROR_KEYBOARD_DEFINITION_NOT_FOUND,
+          errorMessage: `The user is not a member of the organization ${organizationId}.`,
+        };
+      }
+    } else {
+      if (keyboardDefinition.author_uid !== uid) {
+        return {
+          success: false,
+          errorCode: ERROR_KEYBOARD_DEFINITION_NOT_FOUND,
+          errorMessage: `The user is not an owner of the keyboard definition ${keyboardDefinitionId}.`,
+        };
+      }
     }
     return {
       success: true,