Используйте имена пользователей в списке участников

Привет!

Мы заметили, что список участников мероприятия основан на имени пользователя. Мы считаем, что было бы неплохо использовать имя пользователя (а имя пользователя использовать как запасной вариант).

Вам интересна эта функция? Если да, я бы модифицировал компонент User (plugins/discourse-calendar/assets/javascripts/discourse/components/modal/post-event-invitees/user.gjs.

Если вы считаете, что это плохая идея, можем ли мы предложить настройку сайта для переключения этой функции?

3 лайка

Это хорошая идея. Возможно, отображение имени пользователя или полного имени можно привязать к настройке сайта «Включить имена»? Что скажете, @j.jaffeux?

4 лайка

Честно говоря, я удивлён, что этого ещё нет. На мой взгляд, эта настройка должна покрывать большинство, если не все, случаи появления имени пользователя.

3 лайка

Есть ещё несколько пробелов в приоритете полного имени в уведомлениях и в всплывающей подсказке в плагине «Кто онлайн». Стоит ли сообщать об этом?

3 лайка

Посмотрим, к чему приведёт эта тема, когда подключится наш разработчик; возможно, всё будет решено сразу. Может быть, также стоит спросить мнение @lindsey?

3 лайка

По моему мнению, мы должны привязать все отображения имени пользователя/имени к настройкам сайта «Включить имена» и «Приоритет имени пользователя в UX».

4 лайка

Есть и другие области, где мы могли бы быть более последовательными в настройках enable names и prioritize username in ux, но, на мой взгляд, нам не нужно ждать полного списка, чтобы начать их исправлять.

@j.jaffeux, не могли бы вы рассмотреть это как pr-welcome, чтобы список участников событий отображал полное имя, когда enable names равен «true», а prioritize username in ux — «false»? Это сделало бы поведение более согласованным с тем, как мы обрабатываем имена и имена пользователей на карточках пользователей, профилях и т. д.

6 лайков

Я займусь этим, как только оно будет принято, а затем и остальными местами, если у меня будет список :wink:

4 лайка

Я вижу :+1: от Жоффри в ответ на ваш вопрос, @lindsey, поэтому я добавил pr-welcome сюда.

3 лайка

Пост был разделён на новую тему: Рекомендации по реализации приоритета отображаемого имени

Привет!

Продолжение: я открыл pull request :slight_smile:

4 лайка

Привет @manut

Это уже включено, большое спасибо!

1 лайк

Отлично!

Не стесняйтесь писать мне, если есть другие места, где можно внести изменения.

2 лайка

Я нашёл ещё два места:

Уведомления об упоминаниях используют только имя пользователя, даже если настройка «Предпочитать имя пользователя» отключена или включены отображения имён.

Кроме того, всплывающая подсказка «Кто онлайн» для отображаемых аватаров также показывает имя пользователя, даже если при наведении на аватары в других местах (например, в списке последних тем) отображается имя.

2 лайка

Отлично, мне стоит создать новые темы для этого?

1 лайк

Нет необходимости открывать новую тему только для этого :wink:

Если вы хотите сделать PR(ы), напишите мне, и я их проверю. В противном случае я могу сделать это сам.

3 лайка

Спасибо!

Я займусь этим: мне действительно нужно углубить свои знания о внутреннем устройстве Discourse, так как я работаю над плагином для кооператива и хочу, чтобы он был реализован с использованием лучших практик. Мне ещё многому предстоит научиться :slight_smile:

4 лайка

@zogstrip Я начал работу над этим; я собираюсь создать тип уведомления mentionedfrontend/discourse/app/lib/notification-types/).

Я использую другие типы уведомлений в качестве примера и заметил использование this.siteSettings.prioritize_full_name_in_ux (обратите внимание, это не prioritize_username_in_ux).
Её значение, похоже, всегда false (например, в уведомлении «лайк» используется имя пользователя вместо полного имени).

Небольшой поиск по prioritize_full_name_in_ux = показывает, что значение ей присваивается только в тестах, поэтому она сохраняет своё значение по умолчанию, определённое в config/site_settings.yml:1005 (false).

Я что-то упускаю или здесь есть более серьёзная проблема?

3 лайка

Возможно, это было упущено при переименовании? Я разберусь :eyes:

3 лайка

Привет!

Не хочу давить на вас, но у вас уже было время посмотреть в проблему?
Дайте знать, если я могу помочь :slight_smile:

1 лайк