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.

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

feat: 권한 인증/인가, 초대 기능 구현 #37

Merged
merged 24 commits into from
May 25, 2024
Merged

feat: 권한 인증/인가, 초대 기능 구현 #37

merged 24 commits into from
May 25, 2024

Conversation

w8385
Copy link
Member

@w8385 w8385 commented May 25, 2024

업데이트 유형

  • Hot Fix
  • Release
  • Develop
  • Others

업데이트 개요

업데이트 요약

서비스적 변경

  • owner, editor, viewer 세 단계로 권한 구분
    • Group.auth 및 User.auth에 각각 배열로 존재함
  • Swagger /api 페이지 내 모든 엔드포인트에 JWT 검증 추가
    • 조회: viewer 이상
    • 생성/수정: editor 이상
    • 모임 삭제: owner만
  • 모임 내 등록된 권한에 한해서만 조회/수정 가능
    • 권한 수정은 아직 없음
  • 다른 사용자 userId로 모임에 초대 가능

개발적 변경

  • 파라미터 ObjectId Pipe 구축
    • 잘못된값 들어왔을 때 예외처리
  • 서비스 로직 내 Group 검증 및 예외처리
  • ESLint Prettier max-width 120으로 늘림

해결한 문제

미해결 문제

테스트

  • 유닛 테스트
  • 빌드 테스트 (통합 테스트)
  • 기타 유효성 테스트

수정 사항 진단

  • 코드가 이 프로젝트의 스타일 지침을 따릅니다.
  • 코드에 대한 자체 리뷰를 수행했습니다.
  • 변경 내역에 대해 주석을 작성했습니다.
  • 해당 PR에 대한 문서를 변경했습니다.
  • 기능, 정책, 수정 사항, 이슈에 대한 테스트 코드를 추가했습니다.
  • 변경 내용이 로컬 개발환경에서의 테스트를 통과했습니다.
  • 모든 종속 변경 사항이 병합되어 다운스트림 모듈에 게시되었습니다.

@w8385 w8385 requested a review from ssoxong May 25, 2024 01:59
@w8385 w8385 added the enhancement New feature or request label May 25, 2024
src/group/group.service.ts Fixed Show fixed Hide fixed
Copy link
Contributor

@ssoxong ssoxong left a comment

Choose a reason for hiding this comment

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

짱인데요

@w8385
Copy link
Member Author

w8385 commented May 25, 2024

로컬에서 테스트 해보고 잘 되면 붙이는걸로

@w8385 w8385 merged commit 1460df8 into dev May 25, 2024
5 checks passed
@w8385 w8385 deleted the feat/auth branch May 25, 2024 05:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

사용자 권한 관리
2 participants