전체적으로 생년월일 optional 하게 처리하도록 수정 #284

hyejungg · 2022-12-25T15:30:29Z

✒️ 관련 이슈번호

Onboarding 엔티티 birthday 데이터타입 LocalDate -> String
- 생년월일을 생략하는 경우 ""로 클라에서 보내신다고 하셔서 String으로 수정했습니다. 괜찮나요?! (기존에 nullable = false 였길래요!)
- 기존 형태(LocalDate) 유지하고 nuallble = true로 수정하려고 했는데
  - 안드한테 물어보니까 어차피 String 형태로 주고 있고, 기존에도 LocalDate -> String 형태로 박는거였어서 String 으로 변경함!
회원가입 시 birthday optional 반영 (birthday="", isPublic=false)
- birthday="", isPublic=true이면 400 에러 던집니다.
프로필 정보 수정 시 birthday optional 반영 (birthday="", isPublic=false)
나/룸메이트 조회 시 생년월일이 없는 경우 빈 스트링 보내도록 response 수정
호미의 탄생 배지 스케줄러 돌릴 때 엔티티 데이터 타입 변경으로 DateUtils 에 오늘날짜.toString() 하는 함수 만들어서 수정

나/룸메이트 조회 시 age, birthday 부분이 없는 경우
- "" 또는 null로 전달해주는게 의진오빠가 더 편하다고 해서 호옥시 안드는 null 처리를 안했을까봐 ""로 수정했습니다!

orijoon98

고생하셨어요!! 네이밍 하나만 수정하구 머지해주실 수 있을까요?!

orijoon98 · 2022-12-26T01:05:22Z

src/main/java/hous/server/domain/user/repository/UserRepositoryCustom.java


-    List<User> findAllUserByBirthday(LocalDate birthday);
+    List<User> findAllUserByBirthday(String birthday);


이거 네이밍 findAllUsersByBirthday 가능할까요?!

hyejungg added 4 commits December 26, 2022 00:10

#283 [fix] Onboarding 엔티티에서 birthday 데이터 타입 String으로 변경

696125f

#283 [fix] 회원가입 시 생년월일 Optional 하게 처리하도록 수정

f261155

#283 [fix] 프로필 정보 수정 시 생년월일 Optional 하게 변경 가능하도록 수정

ed36b18

#283 [fix] 나/룸메이트 정보 조회 시 생년월일이 없는 경우 age와 생년월일 모두 빈스트링("")으로 전달하도록 수정

ec3f67c

hyejungg added 🐣 혜정 🐣 혜정이가 했다아~ 🔧 Fix 간단한 수정 labels Dec 25, 2022

hyejungg added this to the Hous 1차 릴리즈 milestone Dec 25, 2022

hyejungg self-assigned this Dec 25, 2022

orijoon98 self-requested a review December 26, 2022 00:39

orijoon98 approved these changes Dec 26, 2022

View reviewed changes

#283 [fix] 호미의 탄생 배지 스케줄러 동작 시 생년월일 매개변수 LocalDate -> String으로 수정

2740050

hyejungg force-pushed the fix/#283 branch from fb1f56f to 2740050 Compare December 26, 2022 01:25

hyejungg merged commit affae9a into develop Dec 26, 2022

hyejungg deleted the fix/#283 branch December 26, 2022 01:27


		List<User> findAllUserByBirthday(LocalDate birthday);
		List<User> findAllUserByBirthday(String birthday);