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

Abierto Apto para principiantes
  • ¿Algo no está bien?
#1,562 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
70/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
javascript
Área
frontend

Línea de trabajo

El mensaje fijo de 422 para los guardados de diseño está en la ruta de guardado del editor de diseño del dashboard, y el issue no nombra el archivo. Encuentra dónde se establece ese mensaje y luego bifurca según si el tipo del primer elemento de error es split_url_config en la ubicación ["body","variants"]. Está terminado cuando ese caso muestra el nuevo texto y una prueba de jest lo cubre, mientras que un 422 genérico sigue mostrando el texto anterior.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

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.

Lenguaje dominante
Python
Estrellas
0
Forks
4
Merge medio
5 h 36 min
PR fusionados (30 d)
509

Preparar el entorno

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de Controlvar/controlvar

Todos los issues de Controlvar/controlvar

Issues similares

Más issues de Python

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.