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 #16 - 유치원 아이디 추출 및 검증을 위한 미들웨어와 퍼미션 구현 #17

Merged
merged 6 commits into from
Jun 25, 2024
Merged

Feat #16 - 유치원 아이디 추출 및 검증을 위한 미들웨어와 퍼미션 구현 #17

merged 6 commits into from
Jun 25, 2024

Conversation

DongwookKim0823
Copy link
Member

PR 체크리스트

아래 항목을 확인해 주세요:

  • 커밋 메시지가 우리의 가이드라인을 따르고 있는지 확인하세요
  • 변경 사항에 대한 테스트가 추가되었는지 확인하세요 (버그 수정 / 기능 추가)
  • 문서가 추가되거나 업데이트되었는지 확인하세요 (버그 수정 / 기능 추가)

PR 유형

이 PR은 어떤 종류의 변경을 가져오나요?

  • 버그 수정
  • 새로운 기능 추가
  • 기능 변경/삭제
  • 코드 스타일 업데이트 (서식, 로컬 변수)
  • 리팩터링 (기능 변경 없음, API 변경 없음)
  • 빌드 관련 변경
  • CI 관련 변경
  • 문서 내용 변경
  • 애플리케이션 / 인프라 변경
  • 기타... 설명:

관련 이슈

이슈 번호: #16

어떤 동작은 무엇인가요?

  • 반려동물 유치원 아이디를 검출하는 미들웨어 구현
  • 접근 가능한 유치원인지 검증하는 퍼미션 구현

기타 정보

@DongwookKim0823
Feat : CustomJWTMiddleware 구현
551ec06 
- JWT 토큰에서 pet_kindergarden_id를 추출하는 미들웨어 구현
@DongwookKim0823
Feat : 반려동물 유치원 접근 권한 검증 로직 추가
944daec 
- PetKindergardenAccessPermission 생성
- 유저가 반려동물 유치원에 속해 있는지 검증
@DongwookKim0823
Style : 오타 수정
c002802 
- CustomJWTMiddleware 미들웨어 추가 구문 오타 수정
@DongwookKim0823
Refact : 의존성 주입 방식 수정
8910ed7
@DongwookKim0823
Feat : ExtendedAPIAuthMixin 클래스 생성
839d09f 
- 반려동물 유치원 접근 권한을 확인하기 위한 믹스인 추가
@DongwookKim0823 DongwookKim0823 added the ✨ Feature 기능 개발(API, 모듈 등) label Jun 25, 2024
@DongwookKim0823 DongwookKim0823 self-assigned this Jun 25, 2024
JaeHyuckSa
JaeHyuckSa reviewed Jun 25, 2024
View reviewed changes
Copy link
Member

@JaeHyuckSa JaeHyuckSa left a comment

Choose a reason for hiding this comment

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

고생하셨습니다.

@DongwookKim0823
Style : 클래스명 및 독스트링 관련 수정
50061a7 
- CustomJWTMiddleware 이름 명확하도록 수정
- ExtendedAPIAuthMixin 이름 명확하도록 수정
- PetKindergardenAccessPermission 이름 일관성 있게 수정
JaeHyuckSa
JaeHyuckSa approved these changes Jun 25, 2024
View reviewed changes
Copy link
Member

@JaeHyuckSa JaeHyuckSa left a comment

Choose a reason for hiding this comment

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

고생하셨습니다

@DongwookKim0823 DongwookKim0823 merged commit 8f2106f into develop Jun 25, 2024
1 check passed
@DongwookKim0823 DongwookKim0823 deleted the feature/issue-016/extract-and-validate-pet-kindergarden-id branch June 25, 2024 15:18
DongwookKim0823 added a commit that referenced this pull request Jun 26, 2024
@DongwookKim0823
Feat #16 - 유치원 아이디 추출 및 검증을 위한 미들웨어와 퍼미션 구현 #17 (#19)
d50d4a4 
- 로그인하는 유저가 게스트 그룹에 속하지 않는 파트너라면 게스트 그룹에 추가하는 로직 추가
- 게스트 퍼미션 검사 로직 수정

Close #18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JaeHyuckSa JaeHyuckSa JaeHyuckSa approved these changes

Assignees

@DongwookKim0823 DongwookKim0823

Labels
✨ Feature 기능 개발(API, 모듈 등)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@DongwookKim0823 @JaeHyuckSa