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

[feat #178] 질문글 작성 전 최소한의 크레딧을 보유하는지 검증한다. #202

Merged
merged 7 commits into from
Feb 2, 2025

Conversation

dudxo
Copy link
Collaborator

@dudxo dudxo commented Feb 2, 2025

관련 이슈

📑 작업 상세 내용

  • 질문글 작성 전 최소한의 크레딧(질문글 최소 채택 크레딧 2_000)을 보유하고 있는지 검증한다.

💫 작업 요약

  • 질문글 작성 전 최소한의 크레딧(질문글 최소 채택 크레딧 2_000)을 보유하고 있는지 검증한다.

🔍 중점적으로 리뷰 할 부분

  • 보유 크레딧이 최소 채택 크레딧2_000 보다 적을 경우 false가 아닌 예외를 던질까 생각했는데, 질문 글 작성 가능 여부를 판단하는 체크리스트 도중 비정상적인 행위는 아니라고 생각해서 논리형 응답으로 처리했습니다.
  • 해당 API를 Member 패키지에 추가할까 하다 질문글 작성 전 검증하는 체크리스트라서 QuestionPost 패키지로 선택했습니다.
  • Merge 후 프론트 측에 공유하겠습니다.
  • 노션 API 명세서에 추가했습니다.

@dudxo dudxo added the ✨ feat 기능 추가 label Feb 2, 2025
@dudxo dudxo requested a review from hyun2371 February 2, 2025 09:18
@dudxo dudxo self-assigned this Feb 2, 2025
@dudxo dudxo linked an issue Feb 2, 2025 that may be closed by this pull request
1 task
Copy link

github-actions bot commented Feb 2, 2025

Code Coverage

Overall Project 84.57% 🍏
Files changed 100% 🍏

File Coverage
QuestionPostController.java 100% 🍏
Member.java 93.85% 🍏
QuestionPostService.java 84.75% 🍏

Copy link

github-actions bot commented Feb 2, 2025

Test Results

 28 files   28 suites   13s ⏱️
145 tests 143 ✅ 2 💤 0 ❌
146 runs  144 ✅ 2 💤 0 ❌

Results for commit aa0511e.

♻️ This comment has been updated with latest results.

Copy link
Member

@hyun2371 hyun2371 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다~

@dudxo dudxo merged commit 427ec18 into dev Feb 2, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ feat 기능 추가
Projects
None yet
Development

Successfully merging this pull request may close these issues.

✨ 질문글 작성 전 최소 크레딧 보유 여부
2 participants