OUT-2650-re: proper error handling

src/features/sync/components/Table.tsx

@@ -83,7 +83,7 @@ const MappingTableRow = ({ openConfirmDialog }: { openConfirmDialog: (id?: strin
                   mapItem.fileChannelValue?.[0],
                 )}
                 onChange={(val) => onUserSelectorValueChange(val, index)}
-                disabled={!!mapItem.status}
+                disabled={!!mapItem?.id || mapItem.status === null}
                 options={unselectedChannelList || {}}
               />
             </td>
@@ -93,7 +93,7 @@ const MappingTableRow = ({ openConfirmDialog }: { openConfirmDialog: (id?: strin
                 onChange={(val) => onDropboxFolderChange(val, index)}
                 options={tempFolders || []}
                 placeholder="Search Dropbox folder"
-                disabled={!!mapItem.status}
+                disabled={!!mapItem?.id || mapItem.status === null}
               />
             </td>
             <td className="w-[150px] whitespace-nowrap px-6 py-2 text-gray-500 text-sm">

src/features/sync/lib/MapFiles.service.ts

@@ -1,4 +1,4 @@
-import { and, asc, eq, isNotNull, isNull, sql } from 'drizzle-orm'
+import { and, asc, eq, isNotNull, isNull, or, sql } from 'drizzle-orm'
 import httpStatus from 'http-status'
 import { ApiError } from 'node_modules/copilot-node-sdk/dist/codegen/api'
 import z from 'zod'
@@ -16,6 +16,7 @@ import {
   type FileSyncUpdatePayload,
   fileFolderSync,
 } from '@/db/schema/fileFolderSync.schema'
+import APIError from '@/errors/APIError'
 import type {
   DropboxFileListFolderResultEntries,
   MapList,
@@ -189,7 +190,11 @@ export class MapFilesService extends AuthenticatedDropboxService {
       .where(
         and(
           eq(channelSync.portalId, this.user.portalId),
-          eq(channelSync.assemblyChannelId, payload.assemblyChannelId),
+          or(
+            eq(channelSync.assemblyChannelId, payload.assemblyChannelId),
+            eq(channelSync.dbxRootPath, payload.dbxRootPath),
+          ),
+          isNull(channelSync.deletedAt),
         ),
       )
     logger.info('MapFilesService#getOrCreateChannelMap :: Got channel map', channel)
@@ -200,6 +205,15 @@ export class MapFilesService extends AuthenticatedDropboxService {
         .returning()
       channel = newChannel[0]
       logger.info('MapFilesService#getOrCreateChannelMap :: Created channel map', channel)
+    } else {
+      if (
+        channel.assemblyChannelId !== payload.assemblyChannelId ||
+        channel.dbxRootPath !== payload.dbxRootPath
+      )
+        throw new APIError(
+          'Mapping already exists for the given file channel or dropbox folder',
+          httpStatus.BAD_REQUEST,
+        )
     }
 
     return channel

src/helper/fetcher.helper.ts

@@ -3,27 +3,35 @@ export const postFetcher = async (
   headers: Record<string, string>,
   options: Record<string, string>,
 ) => {
-  return await fetch(url, {
+  const response = await fetch(url, {
     method: 'POST',
     headers: {
       'Content-Type': 'application/json',
       ...headers,
     },
     ...options,
   })
+  if (!response.ok) {
+    throw new Error('Failed to post data')
+  }
+  return response
 }
 
 export const deleteFetcher = async (
   url: string,
   headers: Record<string, string>,
   options: Record<string, string>,
 ) => {
-  return await fetch(url, {
+  const response = await fetch(url, {
     method: 'DELETE',
     headers: {
       'Content-Type': 'application/json',
       ...headers,
     },
     ...options,
   })
+  if (!response.ok) {
+    throw new Error('Failed to delete data')
+  }
+  return response
 }