[NB-248] 사용자 약관 동의 기능 구현 #65
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
사용자의 개인정보 처리 방침과 서비스 이용 약관 동의를 처리하는 기능 구현 주요 변경사항: - User 엔티티에 약관 동의 필드 추가 - privacyAgreement : 개인정보 처리 방침 동의 - serviceAgreement : 서비스 이용 약관 동의 - API 엔드 포인트 구현 - PATCH "/v1/users/agreement" - 사용자별 약관 동의 상태 업데이트 구현 상세: - Repository : 약관 동의 상태 업데이트 쿼리 추가 - Service : 약관 동의 처리 및 예외 처리 로직 구현 - Controller : 약관 동의 엔드포인트 구현 - DTO : 약관 동의 요청/응답 구조 정의 - Exception : 다중 사용자 업데이트 방지를 위한 예외 정의
liljoon
requested changes
Nov 4, 2024
약관 동의 업데이트 실패 시 예외 던지도록 수정 주요 변경사항: - 약관 동의 처리 로직 수정 - 실패 시 UserAgreementUpdateFailException 발생 구현 상세: - Service : 약관 동의 실패 처리 로직 수정 - Exception : 사용자 약관 동의 업데이트 실패를 위한 예외 정의
약관 동의 업데이트 실패 시 발생되는 예외의 상태코드를 404 -> 500 으로 변경함 변경 사유: - 업데이트 되지 않았다는 의미에서 404 처리를 하려 했으나, 팀원과의 논의 상 업데이트 되지 않는 상황이라면 userId가 잘못된 것일 텐데 access token의 userId가 잘못되었다면 서버 측의 발급이 잘못되었을 것이다 라는 논의가 있어 수정함.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
개요
NB-248
주요 변경사항:
User 엔티티에 약관 동의 필드 추가
privacyAgreement : 개인정보 처리 방침 동의
serviceAgreement : 서비스 이용 약관 동의
API 엔드 포인트 구현
PATCH "/v1/users/agreement"
사용자별 약관 동의 상태 업데이트
구현 상세:
PR 유형
어떤 변경 사항이 있나요?
PR Checklist
PR이 다음 요구 사항을 충족하는지 확인하세요.