[hotfix] 지원자 상세 페이지에서 이전/다음 지원자 이동 시 흰 화면 표시 버그#1033
Conversation
- 이전/다음 지원자 버튼 클릭 시 URL 경로 수정 - 지원자 선택 드롭다운 onChange URL 경로 수정 - /admin/applicants → /admin/applicants-list/:applicationFormId/:questionId
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Warning
|
| 내용 / 파일명 | 변경 요약 |
|---|---|
지원자 상세 페이지 네비게이션 경로 수정frontend/src/pages/AdminPage/tabs/ApplicantsTab/ApplicantDetailPage/ApplicantDetailPage.tsx |
이전/다음 지원자 네비게이션 URL에 applicationFormId 추가: /admin/applicants/{id} → /admin/applicants-list/{applicationFormId}/{id}지원자 선택 네비게이션 URL에 applicationFormId 추가: /admin/applicants-list/{id} → /admin/applicants-list/{applicationFormId}/{id} |
Estimated code review effort
🎯 1 (Trivial) | ⏱️ ~5 minutes
Possibly related issues
- [fix] MOA-512 지원자 상세 페이지에서 이전/다음 지원자 이동 시 흰 화면 표시 버그 #1032: 지원자 상세 페이지의 네비게이션 코드 경로를 동일하게 업데이트하여 이전/다음 및 선택 URL에 applicationFormId를 포함시키므로 관련이 있습니다.
Possibly related PRs
- [refactor] 선택한 지원서 양식 id 새로고침해도 저장된다 #902: ApplicantDetailPage에서 applicationFormId를 context에서 route parameter로 전환하고 네비게이션 경로를 업데이트하는 동일한 변경사항을 포함하므로 직접 관련이 있습니다.
- [feature] 지원서 활성화 버튼 기능 구현 #866: 지원자 상세 페이지 네비게이션을
/admin/applicants-list/:applicationFormId/:id경로로 변경하는 것으로, 본 PR의 경로 리팩터와 정확히 동일합니다. - [feature] 지원서 폼 내에서 다른 지원서로 이동할 수 있다. #652: ApplicantDetailPage.tsx의 지원자 네비게이션(이전/다음 및 선택)을 수정하여 applicationFormId를 포함시키므로 관련이 있습니다.
Suggested labels
🐞 Bug, 💻 FE
Suggested reviewers
- lepitaaar
- oesnuj
- Zepelown
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
| Check name | Status | Explanation |
|---|---|---|
| Description Check | ✅ Passed | Check skipped - CodeRabbit’s high-level summary is enabled. |
| Title check | ✅ Passed | PR 제목이 변경 사항의 주요 목적을 명확하게 설명하고 있습니다: 지원자 상세 페이지에서 이전/다음 이동 시 발생하는 흰 화면 버그 수정. |
| Linked Issues check | ✅ Passed | PR의 코드 변경 사항이 MOA-512 이슈의 목표를 충족합니다: 네비게이션 URL 경로를 수정하여 흰 화면 버그를 해결했습니다. |
| Out of Scope Changes check | ✅ Passed | 모든 변경 사항이 지원자 상세 페이지의 네비게이션 URL 수정이라는 범위 내에 있으며, 추가적인 범위 외 변경은 없습니다. |
| Docstring Coverage | ✅ Passed | No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. |
✏️ Tip: You can configure your own custom pre-merge checks in the settings.
✨ Finishing touches
- 📝 Generate docstrings
📜 Recent review details
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Disabled knowledge base sources:
- Jira integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (1)
frontend/src/pages/AdminPage/tabs/ApplicantsTab/ApplicantDetailPage/ApplicantDetailPage.tsx
🔇 Additional comments (2)
frontend/src/pages/AdminPage/tabs/ApplicantsTab/ApplicantDetailPage/ApplicantDetailPage.tsx (2)
125-137: LGTM! 네비게이션 URL 수정이 올바르게 적용되었습니다.이전/다음 지원자 버튼의 네비게이션 경로가
/admin/applicants-list/${applicationFormId}/${applicantId}패턴으로 일관되게 수정되었습니다.applicationFormId는 라인 91-93에서 이미 유효성 검사가 되어 있으므로 안전하게 사용됩니다.
150-164: LGTM! 드롭다운 선택 시 네비게이션도 동일한 패턴으로 수정되었습니다.지원자 선택 드롭다운의
onChange핸들러가 이전/다음 버튼과 동일한 URL 패턴(/admin/applicants-list/${applicationFormId}/${applicantId})을 사용하도록 수정되어 일관성이 유지됩니다.
✏️ Tip: You can disable this entire section by setting review_details to false in your review settings.
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.
Comment @coderabbitai help to get the list of available commands and usage tips.
suhyun113
left a comment
There was a problem hiding this comment.
applicationFormId가 빠져서 생긴 문제였군뇨
빠른 수정 좋아요~!!
#️⃣연관된 이슈
📝작업 내용
/admin/applicants→/admin/applicants-list/:applicationFormId/:questionId중점적으로 리뷰받고 싶은 부분(선택)
논의하고 싶은 부분(선택)
🫡 참고사항
Summary by CodeRabbit
릴리스 노트
✏️ Tip: You can customize this high-level summary in your review settings.