fix banner as well

Headpat-Community · May 22, 2024 · 65627c0 · 65627c0
1 parent 0888e79
commit 65627c0
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 7 deletions.
diff --git a/src/app/(account)/account/profile/frontpage/uploadAvatar.tsx b/src/app/(account)/account/profile/frontpage/uploadAvatar.tsx
@@ -1,6 +1,6 @@
 'use client'
 import { useToast } from '@/components/ui/use-toast'
-import { UserAvatarsDocumentType, UserDataDocumentsType } from '@/utils/types'
+import { UserDataDocumentsType } from '@/utils/types'
 import { databases, ID, storage } from '@/app/appwrite-client'
 import * as Sentry from '@sentry/nextjs'
 import { Input } from '@/components/ui/input'
@@ -61,7 +61,7 @@ export default function UploadAvatar({
       // If the user already has an avatar, delete it
       if (avatarDocument.avatarId) {
         // Delete the old avatar
-        const data = await storage.deleteFile('avatars', avatarDocument.avatarId).catch((error) => {
+        await storage.deleteFile('avatars', avatarDocument.avatarId).catch((error) => {
           //console.error(error)
           Sentry.captureException('Failed to delete old avatar.', error)
         })

diff --git a/src/app/(account)/account/profile/frontpage/uploadBanner.tsx b/src/app/(account)/account/profile/frontpage/uploadBanner.tsx
@@ -1,6 +1,6 @@
 'use client'
 import { useToast } from '@/components/ui/use-toast'
-import { UserAvatarsDocumentType } from '@/utils/types'
+import { UserAvatarsDocumentType, UserDataDocumentsType } from '@/utils/types'
 import { databases, ID, storage } from '@/app/appwrite-client'
 import * as Sentry from '@sentry/nextjs'
 import { Input } from '@/components/ui/input'
@@ -56,19 +56,21 @@ export default function UploadBanner({
       setIsUploading(true) // Set isUploading to true before making the API call
 
       // Get the user's banner document
-      const bannerDocument: UserAvatarsDocumentType =
+      const bannerDocument: UserDataDocumentsType =
         await databases.getDocument('hp_db', 'userdata', userMe.$id)
       // If the user already has an banner, delete it
       if (bannerDocument.profileBannerId) {
         // Delete the old banner
-        console.log('deleting')
-        await storage.deleteFile('banners', bannerDocument.profileBannerId)
+        await storage.deleteFile('banners', bannerDocument.profileBannerId).catch((error) => {
+          //console.error(error)
+          Sentry.captureException('Failed to delete old banner.', error)
+        })
       }
 
       // Upload the new banner
       const fileData = storage.createFile(
         'banners',
-        ID.unique(),
+        bannerDocument.$id,
         (document.getElementById('banner-upload') as HTMLInputElement).files[0]
       )