fix: comments

Signed-off-by: Uroš Marolt <uros@marolt.me>

Author: themarolt

services/apps/data_sink_worker/src/service/activity.service.ts

@@ -1068,7 +1068,7 @@ export default class ActivityService extends LoggerBase {
         segmentIds: Set<string>
         resultIds: Set<string>
         integrationId: string
-        platform: string
+        platform: PlatformType
         username: string
         timestamp: string
       }
@@ -1144,7 +1144,7 @@ export default class ActivityService extends LoggerBase {
               organizations: value.member.organizations,
               reach: value.member.reach,
             },
-            value.platform as PlatformType,
+            value.platform,
           )
           .then((memberId) => {
             // map ids for members

services/libs/common/src/member.ts

@@ -14,12 +14,7 @@ export async function setAttributesDefaultValues(
   for (const attributeName of Object.keys(attributes)) {
     if (typeof attributes[attributeName] === 'string') {
       // we try to fix it
-      try {
-        attributes[attributeName] = JSON.parse(attributes[attributeName] as string)
-      } catch (err) {
-        this.log.error(err, { attributeName }, 'Could not parse a string attribute value!')
-        throw err
-      }
+      attributes[attributeName] = JSON.parse(attributes[attributeName] as string)
     }
 
     const nonEmptyPlatform = Object.keys(attributes[attributeName]).filter((p) => {