Add optimistic concurrency (RowVersion) to user-editable entities
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 48/100
調査の方向性
LearnStack.Core/Data/Models/LearningResource.cs と LearnStack.Core/Services/LearningResourceService.cs:105-130 から始め、続いて 157-176 行にあるサービスの一括並べ替えの処理経路を追跡します。対応する ContentIdea モデルとサービスを特定し、DbUpdateConcurrencyException がユーザー向けのフローに到達する仕組みを確認します。同時編集時に競合メッセージが表示され、古いクライアント側の並べ替えによって変更が暗黙的に失われなくなれば完了です。
索引モデルが issue の本文から書いたものです。
説明
Tier 2 — correctness
Problem
No entity carries a concurrency token, and the update services copy every field from the incoming object onto the tracked entity. Two browser tabs — very common in a Blazor Server app, and the resource list is also edited from dialogs — silently overwrite each other, last write wins. Drag-and-drop reordering makes this worse because it rewrites CustomOrder across many rows at once.
Evidence
LearnStack.Core/Data/Models/LearningResource.cs— noRowVersionLearnStack.Core/Services/LearningResourceService.cs:105-130— wholesale field copyLearnStack.Core/Services/LearningResourceService.cs:157-176— bulkCustomOrderrewrite
Proposed fix
- Add
[Timestamp] public byte[]? RowVersion { get; set; }toLearningResourceandContentIdea. - Handle
DbUpdateConcurrencyExceptionin the services and surface a "this item changed elsewhere" message instead of silently overwriting.
Acceptance criteria
- Concurrent edits produce a conflict message rather than silent data loss
- Reordering is resilient to a stale client-side list
- 主要言語
- 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
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
agentic-os-org/ANOLISA#6742 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
メンテナーはふだん 1 日以内に返信
-
type/automation type/tech-debt
難易度 1/5 1時間未満 初心者へのやさしさ 72/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
paperclipai/paperclip#15628 ·
メンテナーはふだん 1 日以内に返信
-
feature/cohorts feature/feature-flags team/feature-flags
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
メンテナーはふだん 1 日以内に返信