Remove the dead CultureMiddleware and centralize the supported-culture list
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 初心者へのやさしさ
- 76/100
- issue の種類
- リファクタリング
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- csharp
- 領域
- backend, localization
調査の方向性
LearnStack/Program.cs:85 から始め、Middleware/CultureMiddleware.cs、Controllers/CultureController.cs、Extensions/LocalizationExtensions.cs のカルチャーリストを比較します。使用されていないミドルウェアを削除し、LocalizationExtensions を通じてリストを一元化します。その後、cookie と Accept-Language による言語切り替えが引き続き機能し、サポート対象カルチャーのリストが 1 つだけ残っていることを確認します。
索引モデルが issue の本文から書いたものです。
説明
Tier 3 — cleanup
Problem
CultureMiddleware implements full cookie + Accept-Language culture negotiation but is never registered — Program.cs uses UseRequestLocalization() instead, which does the same job. The dead middleware is a maintenance trap: the supported-culture list is now duplicated in three places that can silently drift apart.
Evidence
LearnStack/Middleware/CultureMiddleware.cs— never referenced inProgram.csLearnStack/Program.cs:85—app.UseRequestLocalization()- Duplicated culture lists:
Middleware/CultureMiddleware.cs:10,Controllers/CultureController.cs:10,Extensions/LocalizationExtensions.cs:7
Proposed fix
- Delete
CultureMiddleware.cs. - Expose one
SupportedCulturesconstant (inLocalizationExtensions) and reference it fromCultureController.
Acceptance criteria
- One source of truth for supported cultures
- Language switching still works via cookie and
Accept-Language
- 主要言語
- HTML
- スター
- 10
- フォーク
- 0
- 平均マージ
- 2時間 35分
- マージ済み PR（30日）
- 21
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
kasuken/LearnStack のほかの issue
-
enhancement
難易度 1/5 1時間未満 初心者へのやさしさ 91/100
kasuken/LearnStack#59 ·
メンテナーはふだん 1 日以内に返信
-
enhancement
難易度 5/5 1週間以上 初心者へのやさしさ 35/100
kasuken/LearnStack#72 ·
メンテナーはふだん 1 日以内に返信
-
enhancement
難易度 5/5 1週間以上 初心者へのやさしさ 30/100
kasuken/LearnStack#71 ·
メンテナーはふだん 1 日以内に返信
-
enhancement
難易度 5/5 1週間以上 初心者へのやさしさ 35/100
kasuken/LearnStack#70 ·
メンテナーはふだん 1 日以内に返信
-
enhancement
難易度 5/5 1週間以上 初心者へのやさしさ 45/100
kasuken/LearnStack#69 ·
メンテナーはふだん 1 日以内に返信
kasuken/LearnStack の issue をすべて見る
似ている issue
-
[Bug][Python SDK] RecallMemory drops RFC-3339 created_at write-time timestamp returned by relayerオープン
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
MystenLabs/MemWal#1160 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
-
Idle compaction monitors LIST the replica every tick when the newest destination file spans more than one TXID対応中かも @pishuv が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
benbjohnson/litestream#1563 ·
メンテナーはふだん 2 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
メンテナーはふだん 1 日以内に返信
-
Fix Math.ceilDiv wrong result for exact positive divisions対応中かも @pamod-madubashana が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
scala-native/scala-native#5094 ·
メンテナーはふだん 1 日以内に返信
-
難易度 1/5 1時間未満 初心者へのやさしさ 88/100
メンテナーはふだん 1 日以内に返信