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

[FEAT] User API 수정 #55

Merged
merged 1 commit into from
Aug 28, 2024
Merged

[FEAT] User API 수정 #55

merged 1 commit into from
Aug 28, 2024

Conversation

EeeasyCode
Copy link
Member

Issue Number

feature

작업 내용

  • get user API 수정
    • 실제 user server와 연동 후 session에 저장된 userId를 통해 user info를 find 하도록 수정함
  • authInfo 데코레이터 사용
    • 기존에 session.user로 가져오던 정보를 authInfo 데코레이터로 변경함

Reference

Check List

@EeeasyCode EeeasyCode self-assigned this Aug 28, 2024
leobang17
leobang17 approved these changes Aug 28, 2024
View reviewed changes
Copy link
Contributor

@leobang17 leobang17 left a comment

Choose a reason for hiding this comment

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

good 근데 내꺼 먼저 merge 해야함

EeeasyCode
EeeasyCode commented Aug 28, 2024
View reviewed changes
Copy link
Member Author

@EeeasyCode EeeasyCode left a comment

Choose a reason for hiding this comment

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

이제 merge 하겠습니다잇!

@EeeasyCode EeeasyCode merged commit ea43fe3 into development Aug 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@leobang17 leobang17 leobang17 approved these changes

Assignees

@EeeasyCode EeeasyCode

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@EeeasyCode @leobang17