Skip to content

[REFACTOR] 앱에서 연결되지 않는 본인 인증 템플릿 화면(/signup/verify·/find-account/verify) 삭제 (#233) - #234

Merged
SinYusi merged 1 commit into
mainfrom
refactor/233-remove-unlinked-verify-pages
Oct 4, 2026
Merged

SinYusi merged 1 commit into
mainfrom
refactor/233-remove-unlinked-verify-pages

Conversation

@SinYusi

@SinYusi SinYusi commented Oct 4, 2026 •

Copy link
Copy Markdown
Contributor

관련 이슈

closes #233

변경 사항

  • app/signup/verify/page.tsx(+ loading.tsx)·app/find-account/verify/page.tsx 삭제 — 앱 어디에서도 연결되지 않는 초기 템플릿 화면

리뷰어 참고 사항

  • 연결이 없다는 근거
    • 저장소 안에 두 경로(/signup/verify·/find-account/verify)나 두 컴포넌트를 가리키는 곳이 없습니다(삭제한 파일 자신 제외)
    • /signup/verify는 2025-06-29 커밋 60d98b5에서 로그인 화면 링크 3곳이 /signup으로 바뀐 뒤 들어올 길이 없습니다. /find-account/verify는 처음부터 링크가 없었습니다
  • 두 화면은 아이핀·휴대전화 버튼을 누르면 실제 본인인증 없이 다음 화면으로 넘어가는 템플릿이었습니다. 리디자인 전 스타일(파란 머리 띠·경로 표시·인쇄 버튼·h1 두 개)도 그대로였습니다
  • 주소를 직접 입력하면 이제 404입니다
  • /find-account/reset-password도 지금은 삭제한 /find-account/verify에서만 연결되지만, 이번 범위에서는 남겼습니다

추가 정보

검증

  • pnpm exec tsc --noEmit 통과 · pnpm test 318개 통과 · pnpm build 통과(두 경로가 라우트 목록에서 빠짐)
  • 코드 중복(jscpd 5.2.1, 테스트·components/ui 제외, main 2255200 ↔ head): 1.96% → 0.71%(265 → 94줄), 클론 16 → 8
  • 파일 169 → 166, 줄(빈 줄 제외) 13,561 → 13,239

🤖 Generated with Claude Code

Summary by CodeRabbit

  • 변경 사항
    • 계정 찾기 인증 페이지가 제거되었습니다.
    • 회원가입 인증 페이지와 해당 페이지의 로딩 화면이 제거되었습니다.

@vercel

vercel Bot commented Oct 4, 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 4, 2026 7:50am UTC

@coderabbitai

coderabbitai Bot commented Oct 4, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

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: 9479311a-bf34-42af-be7b-4f3b3da0f3cc
📥 Commits

Reviewing files that changed from the base of the PR and between 2255200 and 1459894.

📒 Files selected for processing (3)
  • app/find-account/verify/page.tsx
  • app/signup/verify/loading.tsx
  • app/signup/verify/page.tsx
💤 Files with no reviewable changes (3)
  • app/signup/verify/loading.tsx
  • app/signup/verify/page.tsx
  • app/find-account/verify/page.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

회원가입 및 계정 찾기의 인증 페이지와 회원가입 인증 페이지의 로딩 UI를 삭제했습니다. 두 인증 페이지는 인증 버튼, 안내 문구, 탐색 UI를 제공했으며, 버튼을 누르면 인증 절차 없이 다음 경로로 이동했습니다.

Changes

인증 템플릿 화면 삭제

Layer / File(s) Summary
인증 페이지와 로딩 UI 삭제
app/signup/verify/page.tsx, app/signup/verify/loading.tsx, app/find-account/verify/page.tsx
회원가입 및 계정 찾기 인증 페이지를 삭제했습니다. 회원가입 인증 페이지의 스켈레톤 로딩 UI도 삭제했습니다.

Priority: ⬇️ Low

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

Change: Refactor

Merge Risk: ⚪ Minimal · up to 14598

This change removes unused verification placeholder screens, so direct visits to those two routes will return 404. No merge-blocking risk was found.

🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed 삭제 대상 경로와 변경 내용이 명확해 PR의 주요 변경 사항을 잘 요약합니다.
Description check ✅ Passed 템플릿의 모든 섹션을 작성했습니다. 삭제 범위, 미사용 근거, 검증 결과와 제외 범위를 구체적으로 설명합니다.
Linked Issues check ✅ Passed [이슈 #233] 요청한 app/signup/verify/page.tsx, app/signup/verify/loading.tsx, app/find-account/verify/page.tsx를 삭제했습니다. head 파일 목록에 두 verify 경로가 없고 /find-account/reset-password는 유지됩니다. PR 설명은 경로 참조…
Out of Scope Changes check ✅ Passed 변경 요약은 이슈 #233의 세 대상 파일 삭제만 보여 줍니다. 별도 범위인 /find-account/reset-password는 head에 남아 있습니다. 무관한 변경은 확인되지 않았습니다.
✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autopilot · 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 commented Oct 4, 2026

Copy link
Copy Markdown
Contributor Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Oct 4, 2026 •

Copy link
Copy Markdown
✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@SinYusi
SinYusi force-pushed the refactor/233-remove-unlinked-verify-pages branch from 1459894 to dbc011d Compare October 4, 2026 07:48
@SinYusi
SinYusi merged commit b8da95d into main Oct 4, 2026
2 of 4 checks passed
@SinYusi
SinYusi deleted the refactor/233-remove-unlinked-verify-pages branch October 4, 2026 07:48

This branch was successfully deployed

1 active deployment
Preview — dbc011da Deployed Oct 4, 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] 앱에서 연결되지 않는 본인 인증 템플릿 화면(/signup/verify·/find-account/verify) 삭제

1 participant