feat: Task 4 — Team Page, linkedin_url, and role-grouped member display#11
Open
sachhayaKO wants to merge 2 commits into
Open
feat: Task 4 — Team Page, linkedin_url, and role-grouped member display#11sachhayaKO wants to merge 2 commits into
sachhayaKO wants to merge 2 commits into
Conversation
- Add linkedin_url field (String 500, nullable) to Membership model + migration 0004 - Add linkedin_url to MembershipPublic, MembershipCreate, MembershipUpdate schemas - Add linkedin_url and role (user's system role) to MemberPublic schema and web endpoint - Update GET /web/v1/members to include linkedin_url and role in response - Add linkedin_url field to ops tool member edit form - Update shared MemberPublic and Membership TypeScript types - Overhaul website MemberCard to show full profile (major/grad_year, hometown, campus_involvements, experience, interests, email, LinkedIn) - Refactor TeamGrid to group members by hierarchy: eboard → directors → PMs → analysts - Update getMembers() API helper to return 4 role-based groups - Update TeamListBlock and TeamPage to use new structure - Update placeholder data with new required fields https://claude.ai/code/session_01Qgt2UQQUmaNSo3ViCXfVnb
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What this does
linkedin_urlfield (String 500, nullable) toMembershipmodel with Alembic migration0004_membership_linkedinMembershipPublic,MembershipCreate,MembershipUpdate, andMemberPublicschemas to includelinkedin_url; addsrole(user sys role) toMemberPublicfor hierarchy groupingGET /web/v1/membersto returnlinkedin_urlandrolein each recordlinkedin_urlinput field to the ops tool member edit form (/members/[id])MemberCardto show full profile: headshot, name, role_title, major + grad_year, hometown, campus_involvements, professional_experience, interests, email, LinkedIn linkTeamGridto group members by hierarchy: Executive Board → Directors → Project Managers → AnalystsgetMembers()to return 4 role-based groups (eboard,directors,pms,analysts)TeamListBlock,TeamPage, and placeholder data to use the new structure@cba/typesMemberPublicandMembershipinterfacesHow to test
alembic upgrade head—linkedin_urlcolumn added tomembershipstable/members/[id]— LinkedIn URL field appears in the formGET /web/v1/members/teamon the public website — members grouped into Executive Board / Directors / Project Managers / AnalystsGET /web/v1/membersreturnslinkedin_urlandrolefields🤖 CBA Feature Agent