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] 유저 프로필(기본정보) 수정 #340

Merged
merged 11 commits into from
Jan 28, 2025

Conversation

m-a-king
Copy link
Collaborator

resolved :

📌 과제 설명

유저의 프로필 업데이트 기능 추가
이는 유저의 회원가입 이후 초기 정보 입력에서도 사용 가능

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

유저의 프로필 업데이트 기능 추가

✅ PR 포인트 & 궁금한 점

- 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 메서드 활용
- 봉사자의 기본 정보를 업데이트하는 테스트 케이스 추가
- 센터의 기본 정보를 업데이트하는 테스트 케이스 추가
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.

수고하셨습니다~


@Tag(name = "User Command API", description = "유저 생성 수정 삭제 관련 API")
@RestController
@Slf4j
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 Author

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("센터 기본 정보 업데이틍 완료");
Copy link
Collaborator

@7zrv 7zrv Jan 28, 2025

Choose a reason for hiding this comment

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

업데이'틍' 이라고 되어있습니다...

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

감사합니다 ㅋㅋㅋㅋ

@7zrv
Copy link
Collaborator

7zrv commented Jan 28, 2025

고생하셨습니다!

@m-a-king m-a-king merged commit b1117c7 into main Jan 28, 2025
3 checks passed
@m-a-king m-a-king deleted the feature/313-add-customize-user-profile branch January 28, 2025 10:51
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] 회원가입 추가정보 입력 (사용자 정보 초기 개인화)
3 participants