Nous avons remarqué que la liste des participants à un événement est basée sur le nom d’utilisateur. Nous pensons qu’il serait plus agréable d’utiliser le nom de l’utilisateur à la place (et le nom d’utilisateur comme solution de secours).
Êtes-vous intéressé par cette fonctionnalité ? Si oui, je modifierais le composant User (plugins/discourse-calendar/assets/javascripts/discourse/components/modal/post-event-invitees/user.gjs).
Si vous pensez que c’est une mauvaise idée, pouvons-nous proposer un paramètre de site pour activer/désactiver cette fonctionnalité ?
C’est une bonne idée. Peut-être que l’affichage du nom d’utilisateur ou du nom complet pourrait être lié au paramètre du site Activer les noms ? Qu’en pensez-vous, @j.jaffeux ?
Je suis surpris que ce ne soit pas déjà le cas, pour être honnête. À mon avis, ce paramètre devrait couvrir la plupart, sinon toutes, les apparitions d’un nom d’utilisateur.
Il y a quelques autres lacunes dans la priorité du nom (complet), dans les notifications et dans le titre de survol du plugin Qui est en ligne. Faut-il les signaler ?
Voyons où ce sujet nous mène une fois que notre développeur aura donné son avis ; tout pourrait être couvert en une seule fois. Peut-être aussi @lindsey pour avoir son opinion ?
IMHO, nous devrions lier l’affichage de tous les noms d’utilisateur/noms aux paramètres du site enable names (activer les noms) et prioritize username in ux (prioriser le nom d’utilisateur dans l’UX).
Il existe d’autres domaines où nous pourrions être plus cohérents avec les paramètres enable names et prioritize username in ux, mais je ne pense pas que nous devions attendre d’avoir une liste complète pour commencer à les corriger.
@j.jaffeux, considéreriez-vous ceci comme un pr-welcome pour garantir que la liste des participants aux événements affiche le nom complet lorsque enable names est « true » et que prioritize username in ux est « false » ? Cela le rendrait plus cohérent avec la façon dont nous gérons les noms et les noms d’utilisateur sur les cartes d’utilisateur, les profils, etc.
Les notifications de mentions n’utilisent que le nom d’utilisateur même lorsque la préférence du nom d’utilisateur est désactivée/les noms sont activés.
Et l’info-bulle du titre “Qui est en ligne” sur les avatars listés est également le nom d’utilisateur, même lorsque l’info-bulle sur les avatars ailleurs (comme la liste des derniers sujets) est le nom.
Je vais le faire : j’ai vraiment besoin d’améliorer mes connaissances sur le fonctionnement interne de Discourse car je travaille sur un plugin pour ou coopérative et je veux qu’il soit implémenté avec les meilleures pratiques. J’ai encore beaucoup à apprendre
J’ai commencé à travailler dessus ; je vais créer un type de notification mentioned (dans frontend/discourse/app/lib/notification-types/).
J’utilise les autres types de notification comme exemple, et j’ai remarqué l’utilisation de this.siteSettings.prioritize_full_name_in_ux (notez que ce n’est pas prioritize_username_in_ux.
Sa valeur semble toujours être false (par exemple, pour une notification “like”, le nom d’utilisateur est utilisé au lieu du nom complet).
Un petit grep pour prioritize_full_name_in_ux = me montre qu’il n’est assigné une valeur que dans les tests, il conserve donc sa valeur par défaut définie dans config/site_settings.yml:1005 (false).
Est-ce que je manque quelque chose ou y a-t-il un problème plus important ici ?