-
-
Notifications
You must be signed in to change notification settings - Fork 422
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feature: Updating profile picture functionality and UI #1218
Comments
Hello @tamalCodes, thank you for raising the issue.
Currently, the issue is marked as https://github.com/MilanCommunity/Milan/labels/status%3A%20todo%20%E2%8F%B3 so please wait until the maintainers/owner review it and provide you with feedback/suggestions to proceed further. Feel free to reach out to Tamal on Twitter, or drop a mail at tamalcodes@gmail.com if you think that this issue is of critical priority. Give us a ⭐ to show some support |
@tamalCodes Can you please add the existing pop up modal |
we can use cloudinary for this it will convert the image into the link and then we just have to modify our backend litte much |
That can be done, but do i need to pay for it ? |
yes according to me cloudynary charge for this , But it is good to store our image as a string in database |
can you assign me this task @tamalCodes ? |
Sure , go ahead @rutupatel53 . Please react to this message so that I know you are working on the same ! |
Description
Currently, there is no way to update the profile picture that is provided by default so updating it or having an option to do the same would be amazing
since the back end is not yet ready for this the idea is to make sure the front-end flow is ready properly along with some changes to the UI for updating a picture.
Proposed Solution
Screenshots
No response
🥦 Browser
Google Chrome
Checklist ✅
The text was updated successfully, but these errors were encountered: