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

fix : 호스트 이벤트 접근 권한 설정 #459

Merged
merged 3 commits into from
Feb 26, 2023
Merged

fix : 호스트 이벤트 접근 권한 설정 #459

merged 3 commits into from
Feb 26, 2023

Conversation

gengminy
Copy link
Member

@gengminy gengminy commented Feb 26, 2023 •
edited
Loading

개요

작업사항

  • 호스트가 관리중인 이벤트 목록 조회 시 초대 수락하지 않은 유저도 접근 가능했던 것 수정
  • 호스트 초대 거절 추가

변경로직

  • 호스트가 관리 중인 이벤트 목록 권한 설정
  • 자신이 관리중인 호스트 이벤트 목록에서 초대 받지 않은 호스트의 이벤트는 쿼리에 포함시키지 않도록
  • 호스트 초대 거절 로직
  • 호스트 <-> 호스트 유저 엔티티 변경

@gengminy gengminy added For: API [이슈 대상] 외부 API Type: Errors/Bugs [이슈 목적] 버그, 애러 수정 labels Feb 26, 2023
@gengminy gengminy added this to the 두둥 스프린트 2차 milestone Feb 26, 2023
@gengminy gengminy self-assigned this Feb 26, 2023
sanbonai06
sanbonai06 approved these changes Feb 26, 2023
View reviewed changes
Copy link
Member

@sanbonai06 sanbonai06 left a comment

Choose a reason for hiding this comment

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

빠른 피드백 수용 멋져용

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 447 Code Smells

21.5% 21.5% Coverage
0.7% 0.7% Duplication

ImNM
ImNM approved these changes Feb 26, 2023
View reviewed changes
Copy link
Member

@ImNM ImNM left a comment

Choose a reason for hiding this comment

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

LGTM

@gengminy gengminy merged commit 5beabef into dev Feb 26, 2023
@gengminy gengminy deleted the fix/458-auth branch February 26, 2023 12:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sanbonai06 sanbonai06 sanbonai06 approved these changes

@ImNM ImNM ImNM approved these changes

@cofls6581 cofls6581 Awaiting requested review from cofls6581 cofls6581 is a code owner

@kim-wonjin kim-wonjin Awaiting requested review from kim-wonjin kim-wonjin is a code owner

Assignees

@gengminy gengminy

Labels
For: API [이슈 대상] 외부 API Type: Errors/Bugs [이슈 목적] 버그, 애러 수정
Projects
두둥
Status: Done
Milestone
두둥 스프린트 2차
Development

Successfully merging this pull request may close these issues.

🐛[BUG] 호스트 관리 권한 오류 대응
3 participants
@gengminy @ImNM @sanbonai06