Skip to content

Conversation

@7zrv
Copy link
Collaborator

@7zrv 7zrv commented Dec 7, 2024

📌 과제 설명

기관/봉사자의 자신에게 온 쪽지 상세 조회 기능입니다.

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

  • 요구사항에 맞는 mapper 클래스 생성
  • 쿼리 메서드 구현
  • 서비스 레이어 메서드 구현
  • 컨트롤러 구현
  • NoSuchElement 예외 구현
  • 예외 메세지 추가

✅ PR 포인트 & 궁금한 점

7zrv added 7 commits December 8, 2024 03:22
- 쿼리 메서드 생성
- mapper record 클래 생성
- 테스트 코드 작성및 검증 완료
- 쿼리 메서드 생성
- mapper record 클래스 생성
- 테스트 코드 작성및 검증 완료
- NoSuchElementException 클래스 추가
- 예외메세지 추가
- 전역 예외처리 핸들러에 NoSuchElementException 추가
- ExceptionMessage enum 값의 내부 메세지가 아닌 ExceptionMessage enum 자체를 이용하여 생성하도록 수정
- 기관의 쪽지 상세보기 메서드 구현
- 봉사자의 쪽지 상세보기 메서드 구현
- 테스트 코드 작성및 검증 완료
- 컨트롤러 구현
- 테스트 코드 작성및 검증 완료
@7zrv 7zrv self-assigned this Dec 7, 2024
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.

수고하셨습니다~

}

@Secured("ROLE_VOLUNTEER")
@Operation(summary = "기관의 자신에게 온 쪽지 상세 조회")
Copy link
Collaborator

Choose a reason for hiding this comment

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

봉사자가 맞는 것 같아요~

note.content,
volunteer.id.as(SENDER_ID),
volunteer.nickname.as(SENDER_NAME),
volunteer.imgUrl.as("senderProfileImgLink"),
Copy link
Collaborator

Choose a reason for hiding this comment

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

이것도 상수로 처리할 수 있을 것 같아요

Copy link
Collaborator

@leebs0521 leebs0521 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다. 재중님 말씀한것만 고치면 되겠네요.

- 컨트롤러 swagger 설명 수정
- 매직 스트링의 상수화
@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 8, 2024

@7zrv 7zrv merged commit baa0659 into main Dec 8, 2024
3 checks passed
@7zrv 7zrv deleted the feature/188-add-get-note-detail branch December 8, 2024 06:07
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.

4 participants