Anleitung zur Implementierung der Anzeige-Namen-Priorität

Entwickler
1

Ich grabe etwas tiefer nach dem besten Ansatz (zuerst dachte ich: „Okay, ändern wir das in der einen Frontend-Datei, wo es das löst“), und bin dabei auf den BasicUserSerializer („app/serializers/basic_user_serializer.rb“) gestoßen.

Was hältst du also davon, eine display_name-Methode im User-Modell hinzuzufügen, die den richtigen Wert basierend auf den Site-Einstellungen zurückgibt, und diese dann in allen Serializern zu verwenden, die Benutzer (oder Teile davon) zurückgeben, und schließlich in allen Frontend-Komponenten?

Das ist sicherlich eine größere Änderung und könnte Auswirkungen auf die Leistung haben, scheint aber ein guter Ausgangspunkt zu sein, um den Wert einfach zu vereinheitlichen, ohne in jeder Ember-Datei eine Prüfung durchzuführen.

2

Hallo @manut,

Vielen Dank, dass du dich damit beschäftigt und über eine umfassendere Lösung nachgedacht hast!

Für diesen PR würde ich jedoch empfehlen, sich einfach auf die Änderung in der Front-Datei zu konzentrieren und somit die Änderung für die Teilnehmerliste des Kalenders zu beheben.

Der umfassendere Ansatz ist definitiv etwas, das es wert ist, in Betracht gezogen zu werden. Aber wir müssen den zu verwendenden Ansatz und seine Auswirkungen separat bewerten, da dies eine größere architektonische Diskussion ist, die über den Rahmen dieser Korrektur hinausgeht.

Danke, dass du bereit bist, dich darum zu kümmern! Ich freue mich auf deinen PR!