Merge pull request #50 from remap-keys/fix-statistics-issue

yoichiro · web-flow · commit c168a22928e1 · 2023-12-19T11:10:02.000+09:00
Fix the creating statistics issue.
diff --git a/functions/src/keyboards/create-keyboard-statistics-command.ts b/functions/src/keyboards/create-keyboard-statistics-command.ts
@@ -122,11 +122,11 @@ export class CreateKeyboardStatisticsCommand extends AbstractCommand<ICreateKeyb
       };
     }
     const keyboardDefinition = keyboardDefinitionSnapshot.data()!;
-    if (keyboardDefinition.uid !== uid) {
+    if (keyboardDefinition.author_uid !== uid) {
       return {
         success: false,
         errorCode: ERROR_KEYBOARD_DEFINITION_NOT_FOUND,
-        errorMessage: `The keyboard definition ${keyboardDefinitionId} is not found.`,
+        errorMessage: `The user is not an owner of the keyboard definition ${keyboardDefinitionId}.`,
       };
     }
     return {

Original file line number	Diff line number	Diff line change
`@@ -122,11 +122,11 @@ export class CreateKeyboardStatisticsCommand extends AbstractCommand<ICreateKeyb`
`122`	`122`	`};`
`123`	`123`	`}`
`124`	`124`	`const keyboardDefinition = keyboardDefinitionSnapshot.data()!;`
`125`		`- if (keyboardDefinition.uid !== uid) {`
	`125`	`+ if (keyboardDefinition.author_uid !== uid) {`
`126`	`126`	`return {`
`127`	`127`	`success: false,`
`128`	`128`	`errorCode: ERROR_KEYBOARD_DEFINITION_NOT_FOUND,`
`129`		- errorMessage: `The keyboard definition ${keyboardDefinitionId} is not found.`,
	`129`	+ errorMessage: `The user is not an owner of the keyboard definition ${keyboardDefinitionId}.`,
`130`	`130`	`};`
`131`	`131`	`}`
`132`	`132`	`return {`