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 #177] 질문 글 작성 응답 DTO 잔여 크레딧 추가 #180

Merged
merged 3 commits into from
Jan 1, 2025

Conversation

dudxo
Copy link
Collaborator

@dudxo dudxo commented Dec 31, 2024

관련 이슈

📑 작업 상세 내용

  • 질문 글 작성 응답 DTO 잔여 크레딧 추가
    -toRegisterQuestionPostResponse()Member member = questionPost.getMember()을 이용한 credit 조회 시 변경감지 적용 전으로 인해 잔여 크레딧 정합성 오류 발생 -> 실제 크레딧이 차감된 Member를 직접 파라미터로 넘겨서 사용하도록 변경

💫 작업 요약

  • 질문 글 작성 응답 DTO 잔여 크레딧 추가

🔍 중점적으로 리뷰 할 부분

@dudxo dudxo added the ✨ feat 기능 추가 label Dec 31, 2024
@dudxo dudxo requested a review from hyun2371 December 31, 2024 16:30
@dudxo dudxo self-assigned this Dec 31, 2024
@dudxo dudxo linked an issue Dec 31, 2024 that may be closed by this pull request
1 task
@dudxo
Copy link
Collaborator Author

dudxo commented Dec 31, 2024

@hyun2371 개발/운영서버 이분화에 따른 개발/운영 브랜치 이분화도 필요할 것 같습니다!

Copy link

Code Coverage

Overall Project 83.59% 🍏
Files changed 100% 🍏

File Coverage
QuestionPostService.java 77.27% 🍏

Copy link

Test Results

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

Results for commit 13212ba.

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.

다른 DTO도 remainingCredit으로 수정해서 통일시킬까 고민이 되네요.. 고생 많으셨습니다!

@hyun2371
Copy link
Member

hyun2371 commented Jan 1, 2025

@hyun2371 개발/운영서버 이분화에 따른 개발/운영 브랜치 이분화도 필요할 것 같습니다!

prod 브랜치 생성했습니다! 주기적으로 dev->prod pr 올려서 배포하면 될 것 같아요

@dudxo
Copy link
Collaborator Author

dudxo commented Jan 1, 2025

다른 DTO도 remainingCredit으로 수정해서 통일시킬까 고민이 되네요.. 고생 많으셨습니다!

단순 credit을 하려다가 잔여 크레딧 표시에 쓰는 데이터다보니 remainingCredit으로 했었어요. 다른 곳은 쓰이는 기능에 맞게 네이밍하는게 좋을 것 같아요!

@dudxo dudxo merged commit b8991ed into dev Jan 1, 2025
3 checks passed
@dudxo dudxo deleted the feat/177/questionpost-remaining-credit branch January 1, 2025 10:44
@dudxo dudxo restored the feat/177/questionpost-remaining-credit branch January 1, 2025 10:44
@dudxo dudxo deleted the feat/177/questionpost-remaining-credit branch January 1, 2025 10:44
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.

✨ 게시글 작성 응답 DTO 잔여 크레딧 추가
2 participants