fix: resolve pr review comments

Signed-off-by: Yeganathan S <63534555+skwowet@users.noreply.github.com>

Author: skwowet

backend/src/services/member/memberOrganizationsService.ts

@@ -218,51 +218,49 @@ export default class MemberOrganizationsService extends LoggerBase {
   ): Promise<IRenderFriendlyMemberOrganization[]> {
     const transaction = await SequelizeRepository.createTransaction(this.options)
     const repositoryOptions = { ...this.options, transaction }
-
+  
     try {
       const qx = SequelizeRepository.getQueryExecutor(repositoryOptions)
 
       const existing = await fetchMemberOrganizationById(qx, id)
       if (!existing || existing.memberId !== memberId) {
         throw new Error404(`Member organization with id ${id} not found!`)
       }
-
+  
       const hasDateStart = data.dateStart !== undefined
       const hasDateEnd = data.dateEnd !== undefined
       const targetDateRange = sanitizeMemberOrganizationDateRange(
         hasDateStart ? data.dateStart : existing.dateStart,
         hasDateEnd ? data.dateEnd : existing.dateEnd,
         true,
       )
-
+  
       const update = lodash.pickBy(
         {
           organizationId: data.organizationId,
           title: data.title,
           dateStart: hasDateStart ? targetDateRange.dateStart : undefined,
           dateEnd: hasDateEnd ? targetDateRange.dateEnd : undefined,
-          source: data.source,
+
           verified: data.verified,
           verifiedBy: data.verifiedBy,
         },
         (v) => v !== undefined,
       ) as MemberOrganizationUpdate
 
-      await cleanSoftDeletedMemberOrganization(qx, memberId, data.organizationId, data)
-      // Any manual edit from the frontend promotes ownership to UI so automated
-      // sources (e.g. email-domain inference) no longer overwrite user intent.
+      await cleanSoftDeletedMemberOrganization(qx, memberId, data.organizationId, update)
       await updateMemberOrganization(qx, memberId, id, {
         ...update,
-        source: OrganizationSource.UI,
+        source: OrganizationSource.UI
       })
-
+  
       // Trigger recalculation for old and new orgs if changed
       const orgsToRecalculate = Array.from(
         new Set([existing.organizationId, data.organizationId]),
       ).filter((orgId): orgId is string => Boolean(orgId))
 
       await this.commonMemberService.startAffiliationRecalculation(memberId, orgsToRecalculate)
-
+  
       const result = await this.list(memberId, transaction)
 
       await SequelizeRepository.commitTransaction(transaction)

services/apps/members_enrichment_worker/src/activities/enrichment.ts

@@ -767,8 +767,8 @@ function sanitizeWorkExperienceDateRanges(
 
     return {
       ...org,
-      startDate: dates.dateStart ? String(dates.dateStart) : null,
-      endDate: dates.dateEnd ? String(dates.dateEnd) : null,
+      startDate: dates.dateStart instanceof Date ? dates.dateStart.toISOString() : dates.dateStart,
+      endDate: dates.dateEnd instanceof Date ? dates.dateEnd.toISOString() : dates.dateEnd,
     }
   })
 }