ScottyLabs / tartanhacks_dashboard_v2

3 stars 7 forks source link

Actang2/inputting profile pic #75

Closed palascat100 closed 1 year ago

palascat100 commented 1 year ago

Description

Allows users to upload profile pictures.

Type of change

[x] New feature (non-breaking change which adds functionality)
[ ] This change requires a documentation update

How Has This Been Tested?

[ ] Deleted profile picture to make sure it successfully handles initial state (when the user does not have profile picture) => should show default pfp from asset
[ ] Selected picture through gallery and saved => should upload to backend and update profile afterward
[ ] Selected picture through gallery and then did not save (through cancel button or click off dialog) => basically resets state of picking profile picture dialog

Test Configuration:

Node.js version:
Python version:
Desktop/Mobile:
OS:
Browser:

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

jhong66 commented 1 year ago

Just resolved merge conflicts with the main branch. It's usually good to checkout the main branch, git pull, then go back to your branch and git merge main before doing your final push. Sorry if you're still working on this branch and the commits mess stuff up!