Skip to content

Conversation

@summerDev96
Copy link
Collaborator

@summerDev96 summerDev96 commented Jul 30, 2025

📦 Pull Request

📝 요약(Summary)

  • 카카오 로그인 api 추가하여 토큰 저장되게 수정하였습니다.
  • 카카오 로그인 요청 후 전역 유저 데이터 저장 추가하였습니다.
  • 모바일 사이즈에서 라벨과 에러메시지가 겹치는 현상을 수정하였습니다. (gap 수정)

💬 공유사항 to 리뷰어

  const params =
    typeof code === 'string'
      ? {
          state: '',
          redirectUri: process.env.NEXT_PUBLIC_KAKAO_REDIRECT_URI ?? '',
          token: code,
        }
      : undefined;

 /* 카카오 로그인 요청 */
  const { data, error } = useQuery({
    queryKey: ['handleKakaoAuth'],
    queryFn: async () => {
     // code 값이 없으면 요청되지 않습니다.
      if (!params) return;
      return await signInKakao(params);
    },
    enabled: !!params,
    retry: false,
  });

 /* 유저 데이터 요청 */
  const { refetch: userRefecth } = useQuery({
    queryKey: ['getUser'],
    queryFn: getUser,
    enabled: false,
    retry: false,
  });
  useEffect(() => {
    if (!data) return;

    /* API 성공 시 전역 유저 데이터 세팅 */
    const setUserData = async () => {
      try {
        // 유저 정보를 받아옵니다.
        const { data: userData } = await userRefecth();
        if (!userData) return;
        // 전역 유저 데이터를 세팅합니다.
        setUser(userData);
        router.push('/');
      } catch (error) {
        if (axios.isAxiosError(error) && error.response) {
          // 유저 정보 API 에러를 모달로 출력
          handleError(error.response?.data as Error);
        }
      }
    };

    setUserData();
  }, [data, router, setUser, userRefecth, handleError]);

🗂️ 관련 이슈

📸 스크린샷

[수정 전]
image

[수정 후]
image

✅ 체크리스트

  • 빌드 및 테스트 통과
  • ESLint/Prettier 검사 통과

@vercel
Copy link

vercel bot commented Jul 30, 2025

@summerDev96 is attempting to deploy a commit to the 626-ju's projects Team on Vercel.

A member of the Team first needs to authorize it.

@summerDev96 summerDev96 changed the title fix: 회원가입/로그인 화면 UI 오류 수정 fix: 카카오 로그인 API 추가, 회원가입/로그인 화면 UI 오류 수정 Jul 30, 2025
@summerDev96 summerDev96 merged commit 56b3e5c into codeit-part3-7:dev Jul 31, 2025
1 of 2 checks passed
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