Skip to content

Conversation

@dbswl701
Copy link
Contributor

💡 PR 제목

✅ 요구사항

  • DMList api 연동
  • 자잘한 수정사항 반영 (팀 사이즈바 이미지 꽉 차게, 서버 생성 모달 닫기 후 초기화)
  • DMList에서 2인 이상인 경우 반영
  • team sidebar, dmchannelSidebar, friend 스켈레톤 코드 구현
  • 친구의 온라인 / 오프라인 정보 표시

⭐ 시연

image image image

✅ 기능1

✅ 리뷰어가 알아야할 사항

  • DMList api의 경우, 추후 api 응답값이 변경될 예정.
  • 사전에 맞춘대로 코드는 작성되어 있으며 DMUserCard의
  // const user = channel.channelMembers;

  // 백엔드 개발 완료되기 전 임시 데이터
  const user = [
    { id: 0, state: 'ONLINE' },
    { id: 1, state: 'OFFLINE' },
  ];

이 부분이 변경될 예정.

  • DM채널 추가는 다른 브랜치에서 작업 예정

✅ 체크리스트

  • 자신의 코드에 대한 리뷰를 진행했습니다
  • 실행 후 심각한 버그나 경고문이 없음을 확인했습니다
  • 이해하기 어려운 부분에 적절한 주석을 첨부했습니다
  • 요구사항 변경, 레이아웃 변경 등 변경사항을 문서에 반영했습니다

@dbswl701 dbswl701 added this to the API milestone Jun 18, 2025
@dbswl701 dbswl701 requested a review from xeunnie June 18, 2025 09:11
@dbswl701 dbswl701 self-assigned this Jun 18, 2025
@dbswl701 dbswl701 added API 백엔드 연결 feature 기능 구현 labels Jun 18, 2025
@netlify
Copy link

netlify bot commented Jun 18, 2025

Deploy Preview for chatflow-project ready!

Name Link
🔨 Latest commit 5b77bba
🔍 Latest deploy log https://app.netlify.com/projects/chatflow-project/deploys/685282a9c569ab000889c300
😎 Deploy Preview https://deploy-preview-65--chatflow-project.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@dbswl701 dbswl701 closed this Jun 18, 2025
@dbswl701 dbswl701 reopened this Jun 18, 2025
@github-actions
Copy link

🛠 PR 검사를 시작합니다. 잠시만 기다려 주세요!

@github-actions
Copy link

CI Status Report

검사 결과

  • Lint: ❌ skipped
  • Format: ❌ skipped
  • Build: ❌ skipped

❌ 일부 검사가 실패했습니다.

@github-actions
Copy link

🛠 PR 검사를 시작합니다. 잠시만 기다려 주세요!

@github-actions
Copy link

CI Status Report

검사 결과

  • Lint: ❌ skipped
  • Format: ❌ skipped
  • Build: ❌ skipped

❌ 일부 검사가 실패했습니다.

@github-actions
Copy link

🛠 PR 검사를 시작합니다. 잠시만 기다려 주세요!

@github-actions
Copy link

CI Status Report

검사 결과

  • Lint: ❌ skipped
  • Format: ❌ skipped
  • Build: ❌ skipped

❌ 일부 검사가 실패했습니다.

@github-actions
Copy link

🛠 PR 검사를 시작합니다. 잠시만 기다려 주세요!

@github-actions
Copy link

CI Status Report

검사 결과

  • Lint: ❌ skipped
  • Format: ❌ skipped
  • Build: ❌ skipped

❌ 일부 검사가 실패했습니다.

@dbswl701 dbswl701 closed this Jun 18, 2025
@dbswl701 dbswl701 reopened this Jun 18, 2025
@github-actions
Copy link

🛠 PR 검사를 시작합니다. 잠시만 기다려 주세요!

@github-actions
Copy link

CI Status Report

검사 결과

  • Lint: ❌ failure
  • Format: ❌ failure
  • Build: ✅ success

❌ 일부 검사가 실패했습니다.

@dbswl701
Copy link
Contributor Author

dbswl701 commented Jun 18, 2025

오... 저희 지금까지 eslint 적용이 안되어 있었네용...?

// eslint.config.js
'react/jsx-max-depth': ['warn', { max: 7 }],

여기서 warn이 warning으로 되어있었습니다...!

@xeunnie xeunnie merged commit 6cd4f1c into develop Jun 20, 2025
4 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

API 백엔드 연결 feature 기능 구현

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants