dashboard: a split URL draft's variant edit shows generic copy when the API refuses it
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 70/100
- issue の種類
- バグ
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- javascript
- 領域
- frontend
調査の方向性
design の保存に対する固定の 422 メッセージは、ダッシュボードの design エディターの保存処理にあります。issue ではファイル名が示されていません。そのメッセージが設定されている箇所を見つけ、最初のエラー項目の type が ["body","variants"] の位置にある split_url_config かどうかで分岐させてください。そのケースで新しい文言が表示され、jest のテストがそれを検証していれば完了です。一般的な 422 では従来の文言が表示され続けます。
索引モデルが issue の本文から書いたものです。
説明
From the release that carries #1560, a split URL experiment's variants must match its split_url_config (the same names, the same split). The dashboard's design editor can still edit the variants of a split URL draft created through the API. Renaming a variant there, or changing its split, now answers 422 at ["body","variants"] with a message that says what to change. The dashboard does not show that message: it shows its fixed text for any 422 on a design save ("Check that the split adds up to 100%, one variant is the control, and every metric has a name and an event name"), which names none of these reasons.
Expected
When the first item of a 422 on a design save has the type split_url_config, the dashboard says that a split URL experiment's variants and its split_url_config change together, through PUT /api/v1/experiments/{id}, and that the dashboard cannot edit split_url_config. A jest test pins the copy, with a planted generic 422 still showing the old text.
Not affected
Metric edits on the same draft still save. The refusal itself is correct: before #1560 the same edit saved a configuration that could never split. /start already shows its reason as sent.
Priority
LOW. Reached only by a split URL draft created through the API, since the dashboard cannot create one. May go after launch.
- 主要言語
- Python
- スター
- 0
- フォーク
- 4
- 平均マージ
- 5時間 36分
- マージ済み PR（30日）
- 509
環境構築
- Dockerfile または Docker Compose ファイルあり
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
Controlvar/controlvar のほかの issue
-
bug post-launch
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
Controlvar/controlvar#1748 ·
メンテナーはふだん 1 日以内に返信
-
Dashboard: the sequential panel's "Ready to stop" banner shows no sample-ratio-mismatch cautionオープンbug post-launch
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
Controlvar/controlvar#1737 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 90/100
Controlvar/controlvar#1447 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
-
Experiment setup: emptying a traffic share box and typing a number shows a leading zero ("034")オープンbug post-launch
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
Controlvar/controlvar#1428 ·
メンテナーはふだん 1 日以内に返信
-
bug post-launch
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
Controlvar/controlvar#1425 ·
メンテナーはふだん 1 日以内に返信
Controlvar/controlvar の issue をすべて見る
似ている issue
-
fuzz fuzz-unconfirmed
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
メンテナーはふだん 1 日以内に返信
-
ready-for-agent
難易度 2/5 1〜3時間 初心者へのやさしさ 86/100
neilsh/stunt-reel#220 ·
メンテナーはふだん 1 日以内に返信
-
[Issue]: use_fp4_cache is accepted but ignored; the launcher always writes FP8 records対応中かも @0z5a が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
メンテナーはふだん 1 日以内に返信
-
area: cli priority: p3 size: S type: bug
難易度 2/5 1〜3時間 初心者へのやさしさ 83/100
finos/open-resource-broker#757 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
メンテナーはふだん 6 日以内に返信