feat: 회원정보(학번, 전화번호) 수정 API 삭제 #156
Conversation
JanooGwan
force-pushed
the
delete/user-info-update-api
branch
from
a46f42e to
af55e3f
Compare
dh2906
left a comment
dh2906
left a comment
There was a problem hiding this comment.
고생하셨습니다.
DTO는 다른 곳에서도 사용하고 있어서 제거하지 않은건가요??
There was a problem hiding this comment.
Pull request overview
이 PR은 회의 결정에 따라 회원정보(학번, 전화번호) 수정 API를 제거합니다. API 엔드포인트, 서비스 로직, 검증 메서드들이 삭제되었습니다.
Changes:
- UserApi 인터페이스에서 PUT /me 엔드포인트 정의 제거
- UserController에서 updateMyInfo 메서드 구현 제거
- UserService에서 updateUserInfo 및 관련 검증 메서드들 제거
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 5 comments.
| File | Description |
|---|---|
| src/main/java/gg/agit/konect/domain/user/controller/UserApi.java | PUT /me API 엔드포인트 정의와 관련 import 제거 |
| src/main/java/gg/agit/konect/domain/user/controller/UserController.java | updateMyInfo 메서드 구현과 관련 import 제거 |
| src/main/java/gg/agit/konect/domain/user/service/UserService.java | updateUserInfo 메서드와 검증 로직(validateStudentNumberDuplication, validatePhoneNumberDuplication), UserUpdateRequest import 제거 |
| import gg.agit.konect.domain.notice.repository.CouncilNoticeReadRepository; | ||
| import gg.agit.konect.domain.studytime.service.StudyTimeQueryService; | ||
| import gg.agit.konect.domain.university.model.University; | ||
| import gg.agit.konect.domain.university.repository.UniversityRepository; |
There was a problem hiding this comment.
UserUpdateRequest DTO 파일이 더 이상 사용되지 않으므로 삭제해야 합니다. 현재 src/main/java/gg/agit/konect/domain/user/dto/UserUpdateRequest.java 파일이 여전히 존재하지만 어디서도 참조되지 않습니다.
| Long unreadCouncilNoticeCount = councilNoticeReadRepository.countUnreadNoticesByUserId(user.getId()); | ||
| Long studyTime = studyTimeQueryService.getTotalStudyTime(userId); | ||
|
|
||
| return UserInfoResponse.from(user, joinedClubCount, studyTime, unreadCouncilNoticeCount, isClubManager); |
There was a problem hiding this comment.
User 모델의 updateInfo 메서드가 더 이상 사용되지 않습니다. 이 메서드는 삭제된 updateUserInfo 서비스 메서드에서만 호출되었으므로, User.java 파일에서도 제거하는 것을 고려해야 합니다.
| Long unreadCouncilNoticeCount = councilNoticeReadRepository.countUnreadNoticesByUserId(user.getId()); | ||
| Long studyTime = studyTimeQueryService.getTotalStudyTime(userId); | ||
|
|
||
| return UserInfoResponse.from(user, joinedClubCount, studyTime, unreadCouncilNoticeCount, isClubManager); |
There was a problem hiding this comment.
User 모델의 hasSameStudentNumber와 hasSamePhoneNumber 메서드가 더 이상 사용되지 않습니다. 이 메서드들은 삭제된 검증 로직에서만 사용되었으므로, User.java 파일에서도 제거하는 것을 고려해야 합니다.
| Long unreadCouncilNoticeCount = councilNoticeReadRepository.countUnreadNoticesByUserId(user.getId()); | ||
| Long studyTime = studyTimeQueryService.getTotalStudyTime(userId); | ||
|
|
||
| return UserInfoResponse.from(user, joinedClubCount, studyTime, unreadCouncilNoticeCount, isClubManager); |
There was a problem hiding this comment.
UserRepository의 existsByUniversityIdAndStudentNumberAndIdNot 메서드가 더 이상 사용되지 않습니다. 이 메서드는 삭제된 validateStudentNumberDuplication 메서드에서만 호출되었으므로, UserRepository.java에서도 제거하는 것을 고려해야 합니다.
| Long unreadCouncilNoticeCount = councilNoticeReadRepository.countUnreadNoticesByUserId(user.getId()); | ||
| Long studyTime = studyTimeQueryService.getTotalStudyTime(userId); | ||
|
|
||
| return UserInfoResponse.from(user, joinedClubCount, studyTime, unreadCouncilNoticeCount, isClubManager); |
There was a problem hiding this comment.
UserRepository의 existsByPhoneNumberAndIdNot 메서드가 더 이상 사용되지 않습니다. 이 메서드는 삭제된 validatePhoneNumberDuplication 메서드에서만 호출되었으므로, UserRepository.java에서도 제거하는 것을 고려해야 합니다.
2 participants
어제 회의한 내용에 따라,
회원정보(학번, 전화번호) 수정 API 삭제했습니다.
@dh2906