Added option to upload photo in onboarding.js

priyanshagra commented 1 year ago

Description step by step: ->Basically after setting up project locally ->connecting to my mongodb login -> then got to page onboarding.js -> then change the section of input image through url to desktop although donot change the name of input that is url ->sothat there might be any clashes in server to fetch the input image to store in database

Fixes # (issue) Added option to upload photo in onboarding.js

Type of change

[ x] Bug fix (non-breaking change which fixes an issue)
[ x] New feature (non-breaking change which adds functionality)
[ x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ x] This change requires a documentation update

Motivation and Context

This saves the time of user and increase comfort in using the web

How Has This Been Tested?

Tested multiple times please see screenshorts no effect in other area

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

Screenshots (if appropriate):

Checklist:

[ x] I have registered myself at Contrihub website.
[ x] My code follows the code style of this project.
[ x] I have commented my code, particularly in hard-to-understand areas
[ x] My changes generate no new warnings
[ x] My change requires a change to the documentation.
[ x] I have updated the documentation accordingly.
[ x] I have read the CONTRIBUTING document.
[ x] Any dependent changes have been merged and published in downstream modules
[ x] I have added tests to cover my changes.
[ x]

All new and existing tests passed.

Test Configuration:

Firmware version:
Hardware:
Toolchain:
SDK:

admirerr commented 1 year ago

@priyanshagra Photo isn't being stored in the database. We are supposed to store image in the database and fetch it from there whenever required.

priyanshagra commented 1 year ago

done i fixed

On Wed, 11 Oct 2023 at 22:17, Shubham Kashyap @.***> wrote:

@priyanshagra https://github.com/priyanshagra Photo isn't being stored in the database. We are supposed to store image in the database and fetch it from there whenever required.

— Reply to this email directly, view it on GitHub https://github.com/ContriHUB/Jodi-Makers/pull/19#issuecomment-1758095613, or unsubscribe https://github.com/notifications/unsubscribe-auth/A6JDHSBPL5OZYQL3RPWUKDTX63ESDANCNFSM6AAAAAA5ZKXVQQ . You are receiving this because you were mentioned.Message ID: @.***>

priyanshagra commented 1 year ago

done sir added to store in database functionality in server

admirerr commented 1 year ago

@priyanshagra Only a predefined path is being stored in the database. It should store the content of the image.

priyanshagra commented 1 year ago

please install module multer and image file name is saved in database and image is saved in multer buffer and can be used through name please merge

admirerr commented 1 year ago

please install module multer and image file name is saved in database and image is saved in multer buffer and can be used through name please merge

Add a line in the Readme file mentioning that you will be required to install multer to store image in its buffer.

priyanshagra commented 1 year ago

Done.

On Fri, 13 Oct 2023, 07:34 Shubham Kashyap, @.***> wrote:

please install module multer and image file name is saved in database and image is saved in multer buffer and can be used through name please merge

Add a line in the Readme file mentioning that you will be required to install multer to store image in its buffer.

— Reply to this email directly, view it on GitHub https://github.com/ContriHUB/Jodi-Makers/pull/19#issuecomment-1760655335, or unsubscribe https://github.com/notifications/unsubscribe-auth/A6JDHSF4LYZFTFRZ7AWEXITX7COULANCNFSM6AAAAAA5ZKXVQQ . You are receiving this because you were mentioned.Message ID: @.***>

ContriHUB / Jodi-Makers