Suppression de ember-addons/ember-computed-decorators

@markvanlan Le déplacement des décorateurs de ember-addons/ember-computed-decorators vers discourse-common/utils/decorators est sur le point de faire planter un certain nombre de plugins et de thèmes dès que les utilisateurs cliqueront sur le gros bouton bleu de mise à jour.

Serait-il possible de conserver les décorateurs existants (en plus des nouveaux) pour nous laisser le temps de faire la transition ?

Cela ne devrait pas constituer une rupture de compatibilité. Ici, dans le fichier discourse-loader, nous avons ajouté un alias afin que chaque fois que vous essayez d’importer ember-addons/ember-computed-decorators, cela charge automatiquement discourse-common/utils/decorator à la place.

Ouais, je l’ai vu. Je ne sais pas trop pourquoi, mais cela ne semble pas fonctionner dans tous les cas. Je rencontre beaucoup d’erreurs RequireJS qui semblent se résoudre une fois que je passe à la nouvelle importation.

Pouvez-vous vérifier si cela se produit lorsque vous utilisez require au lieu de import ?

C’était un bug que je pensais avoir corrigé.

Bonjour,

Voici le message d’erreur que j’ai rencontré sur mon site Discourse :

Screen Shot 2020-02-19 at 2.50.56 PM

Cela doit-il toujours être mis à jour dans la base de code ?

Je ne vois pas exactement d’où vient cette erreur (quel plugin utilise ember-addons/ember-compted-decorators), mais cela devra éventuellement être corrigé.

Il s’agit d’un avertissement plutôt que d’une erreur, car ember-addons/ember-compted-decorators reste fonctionnel pour le moment.

Merci @markvanlan,

Quelle méthode utilisez-vous ou recommandez-vous pour identifier un plugin qui utilise des assets obsolètes ?

Je vérifierais quels plugins sont installés sur votre instance, puis j’irais sur chaque dépôt GitHub pour rechercher ember-addons/ember-computed-decorators.

Je pense que certains de nos plugins Discourse officiels ont encore besoin de cette mise à jour, donc cela pourrait provenir de l’un de nos dépôts.

Pouvez-vous l’élargir et afficher la pile d’appels complète ?

Je ne sais pas comment voir la trace d’appel derrière un avertissement de la console du navigateur ?

Je sais par contre comment chercher dans les plugins :smile:

$ grep -r ember-addons/ember-computed-decorators plugins/|grep import|awk -F / '{print $2}'|uniq|sort
discourse-canned-replies
discourse-local-dates
discourse-presence
discourse-push-notifications
discourse-saved-searches
procourse-memberships

Bonjour, je ne parviens pas à reproduire le problème pour le moment. Cependant, le phénoménal @RGJ vient de mettre à jour les plugins sur notre site et je pense que cela a résolu l’erreur ? Il m’a indiqué ces mises à jour :

DEV: Find/replace deprecated decorators · discourse/discourse-data-explorer@277346b · GitHub

Belle recherche Richard ! :slight_smile:

Je ne pense pas qu’il y ait lieu de paniquer.

C’est aussi aux créateurs de plugins de réduire ces avertissements dès que possible.

J’en ai déjà résolu pas mal récemment, et il y en aura d’autres. C’est une histoire sans fin :wink:

Ce ne sont que des avertissements, rien de vraiment inquiétant à moins qu’ils soient ignorés trop longtemps.

Merci. J’adore voir que Discourse continue de s’améliorer et de maintenir son code !