Skip to content

[refactor] 헤더 컴포넌트 / 기타 리팩토링#213

Merged
MINJI121 merged 17 commits intodevfrom
refactor/#202/header-component
May 24, 2025
Merged

[refactor] 헤더 컴포넌트 / 기타 리팩토링#213
MINJI121 merged 17 commits intodevfrom
refactor/#202/header-component

Conversation

@MySoupRice
Copy link
Contributor

@MySoupRice MySoupRice commented May 24, 2025

🔗 이슈 번호

Closes #202

📋 작업 사항

  • 팀 수정, 유저 정보 수정 시 헤더 즉시 반영

/hooks/useMemberShips

api에서 가져온 최신 객체를 selectedGroup에 덮어씌우기
-> 헤더에 즉시 반영

/app/mypage/_components/AccountUpdateButton

이전 상황: mutation으로 'user' 무효화 / 헤더는 'currentUser' 사용
현재: mutation으로 'currnetUser' 무효화 -> 헤더에 즉시 반영

  • 팀 목록

팀 이름 길어지면 ... 추가 및 가로스크롤 제거
드롭다운 위치 조정
'팀 목록' 상태일때 누르면 첫번째 팀이 아닌, 드롭다운 열리게 변경
팀목록 순서 조정 - ID 작은 순서대로

  • 팀 이름 최대 30자 제한

22

  • 모바일 사이드메뉴

상단에 자유게시판, 팀생성 링크 추가
팀 목록 구분하는 경계선 추가

11

@MySoupRice MySoupRice self-assigned this May 24, 2025
@MySoupRice MySoupRice added the ⚒️ refactor 코드 리팩토링 이슈 label May 24, 2025
@MySoupRice MySoupRice linked an issue May 24, 2025 that may be closed by this pull request
7 tasks
@hyeonjiroh hyeonjiroh requested review from ARON-Y and hyeonjiroh May 24, 2025 04:11
@MySoupRice MySoupRice requested review from MINJI121 and jjanie00 and removed request for hyeonjiroh and jjanie00 May 24, 2025 04:15
ARON-Y
ARON-Y previously approved these changes May 24, 2025
Copy link
Contributor

@ARON-Y ARON-Y left a comment

Choose a reason for hiding this comment

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

사이드 메뉴에서 팀 목록 구분과 자유게시판/ 팀 생성 링크 추가 하신거 좋네요! UX가 더 좋아진것 같아요! 고생하셨습니다 현호님!🙇🏻‍♀️

@FE13-Part4-Team2 FE13-Part4-Team2 deleted a comment from github-actions bot May 24, 2025
@FE13-Part4-Team2 FE13-Part4-Team2 deleted a comment from github-actions bot May 24, 2025
@github-actions
Copy link

Preview Deployment
🔗 https://coworkers-i0o53xdq9-hyeonjirohs-projects.vercel.app

Copy link
Contributor

@MINJI121 MINJI121 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 현호님~!!

const newGroup = memberships.find(
(m) => m.group.id === prev?.id
)?.group;
return newGroup ?? memberships[0].group;
Copy link
Contributor

Choose a reason for hiding this comment

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

최신 데이터 가져오도록 변경해주셨네요!👍👍

@MINJI121 MINJI121 merged commit 1a5794e into dev May 24, 2025
1 check passed
@MINJI121 MINJI121 deleted the refactor/#202/header-component branch May 24, 2025 06:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

⚒️ refactor 코드 리팩토링 이슈

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[refactor] 헤더 리팩토링

3 participants