Skip to content

Feature/411 거래완료된후채팅방생성 - #412

Merged
Chungs0604 merged 2 commits into
developfrom
feature/411-거래완료된후채팅방생성
Aug 13, 2026

Hidden character warning

The head ref may contain hidden characters: "feature/411-\uac70\ub798\uc644\ub8cc\ub41c\ud6c4\ucc44\ud305\ubc29\uc0dd\uc131"
Merged

Feature/411 거래완료된후채팅방생성#412
Chungs0604 merged 2 commits into
developfrom
feature/411-거래완료된후채팅방생성

Conversation

@Chungs0604

Copy link
Copy Markdown
Collaborator

🔗 관련 이슈

📌 작업 내용

  • 판매자가 구매 제안 상세 화면에서 채팅을 먼저 시작할 수 있도록 POST /chats 판매자 분기
    추가
    • 요청에 purchaseOfferId(nullable) 필드 추가, offer 검증(itemId·owner 일치, 상태
      SENT|ACCEPTED) 후 proposer를 buyer로 특정
  • 거래 상세 화면(TradeDetailPage)용 채팅방 생성·조회 엔드포인트 신규 추가 (POST
    /chats/trades/{tradeId})
    • 거래 성사 이후 offer 상태 검증이 불가능한 케이스를 별도 엔드포인트로 분리
    • 구매자·판매자 모두 호출 가능, 비참여자 403
  • ChatRoomResponse.price 로직 정비: 판매중·거래중 → 상품 가격 / 그 외(미판매·판매완료) →
    제안 금액(없으면 null)

📌 변경된 API

  • POST /chats
    • Request body에 purchaseOfferId (Long, nullable) 추가
    • 판매자가 호출 시 purchaseOfferId 필수, 구매자는 기존과 동일
  • POST /chats/trades/{tradeId} (신규)
    • PathVariable: tradeId
    • Response: 기존 ChatRoomResponse와 동일

📌 확인 필요 사항

  • TradeDetailPage에서 호출 시 tradeId PathVariable로 전달하도록 프론트 수정 필요
  • POST /chats 판매자 호출 시 purchaseOfferId 함께 전송하도록 프론트 수정 필요

✅ 체크리스트

  • 기능 요구사항을 충족했는가?
  • API 응답 형식이 통일되어 있는가?
  • 예외 상황이 적절히 처리되었는가?
  • 인증/인가 검증이 누락되지 않았는가?
  • Entity를 직접 응답하지 않았는가?
  • 중복 코드가 과도하지 않은가?
  • 메서드와 클래스의 역할이 명확한가?
  • 테스트가 필요한 핵심 로직에 테스트가 작성되었는가?

@Chungs0604
Chungs0604 changed the base branch from main to develop August 13, 2026 12:18
@Chungs0604
Chungs0604 merged commit a0b1a10 into develop Aug 13, 2026
1 of 2 checks passed
@Chungs0604
Chungs0604 deleted the feature/411-거래완료된후채팅방생성 branch August 13, 2026 13:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants