Skip to content

[fix] 헤더 컴포넌트 - 소속 팀 없을 경우 고장나던 오류 수정#219

Merged
MINJI121 merged 1 commit intodevfrom
refactor/#218/header-component
May 24, 2025
Merged

[fix] 헤더 컴포넌트 - 소속 팀 없을 경우 고장나던 오류 수정#219
MINJI121 merged 1 commit intodevfrom
refactor/#218/header-component

Conversation

@MySoupRice
Copy link
Contributor

🔗 이슈 번호

Closes #218

📋 작업 사항

문제 상황

  • 소속팀이 없을 경우 헤더 표시 및 팀 참여 같은 기능이 정지된다

문제 원인

  1. header/index 에서 selectedGroup이 있을 경우에만 유저 정보 가져오고 있음
  • isLogin && Boolean(selectedGroup)
  1. join-team 에서 membership.userEmail 을 통해 유저 이메일 가져오고 있음
  • const email = memberships[0]?.userEmail

문제 해결

  1. selectedGroup이 없어도 유저 정보 가져오기
  • enabled: isLogin
  1. currentUser.email 을 통해 유저 이메일 가져오기
  • const email = currentUser?.email

@MySoupRice MySoupRice requested review from ARON-Y and MINJI121 May 24, 2025 13:52
@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
2 tasks
@hyeonjiroh hyeonjiroh self-requested a review May 24, 2025 13:52
@MySoupRice MySoupRice removed the request for review from hyeonjiroh May 24, 2025 13:52
@github-actions
Copy link

Preview Deployment
🔗 https://coworkers-8cvtsz4x2-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.

확인했습니다 ~!

@MINJI121 MINJI121 merged commit cd6be3c into dev May 24, 2025
2 checks passed
@MINJI121 MINJI121 deleted the refactor/#218/header-component branch May 24, 2025 13:57
@MINJI121 MINJI121 added ⚒️ fix 버그 수정 이슈 and removed ⚒️ refactor 코드 리팩토링 이슈 labels May 24, 2025
@MINJI121 MINJI121 changed the title [refactor] 헤더 컴포넌트 - 소속 팀 없을 경우 고장나던 오류 수정 [fix] 헤더 컴포넌트 - 소속 팀 없을 경우 고장나던 오류 수정 May 24, 2025
@MINJI121 MINJI121 added the ⚒️ refactor 코드 리팩토링 이슈 label May 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

⚒️ fix 버그 수정 이슈 ⚒️ refactor 코드 리팩토링 이슈

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[refactor] 헤더 컴포넌트 리팩토링

3 participants