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

Feat: Added social media links to user profile section successfully issue 226 #474

Merged
merged 1 commit into from
Jul 22, 2024

Conversation

MeetDOD
Copy link
Contributor

@MeetDOD MeetDOD commented Jul 20, 2024

Pull Request Resolves [#226 ]

Title: Added social media links to user profile section successfully.

Description

  1. In backend, I modified the user prisma schema and also updated the updateUserProfile controller as well as userProfileController and showUserProfileController to fetch the social links and their profiles.
  2. In frontend, I added the field for Twitter, facebook, github, linkedin and portfolio social links.
  3. Also modified the show user profile to see other profile section.
  4. Used icons which are responsive and also work in light/dark mode.

Related Issues

Fixes #226

Changes Made

New feature: Added social media links to user profile section.

Screenshots

image

image

image

image

Video

Style.Share._.View.profile.-.Google.Chrome.2024-07-21.00-08-31.online-video-cutter.com.mp4

I certify that I have carried out the relevant code of conduct and provided the requisite screenshot for validation by submitting this pull request.

Thank You for this contribution.

Copy link

vercel bot commented Jul 20, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
style-share ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 20, 2024 6:54pm

Copy link

Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. In the meantime, please ensure that your changes align with our CONTRIBUTING.md. If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊

@MeetDOD
Copy link
Contributor Author

MeetDOD commented Jul 21, 2024

@VaibhavArora314 can you please give level 3 because of frontend + backend integration and testing ! you review it 👍

Copy link
Owner

@VaibhavArora314 VaibhavArora314 left a comment

Choose a reason for hiding this comment

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

You can remove the Facebook option as the edit user modal currently is too long vertically because of too many options.

@MeetDOD
Copy link
Contributor Author

MeetDOD commented Jul 22, 2024

You can remove the Facebook option as the edit user modal currently is too long vertically because of too many options.

You can remove the Facebook option as the edit user modal currently is too long vertically because of too many options.

@VaibhavArora314 that's why i created the issue #475 i will align nicely in all the devices. But if you want i will remove it , So i remove it or not the facebook icon ?

@VaibhavArora314
Copy link
Owner

You can remove the Facebook option as the edit user modal currently is too long vertically because of too many options.

You can remove the Facebook option as the edit user modal currently is too long vertically because of too many options.

@VaibhavArora314 that's why i created the issue #475 i will align nicely in all the devices. But if you want i will remove it , So i remove it or not the facebook icon ?

Don't remove it then

@VaibhavArora314 VaibhavArora314 merged commit 20d5a40 into VaibhavArora314:main Jul 22, 2024
4 checks passed
Copy link

🎉 Your pull request has been successfully merged! 🎉 Thank you for your valuable contribution to our project. Your efforts are greatly appreciated. Feel free to reach out if you have any more contributions or if there's anything else we can assist you with. Keep up the fantastic work! 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature Request]: Add Social Media Links to User Profile Section
4 participants