Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[fix #179] 채팅 요청 답변자 검증 로직 추가 #185

Merged
merged 5 commits into from
Jan 9, 2025

Conversation

hyun2371
Copy link
Member

@hyun2371 hyun2371 commented Jan 4, 2025

관련 이슈

📑 작업 상세 내용

  • 질문 게시글과 답변자로 답변 존재하는지 여부 확인
    • 채팅 요청 에러 코드 추가
    • 관련 단위 테스트 추가

💫 작업 요약

  • 채팅 요청 API에 답변자 검증 로직 추가

🔍 중점적으로 리뷰 할 부분

@hyun2371 hyun2371 added the 🚨 fix 오류 수정 label Jan 4, 2025
@hyun2371 hyun2371 requested a review from dudxo January 4, 2025 12:52
@hyun2371 hyun2371 self-assigned this Jan 4, 2025
@hyun2371 hyun2371 linked an issue Jan 4, 2025 that may be closed by this pull request
Copy link

github-actions bot commented Jan 4, 2025

Code Coverage

Overall Project 83.65% 🍏
Files changed 100% 🍏

File Coverage
ChatInquiryService.java 93.38% 🍏

Copy link

github-actions bot commented Jan 4, 2025

Test Results

 28 files   28 suites   14s ⏱️
137 tests 135 ✅ 2 💤 0 ❌
138 runs  136 ✅ 2 💤 0 ❌

Results for commit 5a718e3.

Copy link
Collaborator

@dudxo dudxo left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!

@hyun2371 hyun2371 merged commit 1292aed into dev Jan 9, 2025
3 checks passed
@hyun2371 hyun2371 deleted the fix/#179/chat-request-validation branch January 9, 2025 05:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🚨 fix 오류 수정
Projects
None yet
Development

Successfully merging this pull request may close these issues.

🚨 채팅 요청 답변자 검증 로직 추가
2 participants