Skip to content

F9-2j: 前端 LocalizeMenu 傳 project_id 套課程 glossary#77

Merged
dofliu merged 1 commit into
mainfrom
claude/f9-2j-localize-glossary-pid
Jun 14, 2026
Merged

F9-2j: 前端 LocalizeMenu 傳 project_id 套課程 glossary#77
dofliu merged 1 commit into
mainfrom
claude/f9-2j-localize-glossary-pid

Conversation

@dofliu

@dofliu dofliu commented Jun 14, 2026

Copy link
Copy Markdown
Owner

做什麼

把 F9-2i 已落地的選填 project_idPOST /localization/translate 接受、給了就套該課 glossary 的固定譯名)接上前端在地化入口,讓「一鍵在地化」真的用上課程術語表。

  • LocalizeMenu 新增 projectId prop;呼叫 /localization/translate有作用中課程才project_id...(projectId ? { project_id: projectId } : {}))。
  • 影片任務卡TaskCard):esJobToTask 補帶 rec.project_id(F9-2g 已落 JobRecord.project_id),LocalizeMenutask.project_id該 job 自己所屬的課(最精準)。
  • 視覺成品卡VisualCard):VisualStation 把作用中課程 projectId 透傳下去。

為什麼

F9-2(per-course glossary)後端已全接好——schema→儲存→API→翻譯橋接(to_translation_rules)→TTS 讀音透傳——但前端在地化入口從沒傳 project_id,所以課程固定譯名實際上套不到「一鍵在地化」。F9-2i 收尾時把這個前端 slice 標為「後續」,本 PR 補上。F9-2 連同前端在地化入口至此端到端到齊。

怎麼測

  • 純前端改動,未動 server/core/schemas/runner(route F9-2i 已落地且有 tests/test_localization_glossary.py 9 測涵蓋 project_id 行為與 fail-soft),故不需跑 pytest。
  • npm run build(vite, node22)編譯通過。
  • 視覺驗收待人工(此環境無瀏覽器,依既定「前端 build 為準、人後視覺驗收」)。

Reviewer 決策點

  • route project_id選填=沒作用中課程時前端不帶,後端 fail-soft,沿用現行行為、零破壞
  • 不碰 review gate / 狀態機(只影響「術語怎麼譯」,硬規則 Add pronunciation mapping for mathematical and Greek symbols in TTS #1 不受影響)。
  • 視覺卡(VisualCard)目前渲染的是 demo VISUAL_OUTPUTS,套的是作用中課程;影片卡套的是該 job 自己的課——兩者來源不同是刻意的(job 有自己的 project_id,視覺成品尚未綁 job)。

Generated by Claude Code

把 F9-2i 落地的選填 project_id 接上前端在地化入口:
- LocalizeMenu 新增 projectId prop,呼叫 /localization/translate 時有作用中
  課程才帶 project_id,讓後端套該課 glossary 固定譯名、術語前後一致。
- 影片任務卡(TaskCard):esJobToTask 補帶 rec.project_id(F9-2g 已落
  JobRecord.project_id),LocalizeMenu 收該 job 自己所屬的課。
- 視覺成品卡(VisualCard):VisualStation 透傳作用中課程 projectId。

route 欄位選填=沒作用中課程沿用現行行為、零破壞(fail-soft 在後端);
不碰 review gate / 狀態機(只影響術語怎麼譯)。純前端,npm run build 通過。
@dofliu dofliu marked this pull request as ready for review June 14, 2026 09:09
@dofliu dofliu merged commit a914f8b into main Jun 14, 2026
7 checks 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.

2 participants