[Feature/48] 카테고리 validation 검증 로직 및 비기능 개발 #52
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
<body> - CategoryService.java - 유저의 카테고리인지 확인하는 로직 추가 <footer> - 관련: #48
This was
linked to
issues
Mar 14, 2024
<body> <footer> - 관련: #48
luke0408
reviewed
Mar 14, 2024
|
|
||
| public boolean isNotCreatedByUser(Long userId) { | ||
| return this.user.getId() == userId; | ||
| } |
There was a problem hiding this comment.
❗ method 명은 isNotCreatedByUser 인데 메서드의 로직은 해당 유저에 의해 생성된 것을 확인하는 것으로 보입니다. 때문에 isCreatedByUser로 메서드 명을 변경해야하는 것 아닌가 라는 생각이 드는데 어떤가요?
luke0408
changed the title
added 3 commits
March 28, 2024 10:56
<body> - Category.java, CategoryKind.java - 정책 변경에 따라 CategoryKind.java 생성 - Category 에 기본 일정들을 구분 가능한 Kind 생성 - CategoryRequest.java, CategoryName.java, CategoryValidator.java - 기존 검증 로직 변경 및 삭제 - 정책 변경에 따른 이름을 통한 구분 필요 없어짐 CategoryService.java - 정책에 맞게 검증 수준을 변경 <footer> - 관련: #48
<body> - Category.java, CategoryConverter.java - kind 를 Category에 넣게끔 추가 - SocialUtils.java - 네이버 로그인 시에는 nickname이 null 값으로 받아집니다. - 공통 로직 toUser 를 냅두는 방향으로 변경했습니다. - 좀 더 좋은 구조를 고민해 보면 좋을 것 같아요. - CategoryKind.java, UserFacade.java - Enum 타입의 활용을 위한 변수명 추가 - Enum 타입을 활용하기 위한 Facade 변경 <footer> - 관련: #48
<body> - querydsl 을 이용해서 kind 를 기반으로 스케줄을 입력 받게 변경 <footer> - 관련: #48
luke0408
reviewed
Mar 28, 2024
| @@ -46,13 +46,17 @@ public class Category extends BaseTimeEntity { | |||
| @Enumerated(EnumType.STRING) | |||
| private CategoryStatus status; | |||
|
|
|||
| @Enumerated(EnumType.STRING) | |||
| private CategoryKind kind; | |||
There was a problem hiding this comment.
만약 안되어있다면 merge 하기 전에 스키마 수정 부탁드릴게요!
There was a problem hiding this comment.
혹시 스키마에 적용해달라고 하시는게 그 저희 운영 서버에 적용해야하는 건가요?
| @@ -226,16 +227,18 @@ private User saveOrNot(User user) { | |||
|
|
|||
| private void makeBaseCategory(User save) { | |||
| Category baseCategory = CategoryConverter.toCategory( | |||
| "일정", | |||
| CategoryKind.SCHEDULE.getCategoryName(), | |||
luke0408
approved these changes
Mar 30, 2024
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.
Type of change
PR Desciption
유저 카테고리 수정
CategoryService.java
소셜로그인 관련 부분에 기본 카테고리 들어가게 수정
Requirements for Reviewer
PR Log
새롭게 배운 것
고민 중인 사항
첨부 자료
관련 이슈