Skip to content

Conversation

@leebs0521
Copy link
Collaborator

resolved :

📌 과제 설명

👩‍💻 요구 사항과 구현 내용

  • 특정 봉사 활동에 대한 봉사자 지원 조회
  • 봉사 활동 지원자 리스트 정보 조회 (기관)
  • 특정 봉사자가 지원한 봉사 리스트 조회
  • 봉사 활동 지원자 통계 조회

✅ PR 포인트 & 궁금한 점

VolunteerApplyQueryFacadeService 한번씩 봐주시면 감사하겠습니다.

  • 원래 VolunteerApplyQueryService 에서 아래를 구현하려함
    • 특정 봉사자 지원 리스트 기능 -> 'VolunteerQueryUseCase' 의존
    • 특정 봉사 활동에 대한 지원자 리스트 기능 -> 'RecruitBoardQueryUseCase' 의존
      -> 순환 참조 문제가 발생해서 파사드를 만들었습니다.

 - 봉사 지원자 정보에 노출해야는 데이터들이 volunteer, detail 혼재
 - mapper VolunteerSimpleInfo 정의
 - 봉사 지원자 정보에 노출해야는 데이터들이 volunteer, detail 혼재
 - mapper VolunteerSimpleInfo 정의
 - **순환 참조로 인한 파사드 도입**
 - VolunteerApplyQueryFacade**
 - **순환 참조로 인한 파사드 도입**
 - VolunteerApplyQueryFacade**
 - 지원 리스트 정보에 필요한 기능 추가
 - 지원 리스트 정보에 필요한 기능 추가
 - 객체를 가져오고 현 서비스에서 응답 DTO로 감싸주도록 변경
@leebs0521 leebs0521 linked an issue Dec 4, 2024 that may be closed by this pull request
4 tasks
Copy link
Collaborator

@m-a-king m-a-king left a comment

Choose a reason for hiding this comment

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

진짜 수고 많으십니다. 범수 님은 몸이 두 개신가요?

Copy link
Collaborator

@ayoung-dev ayoung-dev left a comment

Choose a reason for hiding this comment

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

수고하셨습니당

@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 5, 2024

@leebs0521 leebs0521 merged commit e517b99 into main Dec 5, 2024
3 checks passed
@leebs0521 leebs0521 deleted the feature/138-add-volunteer-apply-query branch December 5, 2024 00:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEATURE] 봉사 지원 조회 기능

4 participants