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

Refactor: 예약 목록 조회시 관리자 조회 차별화 #298

Merged
merged 2 commits into from
Nov 7, 2024
Merged

Refactor: 예약 목록 조회시 관리자 조회 차별화 #298

merged 2 commits into from
Nov 7, 2024

Conversation

gitseoyeon
Copy link
Member

@gitseoyeon gitseoyeon commented Nov 6, 2024 •
edited
Loading

Summary

Key Changes

  • getGroupReservation 에 isAdmin이라는 관리자 판별 파라미터 값을 주어 관리자일 경우와 일반 사용자일 경우를 구분짓게 했습니다.
  • isAdmin 값이 true일 경우 모든 detail 정보가 반환되며 false일 경우 applyUser 값만 null로 들어가 보이지 않게 처리했습니다.

Testing

  • 일반 사용자 목록 조회 API 호출 시
image
  • 관리자 목록 조회 API 호출 시
image

To Reviewers

질문이나 의견 있으시면 코멘트 남겨주세요!

@gitseoyeon gitseoyeon added the refactor 코드를 개선합니다. label Nov 6, 2024
@gitseoyeon gitseoyeon requested a review from muncool39 November 6, 2024 16:18
@gitseoyeon gitseoyeon self-assigned this Nov 6, 2024
muncool39
muncool39 approved these changes Nov 7, 2024
View reviewed changes
Copy link
Member

@muncool39 muncool39 left a comment

Choose a reason for hiding this comment

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

확인했습니다! 수고하셨습니다! 👍

@gitseoyeon gitseoyeon merged commit 7c7dd26 into dev Nov 7, 2024
3 checks passed
@gitseoyeon gitseoyeon deleted the refactor/#297 branch November 7, 2024 07:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@muncool39 muncool39 muncool39 approved these changes

Assignees

@gitseoyeon gitseoyeon

Labels
refactor 코드를 개선합니다.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[REFACTOR] 관리자와 일반 사용자 시점 예약 목록 조회 데이터 차별화
2 participants
@gitseoyeon @muncool39