Skip to content

Commit 98970cf

Browse files
committed
fix: fix heroku file upload bug
1 parent 4b0c0e7 commit 98970cf

File tree

6 files changed

+15
-6
lines changed

6 files changed

+15
-6
lines changed

middlewares/file-upload/uploadImage.middleware.js

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,13 @@ const getImageExtension = require('../../utils/getImageExtension');
66
// Configuring and validating the upload
77
const storage = multer.diskStorage({
88
destination: (req, file, callbackFunction) => {
9-
callbackFunction(null, 'public/uploads');
9+
const { isUserImg } = req.body;
10+
11+
if (file?.fieldname === 'profileImage') {
12+
callbackFunction(null, 'public/uploads/users');
13+
} else {
14+
callbackFunction(null, 'public/uploads');
15+
}
1016
},
1117

1218
// By default, multer removes file extensions so let's add them back
5.05 KB
Loading
5.05 KB
Loading

routes/admin.route.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ router.get('/users', isAuth, isAdmin, paginationMiddleware(UserModel), adminCont
4444

4545
router.post(
4646
'/users',
47-
uploadImage.single('productImage'),
47+
uploadImage.single('profileImage'),
4848
isAuth,
4949
isAdmin,
5050
adminValidation.signupValidation(),
@@ -74,7 +74,7 @@ router.get('/users/:userId', isAuth, isAdmin, adminValidation.validateID, adminC
7474

7575
router.patch(
7676
'/users/:userId',
77-
uploadImage.single('productImage'),
77+
uploadImage.single('profileImage'),
7878
isAuth,
7979
isAdmin,
8080
adminValidation.validateID,

services/admin.service.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -202,7 +202,7 @@ exports.updateUser = async (req, res, next) => {
202202
user.nationality = nationality || user.nationality;
203203
user.address = address || user.address;
204204
user.favoriteAnimal = favoriteAnimal || user.favoriteAnimal;
205-
user.profileImage = req.file.filename ? `/static/uploads/${req.file.filename}` : user.profileImage;
205+
user.profileImage = req.file.filename ? `/static/uploads/users/${req.file.filename}` : user.profileImage;
206206

207207
const updatedUser = await user.save();
208208

services/auth.service.js

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@ const { WEBSITE_URL, API_VERSION, CLIENT_URL } = require('../configs/environment
1616
*/
1717

1818
exports.signup = async (req, res, next) => {
19+
console.log('error');
1920
const {
2021
firstName,
2122
lastName,
@@ -48,6 +49,7 @@ exports.signup = async (req, res, next) => {
4849
return responseObject;
4950
}
5051

52+
// req.file.filename ? `/static/uploads/users/${req.file.filename}` : user.profileImage;
5153
const newUser = new User({
5254
_id: new mongoose.Types.ObjectId(),
5355
firstName,
@@ -60,9 +62,10 @@ exports.signup = async (req, res, next) => {
6062
cart,
6163
role,
6264
acceptTerms,
63-
profileImage: `/static/uploads/${req.file.filename}`
65+
profileImage: `/static/uploads/users/${req.file.filename}`
6466
});
6567

68+
console.log(' newUser', newUser);
6669
try {
6770
const user = await newUser.save();
6871
let token = await new Token({ userId: user._id });
@@ -535,7 +538,7 @@ exports.updateUser = async (req, res, next) => {
535538
user.nationality = nationality || user.nationality;
536539
user.address = address || user.address;
537540
user.favoriteAnimal = favoriteAnimal || user.favoriteAnimal;
538-
user.profileImage = req.file.filename ? `/static/uploads/${req.file.filename}` : user.profileImage;
541+
user.profileImage = req.file.filename ? `/static/uploads/users/${req.file.filename}` : user.profileImage;
539542

540543
const updatedUser = await user.save();
541544

0 commit comments

Comments
 (0)