Skip to content

[FLIZ-454/trainer] fix: 트레이너 error/bug fix - #294

Open
yjc2021 wants to merge 13 commits into
devfrom
FLIZ-454/fix/trainer
Open

[FLIZ-454/trainer] fix: 트레이너 error/bug fix#294
yjc2021 wants to merge 13 commits into
devfrom
FLIZ-454/fix/trainer

Conversation

@yjc2021

@yjc2021 yjc2021 commented Aug 1, 2025

Copy link
Copy Markdown
Contributor

[FLIZ-454/trainer] fix: 트레이너 error/bug fix

📝 작업 내용

트레이너 서비스를 전반적으로 검토하여 발견한 버그 및 warning을 해결하고, Next Link API를 적용하는 작업을 진행했습니다

  • 유저 리스트 쿼리 호출 시 Suspense fallback flicker UX 개선
  • warning 발생 요인인 미사용 ellipsisIcon prop 제거
  • NotificationItem UI 찌그러짐 bug fix
  • /notification 페이지 loading.tsx 추가
  • ProfileCardFallback UI 수정 (padding-top, padding-bottom 부여)
  • Logo 컴포넌트를 감싸는 불필요 div 요소 제거
  • NotificationSearch 컴포넌트 단계별 list UI의 잘못된 overflow 설정으로 infinite scroll 로직이 작동하지 않는 bug fix
  • bottom navigation 불필요한 state 사용 로직 제거 및 Next Link API 적용
  • 서비스 전반의 라우팅 로직에 Next Link API 적용

📷 스크린샷 (선택)

💬 리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요
ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

@yjc2021 yjc2021 changed the title Fliz 454/fix/trainer [FLIZ-454/trainer] fix: 트레이너 error/bug fix Aug 1, 2025
@yjc2021 yjc2021 self-assigned this Aug 1, 2025
@yjc2021 yjc2021 added the 🐛fix 버그를 고친 경우 label Aug 1, 2025
@MaSeungHyun
MaSeungHyun self-requested a review August 1, 2025 04:33

@MaSeungHyun MaSeungHyun left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🚀 LGTM

@MaSeungHyun

Copy link
Copy Markdown
Contributor

제가 작성한 코드랑 complict 난 거 같아서 컴플릭 잡아놓았습니다.
Link 태그로 구현하신 방향으로 살려놓았습니다!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐛fix 버그를 고친 경우

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants