J’ai réussi à reproduire cela sur deux machines et installations de développement entièrement différentes.
Étapes pour reproduire :
Lancez Discourse localement avec : EMBER_ENV=production bin/ember-cli -u (bien que Ember ENV ne semble pas changer les choses ici)
Assurez-vous qu’aucun plugin tiers ou composant de thème n’est installé.
Accédez à Admin → Personnaliser → Thème
Créez un nouveau composant de thème
Essayez d’ajouter du CSS
Cela entraîne une page blanche et cette erreur dans la 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 si je fais un checkout juste avant cela, tout va bien…
4 « J'aime »
Je pense avoir trouvé la ligne fautive :
" --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 « J'aime »
david
Février 24, 2023, 12:01
4
Merci @merefield - devrait être corrigé par
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 « J'aime »
albjeremias
Février 24, 2023, 12:07
5
une solution de contournement pour l’instant ?
david
Février 24, 2023, 12:18
6
La correction est maintenant en ligne. Si vous avez mis à jour votre site pendant les heures où il était défaillant, vous devrez exécuter une autre mise à jour maintenant pour que les choses fonctionnent à nouveau.
3 « J'aime »
albjeremias
Février 24, 2023, 12:20
7
Oui. Je l’ai fait il y a une heure… :soulagé: Je ne savais pas que ça fonctionnait comme ça. Merci.
2 « J'aime »
Merci @David … J’étais en train de créer une PR, mais j’apprécie la réponse rapide, vous m’avez fait gagner beaucoup de temps : cela m’aurait certainement pris plus de temps !
2 « J'aime »
david
Février 27, 2023, 8:00
9
Ce sujet a été automatiquement fermé après 3 jours. Les nouvelles réponses ne sont plus autorisées.
