From cecd025a78f5c8a926cfe1b15e7766be2e615a18 Mon Sep 17 00:00:00 2001
From: pablohashescobar <118773738+pablohashescobar@users.noreply.github.com>
Date: Mon, 6 Mar 2023 18:56:05 +0530
Subject: [PATCH] feat: delete file assets from storage (#373)

---
 apiserver/plane/api/urls.py        |  5 +++++
 apiserver/plane/api/views/asset.py | 21 +++++++++++++++++++--
 2 files changed, 24 insertions(+), 2 deletions(-)

diff --git a/apiserver/plane/api/urls.py b/apiserver/plane/api/urls.py
index b73c71b28..77e628a9a 100644
--- a/apiserver/plane/api/urls.py
+++ b/apiserver/plane/api/urls.py
@@ -689,6 +689,11 @@ urlpatterns = [
         FileAssetEndpoint.as_view(),
         name="File Assets",
     ),
+    path(
+        "workspaces/<str:slug>/file-assets/<uuid:pk>/",
+        FileAssetEndpoint.as_view(),
+        name="File Assets",
+    ),
     ## End File Assets
     ## Modules
     path(
diff --git a/apiserver/plane/api/views/asset.py b/apiserver/plane/api/views/asset.py
index 657383553..e5af2c080 100644
--- a/apiserver/plane/api/views/asset.py
+++ b/apiserver/plane/api/views/asset.py
@@ -11,7 +11,6 @@ from plane.api.serializers import FileAssetSerializer
 
 
 class FileAssetEndpoint(BaseAPIView):
-
     parser_classes = (MultiPartParser, FormParser)
 
     """
@@ -27,7 +26,6 @@ class FileAssetEndpoint(BaseAPIView):
         try:
             serializer = FileAssetSerializer(data=request.data)
             if serializer.is_valid():
-
                 if request.user.last_workspace_id is None:
                     return Response(
                         {"error": "Workspace id is required"},
@@ -43,3 +41,22 @@ class FileAssetEndpoint(BaseAPIView):
                 {"error": "Something went wrong please try again later"},
                 status=status.HTTP_400_BAD_REQUEST,
             )
+
+    def delete(self, request, slug, pk):
+        try:
+            file_asset = FileAsset.objects.get(pk=pk, workspace__slug=slug)
+            # Delete the file from storage
+            file_asset.asset.delete(save=False)
+            # Delete the file object
+            file_asset.delete()
+            return Response(status=status.HTTP_204_NO_CONTENT)
+        except FileAsset.DoesNotExist:
+            return Response(
+                {"error": "File Asset doesn't exist"}, status=status.HTTP_404_NOT_FOUND
+            )
+        except Exception as e:
+            capture_exception(e)
+            return Response(
+                {"error": "Something went wrong please try again later"},
+                status=status.HTTP_400_BAD_REQUEST,
+            )