Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[feat] 점주는 음식 카테고리를 등록할 수 있다 #67

Merged
merged 19 commits into from
Aug 15, 2024
Merged

[feat] 점주는 음식 카테고리를 등록할 수 있다 #67

merged 19 commits into from
Aug 15, 2024

Conversation

Dr-KoKo
Copy link
Member

@Dr-KoKo Dr-KoKo commented Aug 15, 2024 •
edited
Loading

💡 다음 이슈를 해결했어요.

Issue Link - #55

  • 점주는 Menu를 등록할 때 어떤 카테고리에 속한 Menu인지 설정해주어야 합니다. 이번 이슈는 (Menu 등록하기 전에) MenuCategory를 만들기 위한 기능입니다.
  • "등록되어 있는" Store에 생성할 수 있습니다.
  • Store의 점주만 등록할 수 있습니다. 이외의 사용자는 생성할 수 없습니다.
  • 이미 생성되어 있는 이름으로는 만들 수 없습니다.

💡 이슈를 처리하면서 추가된 코드가 있어요.

  • Store.isOwnedBy(UUID ownerId); 해당 Vendor의 Store가 맞는지 확인

✅ 셀프 체크리스트

  • 내 코드를 스스로 검토했습니다.
  • 필요한 테스트를 추가했습니다.
  • 모든 테스트를 통과합니다.
  • 브랜치 전략에 맞는 브랜치에 PR을 올리고 있습니다.
  • 커밋 메세지를 컨벤션에 맞추었습니다.
  • wiki를 수정했습니다.

Dr-KoKo added 16 commits August 15, 2024 13:53
@Dr-KoKo
[fix] 상위 계층 예외의 생성자 변경
8c073f9 
ErrorCode는 클라이언트에게 보낼 응답 코드 및 메시지 포함
예외를 생성할 때 어떤 문맥에서 발생한 예외인지 추가적인 메시지를 넘겨주게 구현
@Dr-KoKo
[fix] 최상위 계층 예외의 접근제어자 변경
c5e2a7d 
다른 페키지에서의 접근을 금지
@Dr-KoKo
[feat] 메뉴 카테고리 등록 서비스
5816c89
@Dr-KoKo
[fix] 공통 예외처리를 위해 접근제어자 수정
a328c0a 
default -> public
@Dr-KoKo
[feat] 메뉴 카테고리 등록 endpoint
bf396d6 
StoreApiController에 구현
@Dr-KoKo
[chore] .gitkeep 삭제
d487eeb
@Dr-KoKo
[fix] HttpStatusException을 GlobalExceptionHandler에서 처리
5c39a24
@Dr-KoKo
[feat] MenuCategory 생성시 권한 체크
8e517e0 
`+` Store.isOwnedBy
`+` UnauthorizedMenuCategoryCreationException
`~` 예외 메시지를 서버 로그에 부합하게 수정
@Dr-KoKo
[test] MenuCategoryRegistrationService
514f2c4 
`+` Fixture 생성
`+` 성공 / 예외 상황에 대한 테스트 코드 작성
@Dr-KoKo
[fix] GlobalExceptionhandler의 오타 수정
2da4b96 
HttpStatusCodeException -> HttpStatusException
@Dr-KoKo
[fix] deprecated된 예외 변경
bb25f89
@Dr-KoKo
[fix] 메뉴 카테고리 수정 요청에 검증 추가
8af2f8d 
1~10자까지 허용
@Dr-KoKo
[test] 메뉴 카테고리 생성 endpoint 테스트
10cbb65 
`+` 성공 200
`+` 실패 400
`+` 실패 401
`+` 실패 403
@Dr-KoKo
[feat] 메뉴 카테고리에 Unique 제약조건 추가
94927d4 
하나의 store에는 하나의 이름만 가능하다.
@Dr-KoKo
[fix] 컨벤션에 맞게 수정
07e9ba7 
error_code -> errorCode
@Dr-KoKo
[test] MenuCategoryRepository
65471b6 
해당 가게에 이미 같은 이름의 카테고리가 있으면 DataIntegrityException 발생 테스트
@Dr-KoKo Dr-KoKo added the ✨ Feature 기능 개발 label Aug 15, 2024
@Dr-KoKo Dr-KoKo added this to the 프로토타입 만들기 milestone Aug 15, 2024
@Dr-KoKo Dr-KoKo self-assigned this Aug 15, 2024
@Dr-KoKo Dr-KoKo requested review from june-777, kimhyun5u and Hyeon-Uk and removed request for kimhyun5u August 15, 2024 08:11
@Dr-KoKo
Merge remote-tracking branch 'origin/main' into feature/55_Dr-KoKo_점주…
220eb87 
…는_음식_카테고리를_등록할_수_있다

# Conflicts:
#	src/main/java/camp/woowak/lab/store/domain/Store.java
@Dr-KoKo Dr-KoKo linked an issue Aug 15, 2024 that may be closed by this pull request
[기능] 점주는 음식 카테고리를 등록할 수 있다 #55
Closed
kimhyun5u
kimhyun5u approved these changes Aug 15, 2024
View reviewed changes
Copy link
Member

@kimhyun5u kimhyun5u left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM !

june-777
june-777 approved these changes Aug 15, 2024
View reviewed changes
Copy link
Member

@june-777 june-777 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM ^_^

@Dr-KoKo Dr-KoKo requested a review from Hyeon-Uk August 15, 2024 09:55
@Dr-KoKo Dr-KoKo merged commit c778333 into main Aug 15, 2024
1 check passed
@kimhyun5u kimhyun5u deleted the feature/55_Dr-KoKo_점주는_음식_카테고리를_등록할_수_있다 branch August 22, 2024 04:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@june-777 june-777 june-777 approved these changes

@kimhyun5u kimhyun5u kimhyun5u approved these changes

@Hyeon-Uk Hyeon-Uk Awaiting requested review from Hyeon-Uk

Assignees

@Dr-KoKo Dr-KoKo

Labels
✨ Feature 기능 개발
Projects
None yet
Milestone
프로토타입 만들기
Development

Successfully merging this pull request may close these issues.

[기능] 점주는 음식 카테고리를 등록할 수 있다
4 participants
@Dr-KoKo @kimhyun5u @Hyeon-Uk @june-777