Skip to content
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

Added upload Profile Photo feature for both user and ngo user | #243 #344

Merged
merged 3 commits into from
May 19, 2024

Conversation

shilpashingnapure
Copy link
Contributor

Description

1 PR -> #243

  1. Added the Profile photo feature for both user.
  2. the UI is same for both user.

Screenshots / GIFs (if applicable)

[Attach any relevant screenshots or GIFs demonstrating the changes]

  1. for default , it will show first latter of name
defaultphoto 3. on hover UI onHover 4. dialog pop dialogBox

Checklist:

  • I have performed a self-review of my code
  • I have added/updated relevant documentation (if needed)
  • I have tested the changes locally and they function as expected
  • I have ensured my code follows the project's coding standards

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Welcome to Our repository.🎊 Thank you so much for taking the time to point this out.

Copy link
Collaborator

@rishicds rishicds left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great Job! @shilpashingnapure

@codewarnab
Copy link
Collaborator

@shilpashingnapure approved will be meged after backend pr is meged

@rishicds rishicds merged commit cb999d0 into Innovateninjas:main May 19, 2024
1 check passed
@rishicds rishicds added level3 45 points gssoc Girl Script Summer of Code labels May 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
gssoc Girl Script Summer of Code level3 45 points
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants