[FEAT] 다짐메시지 생성 예외처리 추가 #346
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
- 아직 프로필을 등록하지 않은 유저인 경우 - 최신 기수(34)기 회원이 아닌 경우 Related to: #345
jun02160
added
🐞 Bug
|
Hi there 👋 Using this App for a private organization repository requires a paid subscription. You can click If you are a non-profit organization or otherwise can not pay for such a plan, contact me by creating an issue |
|
dev-Crayon
reviewed
Mar 19, 2024
Comment on lines
+27
to
+32
| if (member.getGeneration() == null) { | ||
| throw new ClientBadRequestException("Not exists profile info"); | ||
| } | ||
| if (!member.getGeneration().equals(34)) { // 기수 갱신 시 조건 변경 | ||
| throw new ClientBadRequestException("Only new generation can enroll resolution"); | ||
| } |
There was a problem hiding this comment.
필요한 에러처리인 것 같아서 잘 구현해주신 것 같습니다!
다만 추후 관리를 위해 34라는 값보다는 현재 기수를 관리할 수 있으면 좋을 것 같네요
이 부분 같이 논의해보면 좋을 것 같습니다.
우선은 머지해도 될 것 같아요!
jun02160
added a commit
that referenced
this pull request
Mar 22, 2024
* [CHORE] 프로젝트 전체조회 API Response 추가(member, 창업중, 진행중) (#318) * refactor: booleanexpression 메소드 형태로 변경 * fix: count method by filter * chore: refactor method * chore: response type * add: 창업중, 진행중 response 추가 * chore: member response 추가 * chore: 변수명 수정 (#319) * chore: 변수명 설정 잘못되어있는 것 수정 (#320) * Feature/#313 profile masking (#317) * [FEAT] 프로필 마스킹 작업 멤버 프로필 마스킹 여부를 확인하기위한 필드 추가 isEmailBlind isPhoneBlind Related to: #313 * [FEAT] 프로필 마스킹 작업 멤버 아이디로 프로필 조회 response에 각 필드 추가 isPhoneBlind isEmailBlind Related to: #313 * [FEAT] 프로필 마스킹 작업 멤버 프로필 마스킹 여부 변경 API 구현 Related to: #313 * feat: 멤버가 속한 모임 전체 조회(페이지네이션) (#323) 관련 request dto 정의 관련 response dto 정의 openfeign으로 crew에 api 요청 인터페이스 구현 Related to: #321 * [FEAT] 멤버 프로필 조회 블라인드 처리 반영 response 추가 (#324) 멤버 프로필 조회 API email, phone 블라인드 여부에 따른 응답값 null 처리 Related to: #324 * [FEAT] 전체 멤버 프로필 리스트 조회 블라인드 처리 반영 response 추가 (#324) 전체 멤버 프로필 조회 API email, phone 블라인드 여부에 따른 응답값 null 처리 Related to: #324 * [FIX] 정적 팩토리 메서드를 이용한 응답 필드 반환으로 수정 (#324) 멤버 개별 프로필 조회 API에서 Null 처리가 제대로 반영되지 않는 부분을 Response의 static 메서드로 감싸도록 해결 Related to: #324 * [REFACTOR] Mapper Interface default 메서드로 분리 (#324) isBlind 필드 값에 따라 null 값으로 마스킹해주는 로직을 Mapper의 default 메서드로 분리하여 컨트롤러에서 세부 로직을 감춤 Related to: #324 * [Fix]: 멤버 크루 조회 API id 파라미터 추가 (#330) access token 헤더에서 안받고 유저 id 파라미터로 받기 Related to: #328 * [FIX] Member 프로필 수정 API (#329) * [FIX] Member 프로필 수정 API 수정 (#327) Member의 editActivitiesAble 필드로 활동 정보는 한번만 수정이 가능하도록 허용하는 부분의 분기처리 오류가 있어 예외처리 조건을 추가 Related to: #324 * [FIX] Member 프로필 수정 API 버그 해결 (#327) Member의 editActivitiesAble 필드로 활동 정보는 한번만 수정이 가능하도록 허용하는 부분의 분기처리 오류가 있어 예외처리 조건을 추가 Related to: #324 * [REFACTOR] Activity Request 비교 메서드로 추출 (#327) DTO로 리스트를 파라미터로 받는 비교 메서드 추가 -> 서비스 단에서 메서드명으로 역할을 명확히 하기 위함 Related to: #327 * 4기 메이커스 멤버 White list (#334) Related to #332 * [Fix]: 멤버 크루 조회 에러처리 (#335) GlobalExceptionHandler에 FeignClient Exception 추가 Related to: #331 * [Feat]: 유저 다짐 메시지 저장 Entity Class 구현 (#338) UserResolution.java 파일 생성 Related to: #337 * [Feat] 다짐 메시지 생성 API 개발 (#340) * [Fix]: editActivitiesAble 기본값 변경 (#342) true -> false Related to: #341 * [Feat]: 유저 다짐 메세지 조회 API 구현 (#344) * [Feat]: 유저 다짐 메세지 조회 API 구현 Mapper Class 생성 관련 로직 작성 Related to: #343 * [Fix]: 테스트용 멤버 id 삭제 Related to: #343 * [Update]: 다짐 메세지 반환 데이터 변경 태그 id값이 아닌 태그명 반환 Related to: #343 * [FEAT] 다짐메시지 생성 예외처리 추가 (#345) (#346) - 아직 프로필을 등록하지 않은 유저인 경우 - 최신 기수(34)기 회원이 아닌 경우 Related to: #345 * Hotfix/#348 cors (#349) * [Hotfix]: CORS 에러 처리 * [Hotfix]: CORS 에러 처리 * [Hotfix]: CORS 에러 처리 복구 * Update application.yml * [Feat]: 유저 프로필 기수 확인 여부 API (#354) Related to: #353 * [Chore] 다짐 메시지 변경사항 적용 (#350) * [CHORE] 다짐메시지 태그 수정 (#347) - 와이어프레임 상에 변경된 태그명 적용 Related to: #347 * [CHORE] 유저 당 1회 등록 제한 (#347) - 다짐메시지 등록 API 호출 1회로 제한하도록 설정 Related to: #347 * [FIX] 잘못된 필터링 수정(#347) - 다짐메시지 등록 API 호출 1회로 제한하도록 설정 Related to: #347 * [FIX] 다짐메시지 등록 기수 당 1명 제한(#347) - 한 기수 당 하나의 등록만 가능하도록 수정 Related to: #347 * [FEAT] 다짐메시지 유효성 검사 API 추가 (#347) - 프로필 이미지 정보(다짐 메시지 등록 시) - 등록 여부의 validation 체크 위 2가지 목적을 가진 API 추가 Related to: #347 * [CHORE] 리뷰 반영 (#347) - countBy -> existsBy JPA 메서드 변경 - 공백 정리 Related to: #347 * [FIX] 마스킹 버그 해결 (#355) - email, phone이 함께 블라인드 처리되어 있던 부분 수정 Related to: #325 * [FIX] 마스킹 버그 해결 (#325) - email, phone이 함께 블라인드 처리되어 있던 부분 수정 Related to: #325 * [Chore]: 마스킹 API 통합 (#357) Related to: #356 --------- Co-authored-by: 한유진 <[email protected]> Co-authored-by: jun02160 <[email protected]> Co-authored-by: Yejun Park <[email protected]>
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.
와이어프레임 상에서 반영되지 않은 다짐메시지 생성 시의 예외를 처리했습니다
closed Issue #345