Skip to content

Conversation

@jyjyjy25
Copy link
Member

@jyjyjy25 jyjyjy25 commented Mar 6, 2024

🔥 Related Issue

🏃‍ Task

  • 유저 Role 조회 API 구현📍 관련커밋: {575497b}

📄 Reference

  • None

✅ Check List

  • PR의 제목은 팀 내 규칙을 준수하여 알맞게 작성하였는가?
  • Merge 하는 브랜치가 올바른가? (main branch에 실수로 PR 생성 금지)
  • 팀의 코딩 컨벤션을 준수하는가?
  • PR과 관련없는 변경사항이 들어가지는 않았는가?
  • 내 코드에 대한 자기 검토가 되었는가?
  • Reviewers, Assignees, Lables, Project, Milestone은 적절하게 선택하였는가?
  • 관련한 issue를 닫아야 하는지 점검해보고 적용했는가?

@jyjyjy25 jyjyjy25 added the 💫 feature 기능 구현 label Mar 6, 2024
@jyjyjy25 jyjyjy25 requested review from dionisos198 and songhee1 March 6, 2024 05:28
@jyjyjy25 jyjyjy25 self-assigned this Mar 6, 2024
Copy link
Contributor

@dionisos198 dionisos198 left a comment

Choose a reason for hiding this comment

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

LGTM 입니다.!!
근데 생각해보니까
SuccessHandler 에서

sb.append("http://52.79.200.90:3000/login/oauth2/code")
                .append("?")
                .append("accessToken=")
                .append(accessToken)
                .append("&")
                .append("refreshToken=")
                .append(refreshToken)
                .append("&")
                .append("userId=")
                .append(userId);

이 부분에 Role을 적어서 전달할 수도 있을 것같은데 이에 대해서는 어떻게 생각하시나염? ^^

Copy link
Contributor

@songhee1 songhee1 left a comment

Choose a reason for hiding this comment

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

LGTM👍

@jyjyjy25
Copy link
Member Author

@dionisos198 좋은 방법인 것 같습니다! 그렇게 수정해 볼게요

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

💫 feature 기능 구현

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[API V1] 유저 ROLE 반환 API 구현

3 participants