merefield
Fevereiro 23, 2023, 7:41pm
1
Consegui reproduzir isso em duas máquinas completamente diferentes e instalações de desenvolvimento.
Passos para reproduzir:
Inicie o Discourse localmente com: EMBER_ENV=production bin/ember-cli -u (embora o Ember ENV não pareça mudar as coisas aqui)
Certifique-se de que não há plugins de terceiros ou Componentes de Tema instalados.
Navegue até Admin → Personalizar → Tema
Crie um Novo Componente de Tema
Tente adicionar algum CSS
Isso resulta em uma página em branco e este erro no console:
Error: ember-computed-decorators does not support using getters and setters
committed 03:32PM - 23 Feb 23 UTC
This conversion was achieved using the ember-native-class-codemod, plus a handfu… l of manual fixes/tweaks
@David se eu fizer checkout logo antes disso, tudo fica bem…
4 curtidas
merefield
Fevereiro 23, 2023, 8:14pm
2
Acho que encontrei a linha problemática:
" --mytheme-tertiary-or-quaternary: #{dark-light-choose($tertiary, $quaternary)};\n" +
"}";
return I18n.t("admin.customize.theme.color_definitions.placeholder", {
example: isDocumentRTL() ? `<div dir="ltr">${example}</div>` : example,
});
}
return "";
}
@discourseComputed("fieldName", "currentTargetName", "theme")
get activeSection() {
return this.model.getField(this.currentTargetName, this.fieldName);
}
set activeSection(value) {
this.theme.setField(this.fieldName, value);
return value;
}
@discourseComputed("maximized")
3 curtidas
david
Fevereiro 24, 2023, 12:01am
4
Obrigado @merefield - deve ser corrigido por
committed 11:03PM - 23 Feb 23 UTC
Followup to a433b30650d125e6685fb13f679f613003f246aa
- `discourseComputed` ->… `computed` on a getter (human error)
- `this.model` -> `this.theme` (human error)
- missing `setField` target param (human error)
- `onlyOverriddenChanged` and `save` action method name clashes (native-class-codemod error)
4 curtidas
albjeremias
Fevereiro 24, 2023, 12:07am
5
alguma solução alternativa por enquanto?
david
Fevereiro 24, 2023, 12:18am
6
A correção já está ativa. Se você atualizou seu site durante o período em que ele estava com problemas, precisará executar outra atualização agora para que as coisas voltem a funcionar.
3 curtidas
albjeremias
Fevereiro 24, 2023, 12:20am
7
sim. eu fiz isso uma hora atrás..
2 curtidas
merefield
Fevereiro 24, 2023, 7:09am
8
obrigado @David … Eu estava criando um PR, mas agradeço a resposta rápida, você me poupou muito tempo: certamente teria levado mais tempo!
2 curtidas
david
Fevereiro 27, 2023, 8:00am
9
Este tópico foi fechado automaticamente após 3 dias. Novas respostas não são mais permitidas.
não sabia que funcionava assim. obrigado.