Entfernung von ember-addons/ember-computed-decorators

@markvanlan Die Verlagerung der Decoratoren von ember-addons/ember-computed-decorators nach discourse-common/utils/decorators wird voraussichtlich viele Plugins und Themes brechen, sobald die Nutzer auf den großen blauen Update-Button klicken.

Wäre es möglich, die bestehenden Decoratoren vorerst beizubehalten (zusätzlich zu den neuen), um uns Zeit für die Umstellung zu geben?

Dies sollte kein Breaking Change sein. Hier in der Datei discourse-loader haben wir einen Alias hinzugefügt, sodass beim Importieren von ember-addons/ember-computed-decorators automatisch discourse-common/utils/decorator geladen wird.

Ja, das habe ich gesehen. Ich bin mir nicht sicher warum, aber das scheint nicht in allen Fällen zu funktionieren. Ich bekomme viele RequireJS-Fehler, die sich zu lösen scheinen, sobald ich auf den neuen Import umsteige.

Können Sie prüfen, ob das Problem auftritt, wenn Sie require anstelle von import verwenden?

Das war ein Fehler, von dem ich dachte, ich hätte ihn behoben.

Hallo,

hier ist die Fehlermeldung, die auf meiner Discourse-Seite aufgetreten ist:

Screen Shot 2020-02-19 at 2.50.56 PM

Muss dies noch im Code aktualisiert werden?

Ich kann nicht genau erkennen, woher dieser Fehler stammt (welches Plugin ember-addons/ember-compted-decorators verwendet), aber er muss letztendlich behoben werden.

Das ist eine Warnung und kein Fehler, da ember-addons/ember-compted-decorators vorerst noch funktionsfähig ist.

Danke @markvanlan,

Welche Methode verwendest du oder empfiehlst du, um ein Plugin zu identifizieren, das veraltete Assets nutzt?

Ich würde prüfen, welche Plugins auf deiner Instanz installiert sind, und dann in jedem Repository auf GitHub nach ember-addons/ember-computed-decorators suchen.

Ich glaube, einige unserer offiziellen Discourse-Plugins benötigen dieses Update noch, sodass es sich um eines unserer Repos handeln könnte.

Kannst du das erweitern und den vollständigen Backtrace anzeigen?

Ich weiß nicht, wie man den Backtrace hinter einer Warnung in der Browserkonsole anzeigen kann?

Aber ich weiß, wie man Plugins mit grep durchsucht :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

Hallo, ich kann es derzeit nicht reproduzieren. Aber der großartige @RGJ hat gerade die Plugins auf unserer Website aktualisiert, und ich denke, damit wurde der Fehler behoben? Er hat mich auf diese Updates verwiesen:

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

Schönes Grep, Richard! :slight_smile:

Ich glaube nicht, dass dies Grund zur Panik ist.

Es liegt auch in der Verantwortung der Plugin-Entwickler, diese Warnungen so schnell wie möglich zu minimieren.

Ich habe kürzlich einige davon behoben, und es werden noch mehr folgen. Das ist eine unendliche Geschichte :wink:

Es sind nur Warnungen, und man muss sich wirklich keine Sorgen machen, es sei denn, sie werden zu lange ignoriert.

Danke. Ich finde es großartig, dass Discourse seinen Code weiter verbessert und wartet!