Skip to content

Feat/#220 팀 페이지 내 미구현 기능 구현#221

Merged
SugarSyrup merged 8 commits into
developfrom
feat/#220
Apr 16, 2026
Merged

Feat/#220 팀 페이지 내 미구현 기능 구현#221
SugarSyrup merged 8 commits into
developfrom
feat/#220

Conversation

@SugarSyrup

Copy link
Copy Markdown
Contributor

#️⃣ 연관된 이슈

#220

📝 작업 내용

  • 목표 리스트 무한 스크롤
  • 목표 리스트 반응형 / 로딩 / 에러
  • Summary 반응형 / 로딩 / 에러
  • 멤버 리스트 반응형 / 로딩 / 에러

스크린샷 (선택)

💬 리뷰 요구사항(선택)

@SugarSyrup SugarSyrup self-assigned this Apr 14, 2026
@SugarSyrup SugarSyrup added the feature 기능 (새로운 기능) label Apr 14, 2026
@github-actions

github-actions Bot commented Apr 14, 2026

Copy link
Copy Markdown

🎨 UI Review (Chromatic)

결과 상세 내용
🚀 Storybook Preview 미리보기 바로가기
🛠 Build Details Chromatic 빌드 리포트

알림: 시각적 변경 사항이 발견되면 리뷰어가 Chromatic에서 승인해야 머지가 가능합니다.

Comment on lines +1 to +2
"use client";

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

use client가 붙어 있는데, AsyncBoundary가 ssr: false로 dynamic import 되어있어서 제거해도 괜찮지않나요?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

errorFallback에서 화살표 함수 prop을 전달하는게, 서버 컴포넌트에서는 함수를 전달 못해서 에러가 발생하더라고요

목표
</h2>
<span className="typography-body-1 ml-[-8px] font-medium text-gray-400">
{goalList.length}개

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

혹시 목표의 전체 개수가 필요한 건가요, 아니면 현재 불러온 데이터의 개수를 표출하는 건가요? 무한스크롤이라 스크롤 전엔 일부만 로드되어있을것 같아서요!

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

그러네요 이건 제가 놓친거 같습니다

@SugarSyrup SugarSyrup merged commit 8ec82d9 into develop Apr 16, 2026
3 of 5 checks passed
@SugarSyrup SugarSyrup deleted the feat/#220 branch April 16, 2026 08:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature 기능 (새로운 기능)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants