Skip to content

Conversation

@m-a-king
Copy link
Collaborator

@m-a-king m-a-king commented Dec 1, 2024

resolved:


📌 과제 설명

  • 봉사자 프로필 수정 기능을 추가하였습니다.
  • 사용자는 닉네임, 소개글, 프로필 이미지를 수정할 수 있습니다.
  • RequestDto에 대한 유효성 검사가 추가되었고, 관련 테스트도 구현하였습니다.

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

  1. 봉사자 업데이트 (domain, usecase-service, controller)

  2. build(gradle): validation 의존성 추가

    • 유효성 검사를 위해 spring-boot-starter-validation을 추가하였습니다.
  3. feat(GlobalExceptionHandler): 유효성 예외를 전역 예외 핸들러로 처리

    • MethodArgumentNotValidException 발생 시 사용자에게 명확한 오류 메시지를 제공하도록 GlobalExceptionHandler를 수정하였습니다. 일관성을 위해서 기존의 상속 구조를 포기했습니다.
  4. Volunteer 패키지 내의 명확하지 않은 클래스들 Rename.


PR 포인트 & 궁금한 점

  1. 닉네임 10자, 소개 100자가 적절하지 않다면 알려주세요!

@m-a-king m-a-king self-assigned this Dec 1, 2024
@m-a-king m-a-king linked an issue Dec 1, 2024 that may be closed by this pull request
1 task
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.

LGTM!!

class VolunteerProfileCommandControllerTest extends ControllerTestSupport {

@Autowired
private MockMvc mockMvc;
Copy link
Collaborator

Choose a reason for hiding this comment

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

controllerTestSupport에 들어있어서 지우셔도 됩니다!

Copy link
Collaborator Author

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 1, 2024

@m-a-king m-a-king merged commit f14f2f1 into main Dec 1, 2024
2 checks passed
@m-a-king m-a-king deleted the feature/97-add-volunteer-profile-update branch December 1, 2024 17:38
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] 봉사자 프로필 수정 기능

5 participants