Skip to content

[fix] 애플 로그인 시 이름 스킵#144

Merged
ff1451 merged 1 commit intodevelopfrom
143-fix-애플-로그인-시-회원가입-단계에서-이름-입력-단계-스킵
Feb 28, 2026

Hidden character warning

The head ref may contain hidden characters: "143-fix-\uc560\ud50c-\ub85c\uadf8\uc778-\uc2dc-\ud68c\uc6d0\uac00\uc785-\ub2e8\uacc4\uc5d0\uc11c-\uc774\ub984-\uc785\ub825-\ub2e8\uacc4-\uc2a4\ud0b5"
Merged

[fix] 애플 로그인 시 이름 스킵#144
ff1451 merged 1 commit intodevelopfrom
143-fix-애플-로그인-시-회원가입-단계에서-이름-입력-단계-스킵

Conversation

@ff1451
Copy link
Collaborator

@ff1451 ff1451 commented Feb 28, 2026

Summary by CodeRabbit

릴리스 노트

  • 새로운 기능
    • 회원가입 시 사용자 이름이 자동으로 미리 채워집니다. 이름이 이미 등록된 경우 확인 단계로 바로 진행되며, 그렇지 않으면 이름 입력 단계로 이동합니다.

@ff1451 ff1451 self-assigned this Feb 28, 2026
@coderabbitai
Copy link

coderabbitai bot commented Feb 28, 2026

📝 Walkthrough

Walkthrough

이 PR은 가입 과정에서 사용자 이름을 미리 가져오는 기능을 추가합니다. getSignupPrefill API 함수를 새로 작성하고, 이를 활용하는 useSignupPrefill 커스텀 훅을 구현했습니다. StudentIdStep 컴포넌트에서 해당 훅을 사용하여 이름 데이터 로드 여부에 따라 다음 단계를 분기 처리합니다.

Possibly related issues


추가 피드백

React/TypeScript 컨벤션 관점:

  • useSignupPrefill 훅의 enabled 옵션 추가 검토: 특정 조건에서만 쿼리를 실행해야 한다면 enabled 속성 사용을 고려하세요.
  • isPending 상태 외에 에러 상태(isError) 처리 추가 검토: 에러 발생 시 사용자 경험 개선을 위해 에러 핸들링을 추가하는 것을 권장합니다.
  • StudentIdStep의 네비게이션 로직이 명확하므로 좋습니다. 다만 분기 조건(prefill?.name)의 존재 여부를 먼저 명시적으로 확인하면 코드 가독성이 향상됩니다.
🚥 Pre-merge checks | ✅ 2
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed 제목은 Apple 로그인 시 회원가입 단계에서 이름 입력을 스킵하는 실제 변경사항을 정확히 반영하고 있습니다.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch 143-fix-애플-로그인-시-회원가입-단계에서-이름-입력-단계-스킵

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

🧹 Nitpick comments (1)
src/pages/Auth/SignUp/StudentIdStep.tsx (1)

12-12: 로딩/에러 상태 UX 개선 고려

isPending 동안 버튼이 비활성화되지만, 사용자에게 로딩 중임을 시각적으로 알려주는 것이 좋습니다. isError 상태도 활용할 수 있습니다.

- const { data: prefill, isPending } = useSignupPrefill();
+ const { data: prefill, isPending, isError } = useSignupPrefill();
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@src/pages/Auth/SignUp/StudentIdStep.tsx` at line 12, The signup prefill hook
returns isPending (and likely isError); update the StudentIdStep UI to surface
these states: when useSignupPrefill().isPending is true show a visual loading
indicator (e.g., spinner/LoadingButton state) on the primary action button and
keep it disabled, and when useSignupPrefill().isError is true show a concise
inline error message with an affordance to retry (e.g., a retry button or re-run
the prefill call) so users know why prefill failed; locate the usage of
useSignupPrefill, prefill, isPending and add the loading/error UI and retry
handler accordingly.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@src/pages/Auth/SignUp/StudentIdStep.tsx`:
- Line 12: The signup prefill hook returns isPending (and likely isError);
update the StudentIdStep UI to surface these states: when
useSignupPrefill().isPending is true show a visual loading indicator (e.g.,
spinner/LoadingButton state) on the primary action button and keep it disabled,
and when useSignupPrefill().isError is true show a concise inline error message
with an affordance to retry (e.g., a retry button or re-run the prefill call) so
users know why prefill failed; locate the usage of useSignupPrefill, prefill,
isPending and add the loading/error UI and retry handler accordingly.

ℹ️ Review info

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 6c3e678 and fbfe3d7.

📒 Files selected for processing (3)
  • src/apis/auth/index.ts
  • src/pages/Auth/SignUp/StudentIdStep.tsx
  • src/pages/Auth/SignUp/hooks/useSignupPrefill.ts

@ff1451 ff1451 merged commit 36acf53 into develop Feb 28, 2026
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.

[fix] 애플 로그인 시 회원가입 단계에서 이름 입력 단계 스킵

1 participant