Conversation
|
Could you please add screenshots of any UI changes :) |
Hi~ I just added the screenshots for the contributors/ members part on the pages. Please have a check and test it !~~ |
SafetyInObscurity
left a comment
There was a problem hiding this comment.
Very good job! This is ready for merging except that you've made the socialmediaName field unnecessary (which is a good thing). Before merging we should remove the field from the frontend and backend to simplify things.
According to how the frontend is structured, the |
You are absolutely right, the react-social-icons will sniff the platform URL instead of we manually assign the platform logo. I will do the requested changes shortly. |

#55
Change Summary
Add support for associating social media profiles with members, and displays these links throughout the application. The changes span the backend (models, serializers, admin, migrations) and the frontend (types, hooks, components, and pages). The main focus is to allow each member to have multiple social media links, which are then shown in member profiles and contributor listings.
Backend: Social Media Model and API Enhancements
SocialMediamodel linked toMember, with platform choices, link, and username fields. [1] [2]SocialMediaSerializer. [1] [2]Frontend: Type and Data Model Updates
Frontend: Display of Social Media Links
react-social-icons.Individual Games page:

Game Showcase Page:

Members Page:

Other Technical Updates
react-social-iconswhere social media links are displayed. (client/src/components/main/MemberProfile.tsxR4, client/src/pages/games/[id].tsxR4)SocialMediamodel is included.Change Form
Fill this up (NA if not available). If a certain criteria is not met, can you please give a reason.
Other Information
[Is there anything in particular in the review that I should be aware of?]