fix: handle resource upload errors in frontend and add unsupported er…

…ror when aws bucket is not configured
asdlei99 · Jun 20, 2021 · 404d0d7 · 404d0d7
1 parent 7a2dc21
commit 404d0d7
Show file tree

Hide file tree

Showing 4 changed files with 18 additions and 1 deletion.
diff --git a/frontend/src/shared/components/FileUploadModal.tsx b/frontend/src/shared/components/FileUploadModal.tsx
@@ -99,7 +99,9 @@ export const FileUploadModal = (props: Props) => {
         );
         const resource = res.getValue().resource;
         resources.push(resource);
-      } catch (error) {}
+      } catch (error) {
+        alert(error?.response?.data?.message);
+      }
     }
     props.onDone({
       uploadedResources: resources,

diff --git a/server/src/modules/meeting/useCases/createResource/CreateResourceController.ts b/server/src/modules/meeting/useCases/createResource/CreateResourceController.ts
@@ -36,6 +36,8 @@ export class CreateResourceController extends BaseController<BodySchema, PathPar
                         return res.forbidden(e.message);
                     case CreateResourceUseCaseErrors.MeetingNotFoundError:
                         return res.notFound(e.message);
+                    case CreateResourceUseCaseErrors.ResourceUploadNotSupportedError:
+                        return res.forbidden(e.message);
                     case CreateResourceUseCaseErrors.CanvasNotFoundError:
                         return res.notFound(e.message);
                     default:

diff --git a/server/src/modules/meeting/useCases/createResource/CreateResourceErrors.ts b/server/src/modules/meeting/useCases/createResource/CreateResourceErrors.ts
@@ -25,4 +25,12 @@ export namespace CreateResourceUseCaseErrors {
             } as UseCaseError);
         }
     }
+
+    export class ResourceUploadNotSupportedError extends Result<UseCaseError> {
+        constructor() {
+            super(false, {
+                message: `Resource upload is not supported on this server.`,
+            } as UseCaseError);
+        }
+    }
 }
diff --git a/server/src/modules/meeting/useCases/createResource/CreateResourceUseCase.ts b/server/src/modules/meeting/useCases/createResource/CreateResourceUseCase.ts
@@ -1,3 +1,4 @@
+import { awsConfig } from '../../../../config';
 import { AppError } from '../../../../shared/core/AppError';
 import { Either, left, right } from '../../../../shared/core/Result';
 import { UseCase } from '../../../../shared/core/UseCase';
@@ -40,6 +41,10 @@ export class CreateResourceUseCase implements UseCase<CreateResourceDTO, Promise
                 return left(new CreateResourceUseCaseErrors.CanvasNotFoundError(request.meetingId, request.canvasId));
             }
 
+            if (!awsConfig.mediaBucket) {
+                return left(new CreateResourceUseCaseErrors.ResourceUploadNotSupportedError());
+            }
+
             const resourceId = new UniqueEntityID();
 
             const key = `meetings/${meeting.meetingId.toString()}/resources/${resourceId.toString()}`;