Skip to content

[REFACTOR] 앱에서 연결되지 않고 비밀번호 찾기 탭과 중복인 /find-account/reset-password 화면 삭제 (#246) - #247

Merged
SinYusi merged 1 commit into
mainfrom
refactor/246-remove-reset-password-page
Oct 15, 2026
Merged

SinYusi merged 1 commit into
mainfrom
refactor/246-remove-reset-password-page

Conversation

@SinYusi

@SinYusi SinYusi commented Oct 15, 2026 •

Copy link
Copy Markdown
Contributor

관련 이슈

closes #246

변경 사항

  • app/find-account/reset-password/page.tsx(271줄) 삭제
  • 경로 표시(Breadcrumb) 이름표에서 쓰이지 않게 된 reset-password·verify 제거(verify 경로는 #233에서 삭제됨)

리뷰어 참고 사항

  • 연결이 없다는 근거: 저장소 안에 이 경로로 이동하는 곳이 없습니다. 유일한 입구였던 /find-account/verify는 #233에서 삭제됐습니다
  • 같은 기능이 이미 있습니다: 비밀번호 찾기 탭의 "새 비밀번호" 단계(hooks/useFindPassword.ts)가 같은 임시 토큰(tempPasswordToken)과 같은 API(updatePassword)로 변경을 끝냅니다. 이 화면은 그 단계의 옛 사본이었습니다
  • 주소를 직접 열어도 토큰이 없으면 /find-account로 돌려보내던 화면이라, 404가 되는 것 외에 영향은 없습니다
  • 비밀번호 찾기 탭 흐름은 바뀌지 않습니다(관련 테스트 그대로 통과)

추가 정보

검증

  • pnpm exec tsc --noEmit 통과 · pnpm test 342개 통과 · pnpm build 통과(라우트 목록에서 /find-account/reset-password 빠짐)

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Bug Fixes
    • 비밀번호 재설정 페이지를 제거했습니다. 계정 찾기 흐름에서 비밀번호 재설정 기능을 더 이상 제공하지 않습니다.
  • Style
    • 경로 이동 시 표시되는 navigation breadcrumb의 "계정찾기" 항목 순서를 조정했습니다.

… 쓰이지 않는 경로 이름표 삭제 (#246)

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@vercel

vercel Bot commented Oct 15, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
raillo-frontend Ready Ready Preview Oct 15, 2026 4:58am UTC

@coderabbitai

coderabbitai Bot commented Oct 15, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Repository: penggu-dev/raillo-frontend/.coderabbit.yaml
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: cd8994f2-201d-4b44-8300-91263617d377
📥 Commits

Reviewing files that changed from the base of the PR and between 28b656b and f99bc17.

📒 Files selected for processing: 2
  • app/find-account/reset-password/page.tsx
  • components/layout/Header/Breadcrumb.tsx
💤 Files with no reviewable changes: 2
  • app/find-account/reset-password/page.tsx
  • components/layout/Header/Breadcrumb.tsx

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

Walkthrough

/find-account/reset-password 페이지를 삭제했습니다. Breadcrumb의 PATH_MAP에서는 find-account 항목의 위치를 변경했으며, 키와 값은 유지했습니다.

Changes

계정 경로 정리

Layer / File(s) Summary
재설정 페이지 및 경로 매핑 정리
app/find-account/reset-password/page.tsx, components/layout/Header/Breadcrumb.tsx
ResetPasswordPage를 삭제했습니다. PATH_MAP의 find-account 항목을 complete 뒤로 옮겼습니다. 항목의 키와 값은 변경하지 않았습니다.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Refactor

Merge Risk: ⚪ Minimal · up to f99bc

The obsolete password-reset page and breadcrumb labels have been removed as requested. No issue identified here needs to be resolved before merge.

Pre-merge checks: Passed 4
Passed Passed checks: 4 passed
Check name Status Explanation
Title check Passed 제목이 /find-account/reset-password 화면 삭제와 중복 기능 정리라는 주요 변경 사항을 정확히 설명합니다. 범위와 목적이 명확하고 간결합니다.
Description check Passed 설명서의 모든 필수 섹션을 포함합니다. 관련 이슈, 변경 사항, 리뷰어 참고 사항, 검증 결과를 구체적으로 작성했으며 PR 목적과 변경 내용이 일치합니다.
Linked Issues check Passed #246의 코딩 요구사항을 충족합니다. app/find-account/reset-password/page.tsx를 삭제했고, components/layout/Header/Breadcrumb.tsx에서 verify와 reset-password 라벨을 삭제했습니다. 현재 app/find-account 목록에 삭제된 화면이 없고, PR 설명에 …
Out of Scope Changes check Passed 변경은 #246의 대상인 사용되지 않는 비밀번호 재설정 화면 삭제와 Breadcrumb 라벨 정리에 한정됩니다. 비밀번호 찾기 탭의 동작 변경이나 관련 없는 파일 변경은 확인되지 않습니다.
✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@SinYusi
SinYusi merged commit bce65c7 into main Oct 15, 2026
4 checks passed
@SinYusi
SinYusi deleted the refactor/246-remove-reset-password-page branch October 15, 2026 08:37

This branch was successfully deployed

1 active deployment
Preview — f99bc172 Deployed Oct 15, 2026 by vercel[bot]
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.

[REFACTOR] 앱에서 연결되지 않고 비밀번호 찾기 탭과 중복인 /find-account/reset-password 화면 삭제

1 participant