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/#151] 사용자 정보(이름, 이메일) 조회 API 연결 #155

Merged
merged 5 commits into from
Jan 21, 2025
Merged

[Feat/#151] 사용자 정보(이름, 이메일) 조회 API 연결 #155

merged 5 commits into from
Jan 21, 2025

Conversation

zzz-myam
Copy link
Collaborator

📌 관련 이슈번호

체크리스트

  • 🎋 base 브랜치를 develop 브랜치로 설정했나요?
  • 🖌️ PR 제목은 형식에 맞게 잘 작성했나요?
  • 🏗️ 빌드는 성공했나요? (yarn build)
  • 🧹 불필요한 코드는 제거했나요? e.g. console.log
  • 🙇‍♂️ 리뷰어를 지정했나요?
  • 🏷️ 라벨은 등록했나요?

✅ Key Changes

이번 PR에서 작업한 내용을 간략히 설명해주세요

  1. 사용자 정보(이름, 이메일) 조회 API 연결 성공했습니다.
  • 로그인 시, 마이페이지에서 프로필 카드에 카카오톡에 등록된 이름과 이메일이 보이고,
  • 로그인이 되어 있지 않을 경우, 마이페이지에서 '로그인 하러 가기' 라는 버튼이 보입니다.

@zzz-myam zzz-myam self-assigned this Jan 21, 2025
youtheyeon
youtheyeon approved these changes Jan 21, 2025
View reviewed changes
Copy link
Collaborator

@youtheyeon youtheyeon left a comment

Choose a reason for hiding this comment

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

깔끔한 연결이네요 수고하셨습니다! ✨

thisishwarang
thisishwarang approved these changes Jan 21, 2025
View reviewed changes
Copy link
Collaborator

@thisishwarang thisishwarang left a comment

Choose a reason for hiding this comment

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

고생 많으셨습니다!!! 따로 수정할게 없어보여요~

chaeneey
chaeneey approved these changes Jan 21, 2025
View reviewed changes
Copy link
Collaborator

@chaeneey chaeneey left a comment

Choose a reason for hiding this comment

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

고생 많으셨습니다 !!! 👍🏻🙌🏻

src/apis/myPage/useFetchUser.ts
Copy link
Collaborator

Choose a reason for hiding this comment

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

해당 api는 메인 홈 화면에서 사용자 이름 불러올 때도 사용할 수 있을 것 같아서!
메인 페이지에서 호출하는 부분도 추가해주면 더 더 좋을 것 같단 생각이 들었습니닷 ! 😍

@thisishwarang thisishwarang merged commit 811a697 into develop Jan 21, 2025
1 check passed
@thisishwarang thisishwarang deleted the feat/#151/user-name-api branch January 21, 2025 15:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@youtheyeon youtheyeon youtheyeon approved these changes

@chaeneey chaeneey chaeneey approved these changes

@thisishwarang thisishwarang thisishwarang approved these changes

Assignees

@zzz-myam zzz-myam

Labels
✨ feature 기능 개발 😽 지유
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Feat] 사용자 이름 조회 API 연결
4 participants
@zzz-myam @thisishwarang @chaeneey @youtheyeon