Skip to content

Fix: 빌드 에러 수정#106

Merged
yiyoonseo merged 1 commit into
developfrom
feature/build-error-fix
Jul 26, 2026
Merged

Fix: 빌드 에러 수정#106
yiyoonseo merged 1 commit into
developfrom
feature/build-error-fix

Conversation

@yiyoonseo

@yiyoonseo yiyoonseo commented Jul 26, 2026

Copy link
Copy Markdown
Contributor

🔗 관련 이슈

  • Close #

📝 개요

  • 빌드 에러 수정

⌨️ 작업 상세 내용

  • 타입 강화
  • 일부 필요 없는 파일 삭제

💡 코드 설명 및 참고사항

📸 스크린샷 (UI 변경 시)

🔍 리뷰 요구사항 (Reviewers)

  • [ ]

⚠️ 로컬 실행 시 유의사항

Summary by CodeRabbit

  • New Features

    • Improved the credit purchase page’s loading experience with a dedicated loading state.
  • Bug Fixes

    • Updated credit plan display and purchase confirmation handling.
  • Changes

    • Removed the mock application flow for selecting an application type and entering or reviewing job descriptions.
    • Removed previously available mock application home-page exports and related navigation components.

@coderabbitai

coderabbitai Bot commented Jul 26, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 570d3f9f-3a61-4934-a9e9-5cd41cc96de6

📥 Commits

Reviewing files that changed from the base of the PR and between 877f312 and c06c800.

📒 Files selected for processing (13)
  • jobdri/src/app/credit/page.tsx
  • jobdri/src/app/mockApply/[mockApplyId]/(jd)/jd-input/JdInputPageClient.tsx
  • jobdri/src/app/mockApply/[mockApplyId]/(jd)/jd-input/page.tsx
  • jobdri/src/app/mockApply/[mockApplyId]/(jd)/jd-review/JdReviewPageClient.tsx
  • jobdri/src/app/mockApply/[mockApplyId]/(jd)/jd-review/page.tsx
  • jobdri/src/app/mockApply/[mockApplyId]/(jd)/page.tsx
  • jobdri/src/app/mockApply/apply-type/ApplyTypePageClient.tsx
  • jobdri/src/app/mockApply/apply-type/page.tsx
  • jobdri/src/components/mock-application/MockApplicationHomePageClient.tsx
  • jobdri/src/components/mock-application/home/MockApplicationHomeIntro.tsx
  • jobdri/src/components/mock-application/home/applicationHomeUtils.ts
  • jobdri/src/components/mock-application/home/index.ts
  • jobdri/src/components/mockApply/home/index.ts
💤 Files with no reviewable changes (12)
  • jobdri/src/components/mock-application/home/applicationHomeUtils.ts
  • jobdri/src/app/mockApply/[mockApplyId]/(jd)/jd-input/JdInputPageClient.tsx
  • jobdri/src/app/mockApply/[mockApplyId]/(jd)/page.tsx
  • jobdri/src/components/mock-application/home/MockApplicationHomeIntro.tsx
  • jobdri/src/app/mockApply/apply-type/page.tsx
  • jobdri/src/components/mock-application/MockApplicationHomePageClient.tsx
  • jobdri/src/components/mockApply/home/index.ts
  • jobdri/src/app/mockApply/apply-type/ApplyTypePageClient.tsx
  • jobdri/src/components/mock-application/home/index.ts
  • jobdri/src/app/mockApply/[mockApplyId]/(jd)/jd-input/page.tsx
  • jobdri/src/app/mockApply/[mockApplyId]/(jd)/jd-review/page.tsx
  • jobdri/src/app/mockApply/[mockApplyId]/(jd)/jd-review/JdReviewPageClient.tsx

📝 Walkthrough

Walkthrough

The credit page separates content rendering behind a Suspense boundary. Mock-application JD and apply-type pages and related client modules are removed, while mock-application home re-export shims are deleted.

Changes

Credit purchase page

Layer / File(s) Summary
Credit content and loading boundary
jobdri/src/app/credit/page.tsx
Moves credit fetching, purchase handling, pricing, and UI rendering into CreditContent; CreditPage now provides the layout and Suspense fallback.

Mock-application surface cleanup

Layer / File(s) Summary
JD and apply-type route removal
jobdri/src/app/mockApply/...
Removes the apply-type page, JD layout page, JD input page, and JD review page implementations and related client components.
Home export cleanup
jobdri/src/components/mock-application/..., jobdri/src/components/mockApply/home/index.ts
Removes forwarding exports for mock-application home components, utilities, and types.

Estimated code review effort: 3 (Moderate) | ~25 minutes

Possibly related PRs

Suggested reviewers: minnngo

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title matches the PR objective of fixing build errors and is concise and specific enough.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feature/build-error-fix

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.

@yiyoonseo
yiyoonseo merged commit 74028a2 into develop Jul 26, 2026
1 check 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.

1 participant