Alerta de tema roto sugiere deshabilitar en lugar de actualizar

Una pequeña, pero que causó bastante confusión.

Cuando se detecta un componente de tema roto, aparece una barra roja que dice “Es posible que tu sitio no funcione porque el tema / componente %{theme} tiene errores. Desactívalo en %{path}”.

Sin embargo, dado que (en este caso) Discourse también detectó que este componente de tema no estaba en su último commit, la acción correcta aquí fue “actualizar” en lugar de “desactivar”.

En mi experiencia, ese es el caso para la mayoría de las situaciones en las que aparece este mensaje. La solución rápida sería actualizar el texto a “Actualizar o desactivar”, una solución más extensa sería variar el mensaje dependiendo de si hay una actualización disponible para el componente de tema (actualizar) o no (desactivar).

3 Me gusta

Animar a la gente a hacer clic en el botón de actualización automática en las actualizaciones también es una buena idea.

3 Me gusta

Hemos mejorado este mensaje en la última versión de tests-passed para que se vea algo así:

Tiene sentido modificarlo a “reconfigurar, actualizar o desactivar” :+1:

Ahora es la opción predeterminada, pero no hemos cambiado los temas existentes (porque históricamente la gente podría haberles hecho modificaciones locales y no querríamos sobrescribir automáticamente esos cambios).

4 Me gusta

Este tema se cerró automáticamente después de 3 días. Ya no se permiten nuevas respuestas.