Merge pull request #352 from cofacts/add-userid-in-bq

Add userid in bq analytics
cofacts · May 19, 2023 · 427511a · 427511a
2 parents afa6679 + 0911382
commit 427511a
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 1 deletion.
diff --git a/src/lib/__tests__/ga.js b/src/lib/__tests__/ga.js
@@ -71,6 +71,7 @@ it('returns visitor', () => {
           "extra": "{}",
           "messageSource": "user",
           "text": "",
+          "userId": "userId",
         },
       ],
     ]
@@ -109,6 +110,7 @@ it('sets title when title is given', () => {
       "events": Array [],
       "extra": "{}",
       "messageSource": "room",
+      "userId": "userId",
     }
   `);
   // Expect title is sent to BigQuery
@@ -160,6 +162,7 @@ it('sets events', () => {
           "extra": "{}",
           "messageSource": "user",
           "text": "",
+          "userId": "userId",
         },
       ],
     ]

diff --git a/src/lib/ga.ts b/src/lib/ga.ts
@@ -52,8 +52,10 @@ export default function ga(
       });
       return visitor.event(evt);
     },
+
     send() {
       insertEventBatch({
+        userId: uuid,
         text: documentTitle,
         messageSource,
         events,

diff --git a/src/lineContent.ts b/src/lineContent.ts
@@ -31,6 +31,7 @@ lineContentRouter.get('/', async (ctx) => {
   visitor.send();
 
   insertEventBatch({
+    userId: null,
     createdAt: new Date(),
     text: null,
     messageSource: null,

diff --git a/src/rumors-db b/src/rumors-db