refactor: retire redirected course readers without changing legacy URLs
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 48/100
- issue の種類
- リファクタリング
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- next.js, react, typescript
調査の方向性
Start by tracing imports from src/app/courses/ and the reader components, then check next.config.ts and the protection helpers to distinguish retired code from shared code. Use e2e/checks.mjs and the reader and redirect browser checks to verify legacy URLs preserve nested paths and query strings, render the dashboard destination, and keep unknown destinations at 404. Done when only unreferenced legacy components are removed, architecture guidance names the canonical reader, and the listed checks pass.
索引モデルが issue の本文から書いたものです。
説明
Problem
next.config.ts permanently redirects /courses and /courses/:path* to /dashboard, but the repository retains a second course-reader route tree and its reader components. This makes it harder for contributors to identify the active implementation.
Scope
- Inventory the legacy route/component import graph and confirm which files are unreachable through the supported application routes.
- Remove only duplicate pages/components whose sole consumers belong to that retired tree.
- Preserve the existing permanent redirects, path segments and query strings.
- Keep the active dashboard reader, content safety rules and any shared helpers still used elsewhere.
- Update architecture guidance to identify the canonical reader.
- Treat this as dead-code cleanup; it does not fix an active reading restriction or change the student reading experience.
Starting files
next.config.ts, src/app/courses/, src/app/components/Reader/, src/app/components/Protected/NoCopyWrapper.tsx, src/hooks/useContentProtection.ts, e2e/checks.mjs and docs/architecture.md.
Done when
Tests verify direct legacy requests redirect to the matching dashboard URL, including nested unit paths and query strings. Browser checks confirm the destination renders the expected content and unknown destinations still return 404. Lint, typecheck, unit tests, production build and reader/redirect browser checks pass. Removed components have no remaining consumers.
- 主要言語
- TypeScript
- スター
- 1
- フォーク
- 1
- 平均マージ
- 8時間 58分
- マージ済み PR（30日）
- 37
環境構築
- Dockerfile または Docker Compose ファイルあり
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
procoderji-com/procoderji のほかの issue
-
area:frontend bug
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
procoderji-com/procoderji#104 ·
メンテナーはふだん 1 日以内に返信
-
enhancement good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
procoderji-com/procoderji#36 ·
メンテナーはふだん 1 日以内に返信
-
enhancement good first issue
難易度 1/5 1〜3時間 初心者へのやさしさ 85/100
procoderji-com/procoderji#35 ·
メンテナーはふだん 1 日以内に返信
-
enhancement good first issue
難易度 1/5 1〜3時間 初心者へのやさしさ 79/100
procoderji-com/procoderji#34 ·
メンテナーはふだん 1 日以内に返信
-
enhancement good first issue
難易度 1/5 1〜3時間 初心者へのやさしさ 90/100
procoderji-com/procoderji#33 ·
メンテナーはふだん 1 日以内に返信
procoderji-com/procoderji の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
メンテナーはふだん 1 日以内に返信
-
bug platform-android
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
Termix-SSH/Mobile#146 · コメント 1 件 ·
メンテナーはふだん 2 日以内に返信
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 66/100
devswha/herdr-web-ui#3875 ·
-
難易度 1/5 1時間未満 初心者へのやさしさ 88/100
-
area: runtime chore cli consistency dependencies developer-experience difficulty: easy discovery documentation enhancement good first issue help wanted hygiene json-output packaging priority: P3 tests version
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
メンテナーはふだん 1 日以内に返信