Skip to content

[feat/api/chat/flow 35] 프로필, 채팅, 수정 중 - #72

Merged
xeunnie merged 15 commits into
developfrom
feat/api/chat/FLOW-35
Jul 27, 2025
Merged

[feat/api/chat/flow 35] 프로필, 채팅, 수정 중#72
xeunnie merged 15 commits into
developfrom
feat/api/chat/FLOW-35

Conversation

@xeunnie

Copy link
Copy Markdown
Member

💡 PR 제목

✅ 요구사항

  1. 프로필 관련 작업 (FLOW-35):

    • 프로필 에러 수정.
    • 프로필 추가 기능.
    • 내 정보 수정 관련 테스트 완료.
  2. 채팅 관련 작업 (FLOW-35):

    • 파일 업로드로 채팅 메시지 전송 기능 수정 중.
    • 멘션 리스트 추가 및 멘션 기능 완료.
    • 메시지 전송 상태 관리 추가 (전송 중/성공/참여 멤버 다이얼로그).
    • 팀 관련 정보의 상태 관리 추가.
  3. UI와 상태 관련 개선:

    • Context 메뉴 추가.
    • 로그아웃 시 리다이렉션 추가.

⭐ 시연

✅ 기능1

✅ 리뷰어가 알아야할 사항

✅ 체크리스트

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

@xeunniexeunnie added this to the API milestone Jul 27, 2025
@xeunnie
xeunnie requested review from Copilot and dbswl701July 27, 2025 14:31
@xeunniexeunnie self-assigned this Jul 27, 2025
@xeunniexeunnie added the API 백엔드 연결 label Jul 27, 2025
@netlify

netlifyBot commented Jul 27, 2025

Copy link
Copy Markdown

Deploy Preview for chatflow-project ready!

NameLink
🔨 Latest commit765c099
🔍 Latest deploy loghttps://app.netlify.com/projects/chatflow-project/deploys/68863c235aaf0e0008874777
😎 Deploy Previewhttps://deploy-preview-72--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.

CopilotAI 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.

Pull Request Overview

This PR refactors and enhances chat functionality including profile, file upload, and mention features. The changes improve state management, component architecture, and user interaction capabilities for the chat system.

  • Refactored message handling to use consistent schema and improved prop naming
  • Added file upload functionality with image handling and mention system enhancements
  • Implemented profile editing and status management features

Reviewed Changes

Copilot reviewed 36 out of 36 changed files in this pull request and generated 7 comments.

Show a summary per file
FileDescription
src/view/pages/chat/components/message/ChatMessageItem.tsxRenamed prop from mentions to memberIds for consistency
src/view/pages/chat/components/layout/ChatView.tsxUpdated imports and message rendering to use new schema
src/view/pages/chat/components/input/ChatInput.tsxNew chat input component with file upload and mention functionality
src/view/pages/chat/ChatPage.tsxRefactored message handling and file upload logic
src/view/layout/profile/UserProfileBar.tsxAdded profile context menu and status management
src/service/feature/auth/api/profileApi.tsAdded profile update and status change API functions
src/service/feature/channel/api/channelAPI.tsRefactored API endpoints and improved type safety
Comments suppressed due to low confidence (1)

src/view/layout/sidebar/components/channel/ChannelCategory.tsx:64

  • The prop name 'serverId' should be 'teamId' to match the component's expected prop name based on the ChannelAddDialog component definition.
 <ChannelAddDialog serverId={serverId} onClose={() => setIsDialogOpen(false)} />

Comment threadsrc/view/pages/chat/components/layout/ChatView.tsx Outdated
Comment threadsrc/view/pages/chat/components/input/ChatInput.tsx Outdated
Comment threadsrc/view/pages/chat/components/input/ChatInput.tsx Outdated
);
const profile = useSelector((state: RootState) => (state.auth as any).profile);

const dispatch = useDispatch();

CopilotAIJul 27, 2025

Copy link

Choose a reason for hiding this comment

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

Missing import for useDispatch from 'react-redux'. This will cause a runtime error.

Copilot uses AI. Check for mistakes.
Comment threadsrc/view/layout/profile/UserProfileBar.tsx Outdated
Comment threadsrc/service/feature/chat/hook/useChat.ts Outdated
Comment threadsrc/service/feature/channel/hook/mutation/useChannelMutation.ts Outdated
xeunnieand others added 7 commits July 27, 2025 23:35
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@github-actions

Copy link
Copy Markdown

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

@github-actions

Copy link
Copy Markdown

CI Status Report

검사 결과

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

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

@github-actions

Copy link
Copy Markdown

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

@github-actions

Copy link
Copy Markdown

CI Status Report

검사 결과

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

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

@github-actions

Copy link
Copy Markdown

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

@github-actions

Copy link
Copy Markdown

CI Status Report

검사 결과

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

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

@xeunnie
xeunnie merged commit 1ce7e10 into developJul 27, 2025
4 of 5 checks passed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

API백엔드 연결

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@xeunnie