Skip to content

Comments

fix: 사용자 관련 response수정#20

Merged
guite95 merged 3 commits intodevfrom
fix/user-response
Feb 9, 2026
Merged

fix: 사용자 관련 response수정#20
guite95 merged 3 commits intodevfrom
fix/user-response

Conversation

@guite95
Copy link
Contributor

@guite95 guite95 commented Feb 9, 2026

📋 상세 설명

  • LoginResponse가 role을 반환하도록 수정
  • UserInfoRseponse가 userId, role을 반환하도록 수정

📊 체크리스트

  • PR 제목이 형식에 맞나요 e.g. feat: PR을 등록한다
  • 코드가 테스트 되었나요
  • 문서는 업데이트 되었나요
  • 불필요한 코드를 제거했나요
  • 이슈와 라벨이 등록되었나요

📆 마감일

Close #{issue number}

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

사용자 관련 API 응답 DTO가 role(및 일부는 userId)를 포함하도록 확장하여, 로그인/유저정보 조회 응답에서 권한 정보를 함께 내려주려는 PR입니다.

Changes:

  • LoginResponserole 필드 추가
  • UserInfoResponseuserId, role 필드 추가

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

File Description
src/main/java/until/the/eternity/das/auth/dto/response/LoginResponse.java 로그인 응답에 role을 포함하도록 DTO 확장
src/main/java/until/the/eternity/das/user/dto/response/UserInfoResponse.java 유저 정보 응답에 userId/role을 포함하도록 DTO 확장

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Contributor

@dev-ant dev-ant left a comment

Choose a reason for hiding this comment

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

고생하셨습니다. Approve!

@guite95 guite95 merged commit 87b22e2 into dev Feb 9, 2026
8 checks passed
@guite95 guite95 deleted the fix/user-response branch February 9, 2026 09:35
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.

2 participants