dashboard: a split URL draft's variant edit shows generic copy when the API refuses it

未关闭 适合新手
#1,562 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

维护者通常 1 天内回复

还没有人认领这个 Issue。

评估

难度
2/5
预计耗时
1-3 小时
新手友好度
70/100
Issue 类型
缺陷
描述清晰度
描述清楚
活跃度
活跃
技术栈
javascript
领域
frontend

调研方向

设计保存时固定的 422 提示位于仪表盘设计编辑器的保存流程中,issue 没有指明文件。请找到设置该提示的位置,然后根据第一个错误项的类型是否为 split_url_config(位置为 ["body","variants"])进行分支。当该情况显示新文案且有一个 jest 测试覆盖它,而通用的 422 仍显示旧文案时,即完成。

由索引模型根据 Issue 内容生成。

描述

bug post-launch

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 分钟
30 天内合并 PR
509

环境准备

从这里开始

  1. 先读完整个 Issue,再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库,在一个分支上完成修改。
  4. 提交 Pull Request,并在描述里引用这个 Issue 编号。

Controlvar/controlvar 的其他 Issue

查看 Controlvar/controlvar 的全部 Issue

相似的 Issue

更多 Python Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。