Entfernung von Ember String-Prototyperweiterungen

Das Problem ist, dass @ember/string veraltet ist, scheint es nach Ember 4.x? Ich kann dieses Paket in Ember 5.x nicht sehen … das wäre also nur ein Pflaster, bis Discourse schnell auf 5.x umsteigt (worauf David hier angedeutet hat: Upgrading Discourse to Ember 4 - #14 by david)

@David, was ist hier die richtige Entscheidung – natives Javascript verwenden (wie ich es getan habe)?

Danke für den Bericht – wir hatten nicht erwartet, dass die JQuery-Änderung String-Prototyperweiterungen beeinträchtigt, also untersuchen wir das gerade :eyes:

Soweit ich weiß, sind nur die Prototyperweiterungen veraltet. Etwas wie „mystring“.capitalize() wird also nicht mehr erlaubt sein.

Aber du wirst immer noch Dinge wie diese tun können:

import { capitalize } from "@ember/string";

capitalize("mystring");

Das würde ich dir also für die Komponente topic-list-previews empfehlen :ok_hand:

(Weitere Vorher/Nachher-Beispiele hier)

4 „Gefällt mir“

Keine Sorge, aber es hat mich auch überrascht :sweat_smile:

2 „Gefällt mir“

Ich habe mir all-the-themes und all-the-plugins angesehen, und es gibt nur ein paar öffentliche Plugins, die repariert werden müssen. Ich habe hier PRs erstellt: (cc @merefield @angus)

Godfrey hat für uns recherchiert, warum der JQuery-Commit dies ausgelöst hat, und seine Ergebnisse finden Sie in diesem PR. Es ist also technisch möglich, diese Funktionalität wiederherzustellen, aber es beinhaltet das Anwenden eines Patches auf eine Upstream-Abhängigkeit, was nicht ideal ist.

Angesichts der Komplexität und der Tatsache, dass wir sie ohnehin in ein paar Monaten entfernen würden, halte ich es für sinnvoll, die Entfernung jetzt zu forcieren. Ich habe das in diesem PR formalisiert:

Lassen Sie uns wissen, wenn Sie Fragen oder Bedenken haben.

4 „Gefällt mir“

Danke David. Ich stimme der Strategie zu. Nett von Ihnen, PRs bereitzustellen. :pray:

5 „Gefällt mir“