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

[hoony-lab] 사용자 업데이트 #44

Open
hoony-lab opened this issue Sep 5, 2021 · 0 comments
Open

[hoony-lab] 사용자 업데이트 #44

hoony-lab opened this issue Sep 5, 2021 · 0 comments
Assignees
@hoony-lab
Labels
hoony-lab 후니의 이슈

Comments

@hoony-lab
Copy link
Member

AS-IS

  • SecurityContextHolder

TO-DO

  • @AuthenticationPrinciple
  • PUT /api/user
@hoony-lab hoony-lab self-assigned this Sep 5, 2021
@hoony-lab hoony-lab added the hoony-lab 후니의 이슈 label Sep 5, 2021
hoony-lab added a commit that referenced this issue Sep 5, 2021
@hoony-lab
[#44] feat: add UserUpdateRequest
e62bf5e
hoony-lab added a commit that referenced this issue Sep 5, 2021
@hoony-lab
[#44] feat: add updateUser in User
de002a3 
check all params and update fields which is nonNul
hoony-lab added a commit that referenced this issue Sep 5, 2021
@hoony-lab
[#44] feat: add update in UserService
29f820e 
IDK
hoony-lab added a commit that referenced this issue Sep 5, 2021
@hoony-lab
[#44] feat: add updateUser in UserController
ab73728 
use `@AuthenticationPrincipal` String principal to retrieve current login user info
hoony-lab added a commit that referenced this issue Sep 5, 2021
@hoony-lab
[#44] test: add update, update-exception in UserServiceTest
02f7f73 
update
update_validateExistUser_exception

some refactoring
hoony-lab added a commit that referenced this issue Sep 5, 2021
@hoony-lab
[#44] test: add PUT_api_user_updateUser in UserControllerTest
6bdaa5d 
`SecurityContextHolder.getContext().setAuthentication(new Authentication());`

some refactoring
hoony-lab added a commit that referenced this issue Sep 5, 2021
@hoony-lab
[#44] fix
202b634 
umm
@hoony-lab hoony-lab mentioned this issue Sep 5, 2021
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hoony-lab hoony-lab

Labels
hoony-lab 후니의 이슈
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@hoony-lab