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: AuthProvider 구현 #23

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

feat: AuthProvider 구현 #23

wants to merge 4 commits into from

Conversation

bytrustu
Copy link
Member

@bytrustu bytrustu commented Jul 8, 2023

작업 주제

  • AuthProvider 구현

작업 내용(어떤 부분을 리뷰어가 집중해서 봐야할까요?)

  • 카카오 로그인 할 경우, 유저 닉네임 정보가 없으면 닉네임 설정 페이지로 전환하도록 개선했습니다.
  • 유저 정보를 알 수 있도록 AuthProvider 구현했습니다.

체크리스트(PR 올리기 전 아래의 내용을 확인해주세요.)

  • base, compare branch가 적절하게 선택되었나요?
  • 로컬 환경에서 충분히 테스트 하셨나요?

리뷰 규칙

  • P1: 꼭/적극적 반영해 주세요 (Request changes)
  • P2: 웬만하면 반영해 주세요 (Comment)
  • P3: 반영해도 좋고 넘어가도 좋습니다 (Approve)

@vercel
Copy link

vercel bot commented Jul 8, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
oseek ❌ Failed (Inspect) Jul 8, 2023 10:12pm

setLocalStorageItem(USER_KEY, {
nickname: responseMember.nickname,
location: responseMember.location,
});
await memberFoodKeywordApi.saveMemberFoodKeywordAxios(foodKeywords);
Copy link
Contributor

Choose a reason for hiding this comment

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

혹시 tanstack query로 안한 이유가 있을까요?

Copy link
Member Author

Choose a reason for hiding this comment

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

어떤 부분은 react query로 변경해보면 좋을까요?
제가 아직 익숙하지 않아서 그런것 같네요

Copy link
Contributor

Choose a reason for hiding this comment

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

useMutation을 사용하시면 create/update/delete 메서드 들에 대해서 조금 더 쉽게 처리가 가능할 거 같아요 :) 아무래도 저희가 이번에는 tanstack query를 최대한 사용하는게 목표라서 한번 사용해보시는것도 좋은 방법일 거 같습니다 :)

Copy link
Contributor

Choose a reason for hiding this comment

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

이를 사용해서 onSuccess 같은 곳에서 query key invalidation을 통해 stale한 데이터에 대해서도 invalidate를 하기도 수월할 거 같습니다 :)

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

Successfully merging this pull request may close these issues.

2 participants