fix(app): 코스 탭 스텝 번호를 코스 내 상대 순번으로 표시 (#290) - #291
Conversation
|
Warning Review limit reached
Next review available in: 85 minutes You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
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 |
|
🔍 프리뷰 배포: https://thumbsup-la95uhtlk-thumbsup.vercel.app
|
무엇을 / 왜
quiz_step.step_order는 코스별로 리셋되지 않는 커리큘럼 전역 순번이다(운영 DB 확인: 코스1 운영체제 112, 코스2 디자인 패턴 1314)./course탭이 이 값을 그대로 "STEP N"으로 표시해서, 두 번째 이후 코스를 펼치면 1이 아니라 13부터 시작하는 것처럼 보였다.app/src/features/home는 정확히 같은 문제를 코스 내 리스트 인덱스를 계산해 표시용으로 쓰는 방식으로 이미 해결했다 — 이번 수정도 같은 패턴을 따른다.변경 사항
course-page.tsx:course.steps.map에서 인덱스를 받아displayOrder(1-based, 코스 내 상대 순번)로CourseStepRow에 전달, 배지·"STEP N" 텍스트에step.stepOrder대신 사용getStepHref)는 그대로step.stepOrder(전역 값, API 계약)를 사용 — 표시만 바뀌고 동작은 그대로CourseListResponse가 이미 스텝을 코스별stepOrder오름차순으로 내려주므로 배열 인덱스만으로 충분stepOrder가 1보다 큰 값에서 시작하는 케이스에서 "STEP 1"·"STEP 2"로 보이고 "STEP 13" 등은 안 보이는지 확인관련 이슈
Closes #290
체크리스트