[내 프로필 수정] 내 프로필을 수정할 수 있음(#38) #67

Minjae-An · 2024-04-17T09:52:51Z

작업 대상

내 프로필 수정 비즈니스 로직
내 프로필 수정 API
이메일 연관 로직

📄 작업 내용

내 프로필 수정 비즈니스 로직 및 API 구성
나이, 키 관련 검증 로직 추가 및 변경, 연관 로직 수정
사용자 엔티티의 불필요한 이메일 필드 삭제, 연관 로직 삭제

🙋🏻 주의 사항

나이의 경우 20~40세 범위 내인 지 검증, 선호하는 나이 하한/상한에도 적용
키의 경우 130~220cm 범위 내인 지 검증, 선호하는 키 하한/상한에도 적용
선호하는 키, 나이의 경우 범위 형태로 하한/상한을 저장
사용자가 하한/상한 중 하나의 조건만 설정할 수 있음, 이에 따라 nullable하게 요청 가능하도록 구성
하한/상한 값이 존재할 경우 0 이상이어야 하도록 검증
하한/상한 값이 둘 다 존재할 경우 하한 <= 상한을 충족하는 지 검증

📎 관련 이슈

조회 로직에서 null로 저장된 하한/상한 값을 고려할 예정

레퍼런스

병합 과정에서 잘못 설정된 enum 통합

기본 타입으로 설정할 경우 명세시 @NotNull 제약 조건이 적용되지 않고 null과 매핑될 경우 예외 가능. 이에 기존 기본 타입 필드들 일괄적으로 wrapper 타입으로 변경

- 성인이 아닌 경우 - 하한 값이 상한 값보다 큰 경우

- 요구사항 변경에 따른 필드 추가 - 필드별 검증 조건 추가 - 명세 수정

내 프로필 수정은 다음 과정을 거친다. 1. 사용자 및 사용자 선호 조건 조회(존재 여부 검증) 2. 갱신된 생년월일로 성인 여부 검증 및 새로운 나이 계산 3. 갱신된 음역대, 선호하는 음역대 조회(존재 여부 검증) 4. 갱신된 동물상, 선호하는 동물상 조회(존재 여부 검증) 5. 갱신된 활동 지역, 선호하는 지역 조회(존재 여부 검증) 6. 선호하는 나이 하한 <= 상한 검증 7. 선호하는 키 하한 <= 상한 검증 8. 사용자 프로필, 사용자 선호 조건 업데이트

- 기존에는 정상 처리시 별도의 응답 데이터가 없을 경우 202 코드 사용 - 하지만 202 코드는 작업이 아직 완료되지 않았다는 의미를 내포 - 응답 데이터를 제공하지 않는 경우도 일괄적으로 200으로 처리

- 하한, 상한 값을 null로 요청 가능 - 하한, 상한 값이 존재할 경우 0 이상이어야 함

- 하한, 상한 값이 null로 요청될 수 있음 - 하한, 상한 값이 둘 다 존재할 경우에 하한 > 상한 여부를 검증토록 수정

- 음역대, 동물상, 지역을 조회하지 못하는 경우는 회원 관련 플로우에서 발생 - 회원가입 관련 예외만 모아놓는 JoinException이 아닌 UserException에 소속되도록 위치 변경

날짜를 저장한다는 의미를 잘 나타낼 수 있도록 변경

acceptor-gyu

request에서 필수 입력값이라면 primitive type을 사용하신다고 말씀하시지 않으셨나요? 제가 잘못 이해한건가요? 해당 comment에 대한 답변 남겨주시면 리뷰 진행하겠습니다.

- 음역대, 동물상, 지역 전체 조회후 요청 값으로 필터링하는 구조로 변경 - 업데이트 가능한 필드 추가에 따른 로직 변경

- 선호하는 나이 하한/상한은 20~40 내 범위 값만 가능 - 키, 선호하는 키 하한/상한은 130~220 내 범위 값만 가능

- 기존에는 사용자 나이가 성인(만 18세이상)인 지만 검증 - 서비스 이용 가능 사용자 나이 범위가 정해짐에 따라 로직 재구성

Minjae-An added 19 commits April 15, 2024 21:05

feat: 서비스 정규식 enum 정리(#38)

8aca7d3

병합 과정에서 잘못 설정된 enum 통합

feat: 생년월일 기반 성인 여부 확인 로직 추가(#38)

2eee86c

feat: 프로필 수정 관련 예외 추가(#38)

b772980

- 성인이 아닌 경우 - 하한 값이 상한 값보다 큰 경우

feat: 사용자 기반 사용자 선호 내역 조회 기능 정의(#38)

5d2cb7d

feat: 사용자 기반 사용자 선호 내역 조회 로직 추가(#38)

7abfd40

feat: 내 프로필 수정 요청 DTO 보완(#38)

8cb989a

- 요구사항 변경에 따른 필드 추가 - 필드별 검증 조건 추가 - 명세 수정

feat: 내 프로필 업데이트 로직 추가(#38)

e337251

feat: 선호조건 업데이트 로직 추가(#38)

f8c90b4

feat: 불필요한 constraint exception 핸들링 로직 삭제(#38)

2c85ae7

feat: 하한, 상한 필드들 nullable하게 수정(#38)

849f214

feat: 하한, 상한 값들 제약 조건 수정(#38)

5a14144

- 하한, 상한 값을 null로 요청 가능 - 하한, 상한 값이 존재할 경우 0 이상이어야 함

feat: 내 프로필 수정 비즈니스 로직 수정(#38)

069994a

- 하한, 상한 값이 null로 요청될 수 있음 - 하한, 상한 값이 둘 다 존재할 경우에 하한 > 상한 여부를 검증토록 수정

feat: 예외 enum 위치 적절히 이동(#38)

5a64f6e

- 음역대, 동물상, 지역을 조회하지 못하는 경우는 회원 관련 플로우에서 발생 - 회원가입 관련 예외만 모아놓는 JoinException이 아닌 UserException에 소속되도록 위치 변경

feat: 예외 enum 위치 이동에 따른 사용 위치 수정(#38)

7ee919d

refactor: 현재 날짜 변수명 now에서 currentDate로 변경(#38)

ae1537f

날짜를 저장한다는 의미를 잘 나타낼 수 있도록 변경

feat: 회원 생성 로직에 성인 여부 검증 과정 추가(#38)

6a1c8d3

Minjae-An added the ✨ feat 새로운 기능 추가 label Apr 17, 2024

Minjae-An requested a review from acceptor-gyu April 17, 2024 09:52

Minjae-An self-assigned this Apr 17, 2024

Minjae-An linked an issue Apr 17, 2024 that may be closed by this pull request

[BE] 내 프로필 수정 기능 구현 #38

Closed

acceptor-gyu closed this Apr 26, 2024

Minjae-An added 2 commits April 28, 2024 19:48

Merge branch 'dev' into feat/#38-edit-my-profile

dd2f990

feat: 키 필드, 기본 타입으로 수정(#38)

de9e033

Minjae-An reopened this Apr 28, 2024

acceptor-gyu reviewed Apr 29, 2024

View reviewed changes

feat: 키, 사진 싱크로율 필드 기본 타입으로 수정(#38)

1cbfa27

Minjae-An requested a review from acceptor-gyu April 29, 2024 04:06

Minjae-An added 17 commits April 30, 2024 21:55

feat: 내 프로필 수정 비즈니스 로직 변경(#38)

779a85c

- 음역대, 동물상, 지역 전체 조회후 요청 값으로 필터링하는 구조로 변경 - 업데이트 가능한 필드 추가에 따른 로직 변경

Merge branch 'dev' into feat/#38-edit-my-profile

14b1846

feat: 불필요한 이메일 검증 정규식 삭제(#38)

7c9ea6d

feat: 불필요한 이미 사용 중인 이메일 예외 삭제(#38)

47b2f0e

feat: 사용자 엔티티 이메일 필드 삭제, 연관 로직 수정(#38)

eed0097

feat: 회원가입 요청 dto, 이메일 필드 삭제(#38)

e5b169b

feat: 사용자 상세 프로필 응답 dto, 이메일 필드 삭제(#38)

b687a4a

feat: 이메일 연관 조회 로직 삭제(#38)

b017cac

feat: 이메일 연관 조회 로직 삭제(#38)

65bd1b0

feat: 이메일 연관 로직 삭제(#38)

310c1cd

feat: 사용자 생성 로직에서 이메일 검증 코드 삭제(#38)

b5caf52

feat: 불필요한 이메일 찾기 연관 API 삭제(#38)

ce65021

feat: 삭제된 API, 인가 필요 URL에서 제외(#38)

7b10974

Merge branch 'dev' into feat/#38-edit-my-profile

7930b8c

fix: 병합 과정에서 미처 반영되지 못한 부분 수정(#38)

e919607

feat: 사용자 선호 조건 조회 로직, 단건을 조회하게 수정(#38)

39d66ec

feat: 사용자 선호 조건 조회 로직 수정(#38)

06d778e

Minjae-An reopened this May 1, 2024

Minjae-An closed this May 1, 2024

Minjae-An added 6 commits May 1, 2024 17:37

feat: 사용자 키 130~220cm 범위 값만 가능토록 검증 추가(#38)

f6f54a4

feat: 사용자 키, 나이 관련 검증 추가(#38)

f1cbd93

- 선호하는 나이 하한/상한은 20~40 내 범위 값만 가능 - 키, 선호하는 키 하한/상한은 130~220 내 범위 값만 가능

feat: 유효 범위를 벗어난 사용자 나이 예외 정의(#38)

6d99b57

feat: 사용자 나이 20~40세 범위내인 지 검증하는 로직 구성(#38)

efdff84

- 기존에는 사용자 나이가 성인(만 18세이상)인 지만 검증 - 서비스 이용 가능 사용자 나이 범위가 정해짐에 따라 로직 재구성

feat: 회원가입시 사용자 나이 검증 로직 수정(#38)

1179b45

feat: 프로필 수정시 사용자 나이 검증 로직 수정(#38)

7d4ae23

Minjae-An reopened this May 1, 2024

Minjae-An requested a review from acceptor-gyu May 2, 2024 04:31

acceptor-gyu approved these changes May 6, 2024

View reviewed changes

Merge branch 'dev' into feat/#38-edit-my-profile

4fe6dfe

Minjae-An merged commit 68154a2 into dev May 8, 2024
1 check passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[내 프로필 수정] 내 프로필을 수정할 수 있음(#38) #67

[내 프로필 수정] 내 프로필을 수정할 수 있음(#38) #67

Minjae-An commented Apr 17, 2024 •

edited

Loading

acceptor-gyu left a comment

[내 프로필 수정] 내 프로필을 수정할 수 있음(#38) #67

[내 프로필 수정] 내 프로필을 수정할 수 있음(#38) #67

Conversation

Minjae-An commented Apr 17, 2024 • edited Loading

작업 대상

📄 작업 내용

🙋🏻 주의 사항

📎 관련 이슈

레퍼런스

acceptor-gyu left a comment

Choose a reason for hiding this comment

Minjae-An commented Apr 17, 2024 •

edited

Loading