Alerta de tema quebrado sugere desabilitar em vez de atualizar

Um pequeno, mas que causou bastante confusão.

Quando um componente de tema quebrado é detectado, uma barra vermelha aparece afirmando “Seu site pode não funcionar porque o tema / componente %{theme} tem erros. Desative-o em %{path}.”

No entanto, como (neste caso) o Discourse também detectou que este componente de tema não estava em seu último commit, a ação correta aqui foi “atualizar” em vez de “desativar”.

Na minha experiência, esse é o caso na maioria das situações em que esta mensagem aparece. A correção rápida seria atualizar o texto para “Atualizar ou desativar”, uma correção mais extensa seria variar a mensagem dependendo se há uma atualização disponível para o componente de tema (atualizar) ou não (desativar).

3 curtidas

Incentivar as pessoas a clicar no botão de atualização automática em atualizações também é uma boa ideia.

3 curtidas

Melhoramos esta mensagem na última versão do tests-passed para que fique algo como:

Faz sentido alterar para ‘reconfigurar, atualizar ou desativar’ :+1:

Agora é o padrão, mas não alteramos temas existentes (porque historicamente as pessoas podem ter feito modificações locais neles, e não gostaríamos de sobrescrever automaticamente essas alterações)

4 curtidas

Este tópico foi fechado automaticamente após 3 dias. Novas respostas não são mais permitidas.