manjit-hub / Notorite

https://notorite.vercel.app
MIT License
10 stars 31 forks source link

Fix Note Routes and Controller #44

Closed Bluesparx closed 1 month ago

Bluesparx commented 1 month ago

Implement the following:

  1. Remove use of disk storage in notes router.
  2. Use middleware for image upload in notes controller.
manjit-hub commented 1 month ago

Same here explain how you will ressolve these. What outcome we will get ?

Shivansh-0403 commented 1 month ago

@manjit-hub there were some issues in notes controller because the JWT token was not getting authenticated properly and the file was not getting uploaded. So, I have worked on that issue. Also slight enhancement to display the name of the creater of a note on searching for notes. And, one doubt, do you want to upload the document coming from uploadNote API on cloudinary or simply in the disk storage using multer, because the current codebase of your repo doesn't upload the note on cloudinary. So, I can add that as well.

manjit-hub commented 1 month ago

@Shivansh-0403 go for it. And make uplod to Cloudinary.