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.

Sign up for GitHub

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

Jump to bottom

Feature/#69 내 커리어를 조회하는 api 구현 #70

Merged
merged 17 commits into from
Jul 23, 2023
Merged

Feature/#69 내 커리어를 조회하는 api 구현 #70

merged 17 commits into from
Jul 23, 2023

Conversation

hong-sile
Copy link
Collaborator

@hong-sile hong-sile commented Jul 20, 2023 •
edited by java-saeng
Loading

#️⃣연관된 이슈

📝작업 내용

  • 사용자 본인의 커리어들을 조회하는 API 구현 및 문서화 테스트 작성

close #69

@java-saeng
feat : member id로 사용자의 memberCareer들을 조회하는 기능 추가
7c336a6 
#49
@java-saeng
feat : 내 명함에서 활동을 추가하는 API 구현
9f6731e 
#49
@java-saeng
docs : 내 명함에서 활동을 추가하는 http 테스트 추가
8b8e36a 
#49
@java-saeng
refactor : MemberCareerRequest -> MemberCareerAddRequest로 변경
a997ffc 
- 나중에 Delete에서 DTO Request를 구분하기 위함

#49
@java-saeng
feat : 내 명함에서 활동을 삭제하는 API 구현
8485bb6 
#49
@java-saeng
feat : 내 명함에서 활동을 삭제하는 http 테스트 추가
6330966 
#49
@java-saeng
refactor : member Id를 통해 memberCareer를 조회하지 않고 member를 통해 memberCaree…
15ab790 
…r 조회로 변경

#49
@java-saeng
Merge branch 'main' of https://github.com/woowacourse-teams/2023-emmsale
0d98894 
 into Feature/#49-내_명함에서_활동을_추가_삭제하는_기능
@java-saeng
feat : 요청한 career id가 유효하지 않을 때 Exception 추가
56fb089
@java-saeng
feat : 커리어 추가 및 삭제 API 문서화
aee6b17 
#49
@java-saeng
Merge branch 'main' of https://github.com/woowacourse-teams/2023-emmsale
cec893d 
 into Feature/#49-내_명함에서_활동을_추가_삭제하는_기능
@java-saeng
feat : 존재하는 커리어 전체 조회 API 문서화
2f75d32 
#66
@java-saeng
feat : 초기 사용자 등록 API 문서화
4738ad9 
#66
@java-saeng
feat : 사용자의 커리어 조회 API 구현
0f2cde7 
#69
@java-saeng
feat : 사용자의 커리어 조회 API 문서화 테스트 작성
1592b37 
#69
@java-saeng
docs : 사용자 커리어 조회 API 문서 추가
87e635a 
#69
@hong-sile hong-sile added Backend 백엔드 관련 이슈 기능 추가 새로운 기능 추가 및 기존 기능 변경 labels Jul 20, 2023
hyeonjerry
hyeonjerry approved these changes Jul 21, 2023
View reviewed changes
Copy link
Collaborator

@hyeonjerry hyeonjerry left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!

backend/emm-sale/src/main/java/com/emmsale/member/application/MemberCareerService.java Outdated
@@ -81,5 +81,10 @@ public List<MemberCareerResponse> deleteCareer(

return MemberCareerResponse.from(memberCareerRepository.findAllByMember(member));
}

@Transactional(readOnly = true)
public List<MemberCareerResponse> findCareer(final Member member) {
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.

반영했습니다!

amaran-th
amaran-th approved these changes Jul 22, 2023
View reviewed changes
Copy link
Collaborator

@amaran-th amaran-th left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!!
흠잡을 데가 없네요~~!

@java-saeng java-saeng merged commit b90e2c4 into backend-main Jul 23, 2023
1 check passed
@hyeonjerry hyeonjerry deleted the Feature/#69-내_커리어를_조회하는_API_구현 branch July 31, 2023 04:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hyeonjerry hyeonjerry hyeonjerry approved these changes

@amaran-th amaran-th amaran-th approved these changes

Assignees

@hong-sile hong-sile

@java-saeng java-saeng

Labels
Backend 백엔드 관련 이슈 기능 추가 새로운 기능 추가 및 기존 기능 변경
Projects
2023-emmsale
Status: Archive
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@hong-sile @hyeonjerry @amaran-th @java-saeng