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

[FEAT] User 관련 API 구현 완료 #111

Merged
merged 43 commits into from
Nov 19, 2024
Merged

[FEAT] User 관련 API 구현 완료 #111

merged 43 commits into from
Nov 19, 2024

Conversation

simta1
Copy link
Contributor

@simta1 simta1 commented Nov 12, 2024 •
edited
Loading

작성자: @simta1

close #43

체크 리스트

  • 적절한 제목으로 수정했나요?
  • 상단에 이슈 번호를 기입했나요?
  • Target Branch를 올바르게 설정했나요?
  • Reviewers/Assignees/Labels을 알맞게 설정했나요?

작업 내역

  • user api 브랜치입니다.

비고

  • 감사합니다

simta1 added 30 commits August 9, 2024 00:42
@simta1
feat: getMe API 구현
78e9154
@simta1
refactor: userService 함수명 변경
d8c9626
@simta1
feat: updateMe api 구현
ecc3cef
@simta1
feat: deleteMe api 구현
08648cb
@simta1
feat: getMe test, restDocs 추가
e8c67da
@simta1
feat: updateMe test, restDocs 추가
da1a64d
@simta1
feat: deleteMe test, restDocs 추가
3563c53
@simta1
feat: restDocs 추가
be21975
@simta1
feat: UpdateUserRequest validation 추가
8282e39
@simta1
fix: 회원가입 시 연관 엔티티 삭제되는 문제 해결
a7bf211
@simta1
fix: INACTIVE 유저 포함
1583db0
@simta1
fix: getMe에서 학과정보 확인할 때 오류나는 문제 해결
1d98039
@simta1
fix: updateMe 공백 방지, 버그 수정
ae767c7
@simta1
feat: updateMe patch에서 put으로 변경
a98a8cd
@simta1
feat: getUserInquiries, getUserProposals 구현
3b747ef
@simta1
feat: getUserInquiries, getUserProposals restDocs 추가
4d5180d
@simta1
feat: getDepartments api 구현
6ff147b
@simta1
feat: getDepartments restdocs 추가
9e085a7
@simta1
refactor: userUpdateRequest dto 변수명 auth/register과 통일
032db9f
@simta1
feat: userResponse dto에서 socialLoginId 필드 제거
24f30cd
@simta1
fix: updateMe에서 소속, 직책을 빈 문자열로 바꿀 수 있도록 수정
4105a41
@simta1
feat: 유저 삭제 시 좋아요 정보도 삭제되도록 변경
b522d13
@simta1
feat: getUserFavorites api 구현
5319d2e
@simta1
chore: restdocs 중복정보 제외
5b96b52
@simta1
fix: 소속, 직책 업데이트 로직 수정
1b507a3
@simta1
Merge pull request #60 from SystemConsultantGroup/feat/43-user-api/2
e01996e 
[FEAT] user getMe, updateMe, deleteMe API, restdocs 구현
@simta1
Merge branch 'feat/43-user-api/base' into feat/43-user-api/3
1f20d69
@simta1
chore: restdocs 중복정보 제외
02c8e16
@simta1
chore: department restdocs 수정
60ca860
@simta1
feat: transactional 설정 추가
3aebc9c
@simta1 simta1 added 🚀 Feature 기능 개발 🎯 API API 개발 관련 User Domain User 도메인 관련 labels Nov 12, 2024
@simta1 simta1 self-assigned this Nov 12, 2024
@2tle
Copy link
Contributor

2tle commented Nov 12, 2024

#close 43 -> close #43 으로 수정하면 머지할때 43번 이슈도 함께 close됩니다.

yesjuhee
yesjuhee reviewed Nov 16, 2024
View reviewed changes
Copy link
Contributor

@yesjuhee yesjuhee left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!

딱 하나 마지막에 getUserFavorites 변경 내용이 Docs에 반영이 안된 것 같습니다!
user.adoc 수정 후 html 까지 추가해야될 것 같네용

@yesjuhee
Copy link
Contributor

수고하셨습니다!

딱 하나 마지막에 getUserFavorites 변경 내용이 Docs에 반영이 안된 것 같습니다! user.adoc 수정 후 html 까지 추가해야될 것 같네용

추가브랜치 파서 작업하면 될 것 같습니다!

yesjuhee
yesjuhee approved these changes Nov 19, 2024
View reviewed changes
Copy link
Contributor

@yesjuhee yesjuhee left a comment

Choose a reason for hiding this comment

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

수고하셨습니당

@yesjuhee yesjuhee merged commit 0842843 into develop Nov 19, 2024
1 check passed
@yesjuhee yesjuhee deleted the feat/43-user-api/base branch November 19, 2024 09:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yesjuhee yesjuhee yesjuhee approved these changes

Assignees

@simta1 simta1

Labels
🎯 API API 개발 관련 🚀 Feature 기능 개발 User Domain User 도메인 관련
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

🚀 [FEAT] User 관련 API 구현
3 participants
@simta1 @2tle @yesjuhee