Fix user update and avatar/cover image assignment

mazed2020 · Dec 19, 2023 · 23b80aa · 23b80aa
1 parent 802aa88
commit 23b80aa
Showing 1 changed file with 8 additions and 1 deletion.
diff --git a/src/controllers/user.controller.js b/src/controllers/user.controller.js
@@ -264,7 +264,7 @@ const updateAccountDetails = asyncHandler(async(req, res) => {
         throw new ApiError(400, "All fields are required")
     }
 
-    const user = User.findByIdAndUpdate(
+    const user = await User.findByIdAndUpdate(
         req.user?._id,
         {
             $set: {
@@ -288,6 +288,8 @@ const updateUserAvatar = asyncHandler(async(req, res) => {
         throw new ApiError(400, "Avatar file is missing")
     }
 
+    //TODO: delete old image - assignment
+
     const avatar = await uploadOnCloudinary(avatarLocalPath)
 
     if (!avatar.url) {
@@ -319,6 +321,9 @@ const updateUserCoverImage = asyncHandler(async(req, res) => {
         throw new ApiError(400, "Cover image file is missing")
     }
 
+    //TODO: delete old image - assignment
+
+
     const coverImage = await uploadOnCloudinary(coverImageLocalPath)
 
     if (!coverImage.url) {
@@ -343,6 +348,8 @@ const updateUserCoverImage = asyncHandler(async(req, res) => {
     )
 })
 
+
+
 export {
     registerUser,
     loginUser,