Skip to content

[Fix] Hotfix - 채팅,구매/판매 내역, 카테고리페이지 뒤로가기 버튼#50

Merged
l2juhan merged 10 commits into
developfrom
main
Dec 18, 2025
Merged

[Fix] Hotfix - 채팅,구매/판매 내역, 카테고리페이지 뒤로가기 버튼#50
l2juhan merged 10 commits into
developfrom
main

Conversation

@l2juhan
Copy link
Copy Markdown
Contributor

@l2juhan l2juhan commented Dec 18, 2025

Summary by CodeRabbit

릴리스 노트

  • 새로운 기능

    • 채팅 메시지에 한국 시간 형식 표시 추가
  • 버그 수정

    • API 서버 오류 처리 개선 (HTTP 500 에러 처리)
    • 카카오 로그인 환경 변수 및 URL 검증 추가
    • 채팅방 주문 상태 관리 및 버튼 활성화 로직 개선
  • 스타일

    • 뒤로 가기 버튼을 텍스트에서 아이콘으로 변경
  • 리팩토링

    • 라우터를 HashRouter에서 BrowserRouter로 변경
    • 채팅 메시지 조회 로직 개선 및 상태 관리 최적화

✏️ Tip: You can customize this high-level summary in your review settings.

@l2juhan l2juhan self-assigned this Dec 18, 2025
@l2juhan l2juhan added the bug Something isn't working label Dec 18, 2025
@l2juhan l2juhan merged commit e4a1a5e into develop Dec 18, 2025
1 of 2 checks passed
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Dec 18, 2025

Caution

Review failed

The pull request is closed.

요약

Walkthrough

이 PR은 여러 개선 사항을 포함합니다: 채팅 API에 HTTP 500 에러 처리 추가, 의류 API에 토큰 갱신 기능이 있는 에러 처리 강화, 라우팅을 HashRouter에서 BrowserRouter로 변경, 채팅방 메시지 검색 로직 리팩터링, Kakao 로그인 검증 추가, UI 컴포넌트 업데이트(뒤로가기 버튼 아이콘, 판매 이력 버튼 로직).

Changes

Cohort / File(s) 변경 요약
API 에러 처리
src/api/chatApi.js, src/api/clothesApi.js
chatApi.js에서 getUserChatRooms와 getChatRoomMessages에 HTTP 500 에러 처리 추가. clothesApi.js에서 filterClothes의 응답 래퍼 필드 제거, deleteClothes에서 403 에러 처리 및 토큰 갱신 워크플로우 추가 및 안전한 JSON 파싱 구현
라우팅 변경
src/main.jsx
HashRouter를 BrowserRouter로 교체
채팅 기능
src/pages/Chat/ChatRoomPage.jsx
메시지 검색 로직을 fetchChatMessages 함수로 리팩터링, 동적 주문 상태(currentOrderStatus) 처리 추가, formatChatTime 유틸리티로 타임스탐프 포맷팅, 주문 상태에 따른 UI 컨트롤 활성화/비활성화 처리
포맷터 유틸리티
src/utils/formatters.js
formatChatTime(dateString) 새 함수 추가 - Asia/Seoul 시간대로 한국 시간 형식 문자열 반환
인증 검증
src/pages/auth/AuthStartPage.jsx
handleKakaoLogin에 이벤트 매개변수 추가, 환경 변수(REST_API_KEY, REDIRECT_URI) 런타임 검증, Kakao 인증 URL 유효성 검사 추가, 로그인 버튼 이미지 드래그 방지 처리
UI 컴포넌트
src/pages/MyPage/components/SalesHistoryList.jsx, src/pages/category/CategoryResultPage.jsx
SalesHistoryList에서 디버그 console.log 추가 및 "물품 전달 완료" 버튼 표시 로직 재작업. CategoryResultPage에서 뒤로가기 버튼을 텍스트에서 아이콘(MdArrowBackIosNew)으로 변경

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

  • 주의 필요 영역:
    • src/api/clothesApi.js의 deleteClothes 함수: 토큰 갱신 워크플로우와 안전한 JSON 파싱 로직의 복잡성
    • src/pages/Chat/ChatRoomPage.jsx의 주문 상태 관리 및 메시지 검색 리팩터링: 기존 동작 호환성 확인 필요
    • src/pages/auth/AuthStartPage.jsx의 환경 변수 검증: 실행 흐름과 에러 처리 경로 검토

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Free

📥 Commits

Reviewing files that changed from the base of the PR and between d731c20 and 5f8cdd0.

📒 Files selected for processing (8)
  • src/api/chatApi.js (2 hunks)
  • src/api/clothesApi.js (2 hunks)
  • src/main.jsx (2 hunks)
  • src/pages/Chat/ChatRoomPage.jsx (10 hunks)
  • src/pages/MyPage/components/SalesHistoryList.jsx (2 hunks)
  • src/pages/auth/AuthStartPage.jsx (2 hunks)
  • src/pages/category/CategoryResultPage.jsx (2 hunks)
  • src/utils/formatters.js (1 hunks)

Note

🎁 Summarized by CodeRabbit Free

Your organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Pro by visiting https://app.coderabbit.ai/login.

Comment @coderabbitai help to get the list of available commands and usage tips.

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

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant