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

[Feature] 이동봉사자 봉사 관리 - 목록 조회 API 수정 #158

Merged
merged 4 commits into from
Apr 17, 2024

Conversation

kyeong-hyeok
Copy link
Member

💡 연관된 이슈

close #157

📝 작업 내용

  • 이동봉사자 승인 대기중 목록 조회 API 픽업 시간 추가
  • 이동봉사자 진행중 목록 조회 API 픽업 시간 추가
  • 이동봉사자 봉사 완료 목록 조회 API 근황 id 삭제 및 픽업 시간 추가
  • 이동봉사자 봉사 관리 - 목록 조회 테스트 코드 수정

💬 리뷰 요구 사항

@kyeong-hyeok kyeong-hyeok added ✨ Feature 기능 개발 Priority: Medium 우선순위 중간 🐯 Koeyhk 담당자 labels Apr 16, 2024
@kyeong-hyeok kyeong-hyeok linked an issue Apr 16, 2024 that may be closed by this pull request
4 tasks
Copy link
Member

@hojeong2747 hojeong2747 left a comment

Choose a reason for hiding this comment

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

2차 개발 들어가고 코드리뷰 오랜만에 남기니 좀 설레네요..? ㅎㅎ
고생하셨습니다~!

아, 그리고 Dto 클래스에 ~completed 이런식으로 지은 거 직관적이고 너무 좋네요. 오랜만에 몇 달 전 작성한 남의 코드 보는데도 불구하고 기획서와 바로 매핑되어 아주 이해가 잘 돼요. 다들 클래스명 짓는 스타일도 다른데, 저는 코넥독 스타일이 좋아서 회사에서도 그렇게 쓰고 있답니다👍

String intermediaryName, Boolean isKennel,
Long reviewId, Long dogStatusId) {
Copy link
Member

Choose a reason for hiding this comment

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

근황이 빠진 거 반영하고,

@@ -9,6 +9,7 @@ public record ApplicationVolunteerProgressingResponse(Long postId, String mainIm
LocalDate startDate,
@JsonFormat(shape = JsonFormat.Shape.STRING, pattern = "yyyy-MM-dd", timezone = "Asia/Seoul")
LocalDate endDate,
String pickUpTime,
Copy link
Member

Choose a reason for hiding this comment

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

근황이 빠지고, 픽업시간까지 추가해 목록 조회하기 구현하셨네요!

@kyeong-hyeok
Copy link
Member Author

2차 개발 시작하면서 느낀 건데, 응답 값을 수정하면서 변경해야 하는 코드가 적어 좋았습니다!
비즈니스 로직을 처리하는 부분과 응답 값에 대한 부분을 나누고 repository에도 impl을 만들어 변경하기 용이하게 만든 부분이 좋은 거 같아요!

@kyeong-hyeok kyeong-hyeok merged commit e32f45f into develop Apr 17, 2024
1 check passed
@kyeong-hyeok kyeong-hyeok deleted the feat/157-application-pick-up-time-add branch April 17, 2024 23:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feature 기능 개발 🐯 Koeyhk 담당자 Priority: Medium 우선순위 중간
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature] 이동봉사자 봉사 관리 - 목록 조회 API 수정
2 participants