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/#43/내가 요청한 정산 리스트 api 개발 #46

Merged

Conversation

seongjunnoh
Copy link
Collaborator

📝 요약

내가 요청한 정산 view를 구성하는 데이터를 반환하기 위한 api를 개발하였습니다
유저가 요청한 정산 중 현재 진행중인 정산과 완료된 정산을 구분하여 response dto 를 구성하였고, 이에 대한 간단한 테스트 코드를 작성하여 테스트가 통과하는 것을 확인하였습니다

이슈 번호 : #43

🔖 변경 사항

기존 PayService 에 유저가 요청한 정산에 대한 정보와, 유저가 요청받은 정산에 대한 정보를 받아오는 코드가 있었습니다.
이 코드에 각각 정산의 상태가 현재 진행중인지 완료된 정산인지를 구분하여 정산 정보를 return 할 수 있도록 boolean isComplete 파라미터를 추가하였습니다

✅ 리뷰 요구사항

📸 확인 방법 (선택)

PatServiceTest 클래스에서 테스트 코드를 참고하시면 됩니다.
현재 유저가 같은 스페이스 내의 다른 유저에게 요청한 정산이 현재 진행중인 경우, 완료된 경우에 대한 테스트 코드를 작성해두었습니다.



📌 PR 진행 시 이러한 점들을 참고해 주세요

* P1 : 꼭 반영해 주세요 (Request Changes) - 이슈가 발생하거나 취약점이 발견되는 케이스 등
* P2 : 반영을 적극적으로 고려해 주시면 좋을 것 같아요 (Comment)
* P3 : 이런 방법도 있을 것 같아요~ 등의 사소한 의견입니다 (Chore)

@seongjunnoh seongjunnoh linked an issue Jul 26, 2024 that may be closed by this pull request
3 tasks
Copy link
Collaborator

@drbug2000 drbug2000 left a comment

Choose a reason for hiding this comment

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

LGTM

@seongjunnoh seongjunnoh merged commit 96f7570 into develop Jul 28, 2024
3 checks passed
@hyunn522 hyunn522 deleted the Feat/#43/내가-요청한-정산-리스트-api-개발 branch July 30, 2024 12:27
seongjunnoh added a commit that referenced this pull request Oct 30, 2024
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.

Feat : 내가 요청한 정산 리스트 api 개발
2 participants