Skip to content

Fix: 결과 로딩 중 헤더 수정 [#122] - #123

Merged
minnngo merged 1 commit into
developfrom
feature/#122-header-fix
Jul 31, 2026
Merged

Fix: 결과 로딩 중 헤더 수정 [#122]#123
minnngo merged 1 commit into
developfrom
feature/#122-header-fix

Conversation

@minnngo

@minnngo minnngo commented Jul 31, 2026

Copy link
Copy Markdown
Collaborator

🔗 관련 이슈

📝 개요

  • 결과 로딩 중 헤더 수정

⌨️ 작업 상세 내용

  • [ ]
  • [ ]

Summary by CodeRabbit

  • New Features
    • Resume analysis loading screens now display the relevant company name and job title.
    • Job titles use available fallback details when the primary title is unavailable.

@minnngo
minnngo merged commit e3e3a95 into develop Jul 31, 2026
1 check was pending
@coderabbitai

coderabbitai Bot commented Jul 31, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 8abff176-b237-4d81-8d6a-be3f3ba0efc0

📥 Commits

Reviewing files that changed from the base of the PR and between 86527a0 and 7b38656.

📒 Files selected for processing (2)
  • jobdri/src/app/mockApply/[mockApplyId]/result/resume-analysis-loading/ResumeAnalysisLoadingPageClient.tsx
  • jobdri/src/components/mockApply/ResumeAnalysisLoading.tsx

📝 Walkthrough

Walkthrough

The loading page fetches mock-application job-posting metadata, applies a job-title fallback, and passes the company name and job title to the loading header.

Changes

Resume Analysis Loading Header

Layer / File(s) Summary
Fetch job-posting metadata
jobdri/src/app/mockApply/[mockApplyId]/result/resume-analysis-loading/ResumeAnalysisLoadingPageClient.tsx
The page fetches company and job title data for valid application IDs. It logs failures and prevents state updates after cleanup.
Render loading header metadata
jobdri/src/components/mockApply/ResumeAnalysisLoading.tsx, jobdri/src/app/mockApply/[mockApplyId]/result/resume-analysis-loading/ResumeAnalysisLoadingPageClient.tsx
ResumeAnalysisLoading accepts optional metadata and passes it to Header.

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

Sequence Diagram(s)

sequenceDiagram
  participant ResumeAnalysisLoadingPageClient
  participant JobPostingAPI
  participant ResumeAnalysisLoading
  participant Header
  ResumeAnalysisLoadingPageClient->>JobPostingAPI: fetch job-posting metadata
  JobPostingAPI-->>ResumeAnalysisLoadingPageClient: return company name and job title
  ResumeAnalysisLoadingPageClient->>ResumeAnalysisLoading: pass metadata
  ResumeAnalysisLoading->>Header: pass company name, job title, and application label
Loading

Possibly related PRs

Suggested labels: 🎨 UI

Suggested reviewers: yiyoonseo

✨ 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/#122-header-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.

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