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

사용자 이메일 등록 및 검증 구현 #9

Open
3 of 4 tasks
hyejungg opened this issue Sep 19, 2021 · 0 comments · Fixed by #12
Open
3 of 4 tasks

사용자 이메일 등록 및 검증 구현 #9

hyejungg opened this issue Sep 19, 2021 · 0 comments · Fixed by #12
Assignees
@hyejungg
Labels
feature 기능 구현
Milestone
사용자 관련 기능 구현

Comments

@hyejungg
Copy link
Collaborator

hyejungg commented Sep 19, 2021 •
edited
Loading

  • 사용자 이메일 등록
  • 사용자 이메일 검증 (이미 있는 이메일인지 검증)
  • 통합테스트 코드 추가
  • 유닛테스트 코드 추가
@hyejungg hyejungg added the feature 기능 구현 label Sep 19, 2021
@hyejungg hyejungg self-assigned this Sep 19, 2021
hyejungg added a commit that referenced this issue Sep 20, 2021
@hyejungg
[feat:#9] 이메일 등록, 검증
66adc06 
이메일 등록 후 중복, 타입 일치하지 않은 경우 모두 확인 완료
hyejungg added a commit that referenced this issue Sep 21, 2021
@hyejungg
[fix:#9] 피드백 바탕으로 수정
aab510e 
피드백을 바탕으로 크게 다음과 같이 수정하였습니다.

1. `@Vaildated` 의존성 라이브러리를 추가하여 유효성 검증하도록 수정하면서 관련 메소드 삭제, UserDto에 `@Email` 어노테이션 추가
2. `GlobalExceptionHandler`에서 ~Exception 이라는 사용자 정의 예외를 받아서 처리되도록 수정
3. 예외 혹은 성공 return 시 postman에서 Status 부분이 모두 200 OK로 지정되는 문제가 있어서 각각 ResponseEntity로 return하도록 수정하였음...
hyejungg added a commit that referenced this issue Sep 21, 2021
@hyejungg
[fix:#9] 피드백 바탕으로 재수정
fc688fb 
빠른 피드백 감사합니다 !!!!!

1. `UserService` 불필요한 if-else문 삭제
2. `UserController` 불필요한 null 확인 처리 삭제
3. /test 테스트 코드를 기능 별로 메소드를 생성하여 테스트하도록 수정(UserServiceTest의 타입오류로 인한 테스트는 추후에 수정 필요해 보임..)
hyejungg added a commit that referenced this issue Sep 23, 2021
@hyejungg
Merge pull request #12 from comento-backend-camp/feature/9
4b51789 
[feat:#9] 이메일 등록, 검증
@hyejungg hyejungg linked a pull request Jan 8, 2022 that will close this issue
[feat:#9] 이메일 등록, 검증 #12
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hyejungg hyejungg

Labels
feature 기능 구현
Projects
None yet
Milestone
사용자 관련 기능 구현
Development

Successfully merging a pull request may close this issue.

[feat:#9] 이메일 등록, 검증 comento-backend-camp/ticket-reservation-server-hyejung
1 participant
@hyejungg