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(user-&-room): seperate api for auth and room #301

Open
wants to merge 4 commits into
base: dev
Choose a base branch
from

Conversation

vimkim
Copy link
Collaborator

@vimkim vimkim commented Apr 15, 2024

image

image

image

모듈 추가 (app.module.ts 변경)

RoomsModule이 추가되었습니다. 이는 기존 RoomModule의 기능의 호환을 깨지 않으면서 점차 더 나은 rest api를 설계하기 위함입니다.

차이점: 기존 api: GET /room/:code/users -> GET /rooms/:code/users
s가 추가되었습니다.

인증 상태 확인 기능 (auth.controller.ts 변경)

/status 엔드포인트를 통해 사용자의 로그인 상태를 확인할 수 있는 기능이 추가되었습니다. 이는 요청 객체에서 인증 여부를 확인하여 반환합니다.

엔티티 관계 변경 (room-user.entity.ts 변경)

RoomUser 엔티티에서 Room과 User 엔티티와의 관계에서 eager 로딩 옵션이 제거되었습니다. 이는 관련 데이터를 자동으로 미리 불러오지 않도록 설정한 것입니다.

사용자 컨트롤러 추가 기능 (user.controller.ts 변경)

사용자 정보 조회(users/me) 및 사용자가 참여한 방의 코드를 조회(users/me/room-code)하는 두 개의 새로운 엔드포인트가 추가되었습니다.

사용자 서비스 로직 추가 (user.service.ts 변경)

사용자가 참여한 방 정보를 조회하는 getRoomIfJoined 메소드가 추가되었습니다. 이 메소드는 사용자 ID를 기반으로 참여한 방을 찾고, 참여한 방이 없으면 예외를 발생시킵니다.

@vimkim vimkim linked an issue Apr 15, 2024 that may be closed by this pull request
Copy link
Collaborator

@glowisn glowisn left a comment

Choose a reason for hiding this comment

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

요 PR 머지할까요?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

API 분리
2 participants