Guide sur l'implémentation de la précédence des noms d'affichage

Dev
1

Je creuse un peu pour trouver la meilleure approche (j’ai d’abord pensé : « bon, changeons ceci dans le seul fichier front qui résoudra le problème »), et je suis tombé sur le BasicUserSerializer (« app/serializers/basic_user_serializer.rb »).

Alors, que diriez-vous d’ajouter une méthode display_name dans le modèle User, qui retournerait la bonne valeur en fonction des paramètres du site, et de l’utiliser dans tous les sérialiseurs retournant des utilisateurs (ou des parties), puis dans tous les composants frontaux ?

C’est un changement plus important, c’est certain, et qui pourrait avoir un impact sur les performances, mais cela semble être un bon point de départ pour unifier facilement la valeur sans effectuer de vérification dans chaque fichier Ember.

5 « J'aime »
2

Salut @manut,

Merci d’avoir examiné cela et d’avoir réfléchi à une solution plus large !

Cependant, pour cette pull request (PR), je recommanderais de simplement se concentrer sur le changement du fichier frontal et donc de corriger le changement pour la liste des participants au calendrier.

L’approche plus large mérite certainement d’être envisagée. Mais, nous devrons évaluer séparément l’approche à utiliser et ses implications, car il s’agit d’une discussion architecturale plus vaste qui dépasse le cadre de cette correction.

Merci d’être disposé à vous en occuper ! J’attends avec impatience votre PR !

5 « J'aime »