Yimaru/Yimaru-Admin
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Add lesson detail fallback for new lessons

Browse Source 
Handle environments where sub-module lesson detail is missing by falling back to question-set detail, and keep lesson form/status typing safe.

Made-with: Cursor
This commit is contained in:
Yared Yemane 2026-04-16 06:25:09 -07:00
parent 3ad0f0a63d
commit 5ee897cfad
1 changed files with 60 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

68
src/pages/content-management/HumanLanguagePage.tsx
View File

@ -929,7 +929,7 @@ export function HumanLanguagePage() {
}
}
const loadLessonDetailIfNeeded = async (lessonId: number, forceRefresh = false) => {
const loadLessonDetailIfNeeded = async (lessonId: number, lessonQuestionSetId?: number, forceRefresh = false) => {
let skipFetch = false
setLessonDetailState((prev) => {
const ex = prev[lessonId]
@ -953,6 +953,38 @@ export function HumanLanguagePage() {
[lessonId]: { status: "ok", data },
}))
} catch (error) {
// Fallback: some environments return lessons via question-set data,
// and /sub-module-lessons/:id may not have a corresponding row yet.
if (lessonQuestionSetId && lessonQuestionSetId > 0) {
try {
const setDetail = (await withTimeout(getQuestionSetById(lessonQuestionSetId), 12000)).data?.data
if (setDetail) {
setLessonDetailState((prev) => ({
...prev,
[lessonId]: {
status: "ok",
data: {
id: lessonId,
sub_module_id: 0,
question_set_id: lessonQuestionSetId,
intro_video_url: setDetail.intro_video_url ?? null,
display_order: 0,
is_active: true,
title: setDetail.title ?? "",
description: setDetail.description ?? "",
status: setDetail.status ?? "DRAFT",
set_type: setDetail.set_type ?? "QUIZ",
question_count: Number(setDetail.question_count ?? 0),
},
},
}))
return
}
} catch (fallbackError) {
console.error("Fallback question-set detail failed:", fallbackError)
}
}
console.error("Failed to load lesson detail:", error)
setLessonDetailState((prev) => ({
...prev,
@ -1033,18 +1065,38 @@ export function HumanLanguagePage() {
})
try {
const detail = (await getSubModuleLessonById(lesson.id, { cacheBust: true })).data?.data
let detail: SubModuleLessonDetail | null =
(await getSubModuleLessonById(lesson.id, { cacheBust: true })).data?.data ?? null
if (!detail && lesson.question_set_id > 0) {
const setDetail = (await getQuestionSetById(lesson.question_set_id)).data?.data
detail = setDetail
? ({
id: lesson.id,
sub_module_id: 0,
question_set_id: lesson.question_set_id,
intro_video_url: setDetail.intro_video_url ?? null,
display_order: 0,
is_active: true,
title: setDetail.title ?? "",
description: setDetail.description ?? "",
status: setDetail.status ?? "DRAFT",
set_type: setDetail.set_type ?? "QUIZ",
question_count: Number(setDetail.question_count ?? 0),
} as SubModuleLessonDetail)
: null
}
if (lessonEditFetchIdRef.current !== requestId) return
const normalizedStatus =
detail?.status && ["DRAFT", "PUBLISHED", "ARCHIVED"].includes(detail.status)
? (detail.status as "DRAFT" | "PUBLISHED" | "ARCHIVED")
: "DRAFT"
setLessonForm({
title: detail?.title ?? lesson.title ?? "",
description: detail?.description ?? "",
introVideoUrl: detail?.intro_video_url ?? "",
status:
(detail?.status as "DRAFT" | "PUBLISHED" | "ARCHIVED" | undefined) &&
["DRAFT", "PUBLISHED", "ARCHIVED"].includes(detail.status)
? (detail.status as "DRAFT" | "PUBLISHED" | "ARCHIVED")
: "DRAFT",
status: normalizedStatus,
})
} catch (error) {
if (lessonEditFetchIdRef.current !== requestId) return
@ -1441,7 +1493,7 @@ export function HumanLanguagePage() {
const cur = prev[smKey] ?? { lessonId: null, practiceId: null }
return { ...prev, [smKey]: { ...cur, lessonId: nextLessonId } }
})
if (nextLessonId !== null) void loadLessonDetailIfNeeded(nextLessonId)
if (nextLessonId !== null) void loadLessonDetailIfNeeded(nextLessonId, lessonQuestionSetId)
if (nextLessonId !== null && lessonQuestionSetId > 0) void loadLessonQuestionsIfNeeded(lessonQuestionSetId)
}