-
Notifications
You must be signed in to change notification settings - Fork 1
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] 유저 프로필(기본정보) 수정 #340
Conversation
- VolunteerBasicInfoRequestDto 추가 - 공통 기본 정보(CommonBasicInfoRequestDto), 닉네임, 성별 필드 포함 - CenterBasicInfoRequestDto 추가 - 공통 기본 정보(CommonBasicInfoRequestDto), 홈페이지 링크 필드 포함 - CommonBasicInfoRequestDto 추가 - 이름, 연락처, 이미지 링크, 소개 글 필드 포함
- UserCommonAttribute: CommonBasicInfoRequestDto 기반 업데이트 메서드 추가 - NEWVolunteer: nickname, gender, volunteerStats 업데이트 메서드 추가 - NEWCenter: homepageUrl 업데이트 메서드 추가
- CommonBasicInfoRequestDto 객체 생성 및 활용 추가 - UserCommonAttribute의 customize() 메서드를 update() 메서드로 수정
- UpdateBasicInfoUseCase 인터페이스 정의 - UpdateBasicInfoService 구현 - 사용자 공통 속성 업데이트 로직 추가 - 봉사자 및 센터 속성별 업데이트 로직 포함
- VolunteerBasicInfoRequestDto와 CenterBasicInfoRequestDto를 업데이트하는 메소드에서 roleId 파라미터 삭제
- UserCommandController 클래스 생성 - 봉사자 기본 정보 업데이트 API 추가 - 센터 기본 정보 업데이트 API 추가
- 봉사자 및 센터 기본 정보 업데이트 엔드포인트를 PUT 메서드로 변경 - RESTful API 규칙에 따라 정보 업데이트에 PUT 메서드 활용
- 봉사자의 기본 정보를 업데이트하는 테스트 케이스 추가 - 센터의 기본 정보를 업데이트하는 테스트 케이스 추가
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수고하셨습니다~
|
||
@Tag(name = "User Command API", description = "유저 생성 수정 삭제 관련 API") | ||
@RestController | ||
@Slf4j |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
어노테이션 제거하셔도 될 것 같아요!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
넵! 감사합니다
@RequestBody CenterBasicInfoRequestDto centerBasicInfoRequestDto | ||
) { | ||
updateBasicInfoUseCase.update(userId, centerBasicInfoRequestDto); | ||
return ApiResponse.ok("센터 기본 정보 업데이틍 완료"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
업데이'틍' 이라고 되어있습니다...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
감사합니다 ㅋㅋㅋㅋ
고생하셨습니다! |
Quality Gate passedIssues Measures |
resolved :
📌 과제 설명
유저의 프로필 업데이트 기능 추가
이는 유저의 회원가입 이후 초기 정보 입력에서도 사용 가능
👩💻 요구 사항과 구현 내용
유저의 프로필 업데이트 기능 추가
✅ PR 포인트 & 궁금한 점