feat: 멘토 마이페이지 조회/수정 #375
feat: 멘토 마이페이지 조회/수정 #375
Conversation
nayonsoso
requested review from
Gyuhyeok99,
lsy1307,
whqtker and
wibaek
as code owners
This comment was marked as outdated.
This comment was marked as outdated.
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
nayonsoso
force-pushed
the
feat/374-mentor-my-page
branch
from
1b2efe3 to
68b4cae
Compare
Gyuhyeok99
left a comment
Gyuhyeok99
left a comment
There was a problem hiding this comment.
고생하셨습니다!
채널 관련해서는 어떻게 할지 고민이 되니 회의 때 이야기를 더 나눠보면 좋을 거 같습니다~
추가로 기본 정보 수정도 같이 포함하는 것 좋은 거 같습니다! bruno에만 수정 pr 올려주시면 될 거같습니다!
whqtker
left a comment
whqtker
left a comment
There was a problem hiding this comment.
고생하셨습니다 ! 코멘트 남겼습니다.
저희가 현재 4개의 채널이 있는데, 피그마를 확인하니 4개를 초과하는 채널 and 중복된 채널은 가질 수 없는 것 같습니다. 두 가지 케이스에 대한 추가적인 처리가 있으면 좋을 거 같습니다 ! 잘못 알고 있다면 말씀해주세요
src/test/java/com/example/solidconnection/mentor/repository/MentorBatchQueryRepositoryTest.java
Outdated
Show resolved
Hide resolved
src/main/java/com/example/solidconnection/mentor/controller/MentorMyPageController.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
✍️ 추가 커밋 내용
- 성혁님이 리뷰하신 내용 반영했습니다.
- 불필요한 중괄호 제거
- 사용되지 않는 변수 제거
- 최대 채널 등록 갯수 검증 추가
- 누락한 Dto validation 메세지 추가했습니다.
- 기획팀께 답변받은 내용에 대해, “멘토 페이지에서 사용자 정보 수정하는게 아닌, 마이페이지에서 수정한다”는 내용 반영했습니다. (myPageService에 수정을 위임했기에 코드 변경이 적을 수 있었네요. 휴~)
❗️회의에서는 채널 업데이트를 할 때, ChannelRepository.save()를 호출해야한다고 했었지만❗️
Mentor 에서 List를 CascadeType.ALL로 참조하기 있기 때문에,
mentor 엔티티에 channel 을 추가할 때 영속성 컨텍스트에 올라가서, insert 시에도 반영된다고 합니다.
ChannelRepository에 추가해야한다는 것은 제가 잘못 알고 그렇게 말한 것이었습니다. 😅
따라서 처음 코드 그대로 냅둡니다!
src/main/java/com/example/solidconnection/mentor/controller/MentorMyPageController.java
Outdated
Show resolved
Hide resolved
| @PutMapping | ||
| public ResponseEntity<String> updateMentorMyPage( | ||
| @AuthorizedUser SiteUser siteUser, | ||
| @Valid MentorMyPageUpdateRequest mentorMyPageUpdateRequest |
| private static final int CHANNEL_REGISTRATION_LIMIT = 4; | ||
| private static final int CHANNEL_SEQUENCE_START_NUMBER = 1; | ||
|
|
||
| private final MyPageService myPageService; |
There was a problem hiding this comment.
MyPageService는 이제 주입 안받아도 될 거 같습니다!
whqtker
left a comment
whqtker
left a comment
There was a problem hiding this comment.
변경사항 확인했습니다 ~ approve합니다 !!!! 고생하셨습니다
lsy1307
left a comment
lsy1307
left a comment
There was a problem hiding this comment.
고생하셨습니다! 의문점만 한가지 있는데 답변해주시면 감사할 것 같습니다!
|
|
||
| import static com.example.solidconnection.common.exception.ErrorCode.DATA_INTEGRITY_VIOLATION; | ||
|
|
||
| @Repository |
There was a problem hiding this comment.
제 기억에 @ Repository 작성안하는 걸로 했던 것 같은데 맞을까요...?? 맞다면 지워도 될 것 같아용
There was a problem hiding this comment.
아 JPA Repository가 아니군요... 근데 이 부분은 왜 Service가 아니라 Repository라고 하셨는지 궁금하네요
There was a problem hiding this comment.
오! 좋은 질문 주셔서 감사합니다.
N+1이 발생하지 않게 ❗️자바 코드로 fetch join과 비슷한 동작을 구현❗️ 했기 때문에, repository라고 생각했습니다.
nayonsoso
force-pushed
the
feat/374-mentor-my-page
branch
from
579c868 to
9c37b76
Compare
4 participants
관련 이슈
작업 내용
PR #370 머지 이후에 작업하려고 했는데,
수연님이 아직 머지 안 된 브랜치 위에서 작업하신 걸 PR로 올리신걸 보고
‘나도 기다릴 필요 없겠다! 저렇게 하면 코드 리뷰를 기다리는 시간이 줄어들겠다!’ 라는 생각을 하게 됐습니다 😊👍
"멘토 마이 페이지 조회/수정"에만 해당하는 병경 사항 보러가기 ➡️ 변경사항
특이 사항
"멘토의 마이 페이지 수정"에 대한 저의 해석대로 구현한 부분이 있습니다.
API 문서 회의 때는 멘토의 합격 팁, 자기소개, 채널만 수정하는 줄 알았는데,
피그마를 보다 보니 “그럼 멘토 닉네임/프로필은 어디서 고치는 거지?” 싶더라고요.
그래서 같은 페이지에서 닉네임과 프로필 이미지도 함께 수정하는 것으로 추측해 구현했습니다.
현재 기획팀에 디스코드로 확인 요청해둔 상태예요!