Skip to content

Kakao 회원가입 #119

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

Merged
a06246 merged 3 commits into from
Dec 13, 2025
Merged

Kakao 회원가입 #119

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
0117f08
카카오 로그인 관련 API 업데이트
a06246 Dec 12, 2025
8c6d0fe
카카오 로그인 온보딩 화면 추가 및 라우팅 분기 처리
a06246 Dec 13, 2025
882dd50
Merge branch 'main' of https://github.com/intel-fit/front-end into k
a06246 Dec 13, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions src/navigation/AppNavigator.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@ import LoginScreen from "../screens/auth/LoginScreen";
import SignupScreen from "../screens/auth/SignupScreen";
import FindIdScreen from "../screens/auth/FindIdScreen";
import ResetPasswordScreen from "../screens/auth/ResetPasswordScreen";
import KakaoOnboardingScreen from "../screens/auth/KakaoOnboardingScreen";

// Main Screens
import HomeScreen from "../screens/main/HomeScreen";
Expand Down Expand Up @@ -165,6 +166,7 @@ export default function AppNavigator() {
name={ROUTES.RESET_PASSWORD}
component={ResetPasswordScreen}
/>
<Stack.Screen name="KakaoOnboarding" component={KakaoOnboardingScreen} />
{/* Main Tabs */}
<Stack.Screen name={ROUTES.MAIN} component={MainTabs} />
{/* Payment Stack*/}
Copy link

@github-actions github-actions bot Dec 13, 2025

Choose a reason for hiding this comment

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

코드 패치에 대한 몇 가지 우려 사항이 있습니다.

  1. 확장성 및 유지 관리: KakaoOnboardingScreen을 추가했지만, 해당 스크린이 의도한 대로 작동하는지에 대한 테스트가 필요합니다. 스크린이 연결된 라우트 이름이 하드코딩되어 있으므로, 변경 시 코드 전체를 수정해야 할 수 있습니다. 상수를 사용하는 것이 더 안전할 수 있습니다.

  2. 내비게이션 순서: KakaoOnboardingScreen이 MainTabs 이전에 추가되고 순서가 맞는지 확인해야 합니다. 만약 이 스크린이 사용자 흐름에 필수적이라면, 초기 화면으로 설정되어야 할 수도 있습니다.

  3. 타입 체크 및 PropTypes: 만약 TypeScript를 사용하지 않고 있다면, 새로 추가된 컴포넌트에 대한 PropTypes를 정의하는 것이 좋습니다. 이는 런타임 오류를 줄이고 코드의 안정성을 높입니다.

전반적으로 코드 추가는 필요한 작업으로 보이지만, 위의 지적 사항들을 고려하여 테스트 및 검증을 거친 후 머지하는 것이 좋습니다.

Expand Down
1 change: 1 addition & 0 deletions src/navigation/types.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@ export type RootStackParamList = {
Signup: undefined;
FindId: undefined;
ResetPassword: undefined;
KakaoOnboarding: undefined;
Main: undefined;
Chatbot: undefined;
Diet: undefined;
Expand Down
Loading