Admin.groups.manage.membership.automatic не заменен

Должно ли это быть видно всем?

2 лайка

Я вижу это также на странице группы Администраторы.

Похоже, это происходит на всех автоматических страницах групп. В качестве примера для скриншота я выбрал «модераторов», но также затрагиваются сотрудники, администраторы и группы уровней доверия.

1 лайк

На мобильных устройствах, если вы можете отправлять сообщения в группу, битый текст сдвигает кнопку отправки сообщения.

1 лайк

Примечание… Добавлен тег приоритета, поэтому мы рассмотрим это в ближайшие недели.

1 лайк

Я думаю, что это довольно легко исправить.

В данный момент всплывающая подсказка видна всем, но в ней используется строка для раздела администратора:


По-моему, эта информация актуальна только для тех, кто может добавлять участников группы. К ним могут относиться администраторы, владельцы групп и, в зависимости от настройки «Модераторы управляют категориями и группами», модераторы. Я полагаю, что автоматические группы не могут иметь владельцев, поэтому любой, кому разрешено управлять автоматической группой, должен иметь доступ к строке раздела администратора, а для остальных её можно скрыть.

+       {{#if this.canManageGroup}}
          {{#if this.model.automatic}}
            <DTooltip class="admin-group-automatic-tooltip">
              <:trigger>
                {{d-icon "gear"}}
                {{i18n "admin.groups.manage.membership.automatic"}}
              </:trigger>
              <:content>
                {{i18n "admin.groups.manage.membership.automatic_tooltip"}}
              </:content>
            </DTooltip>
          {{/if}}
+       {{/if}}
4 лайка

Не стесняйся делать PR, @Moin :wink:

Я не могу написать тесты :frowning:

Без проблем, я тогда заберу его :wink:

1 лайк

Спасибо, это будет исправлено после слияния

Отсутствовала проверка на то, является ли текущий пользователь администратором, поскольку управлять автоматическими группами могут только администраторы.

Интересно. Я думал, что модераторы тоже могут это делать, если включена опция «Модераторы управляют категориями и группами». Для них вкладка «Управление» видна.

Именно поэтому я считал полезным объяснить, почему они не могут добавлять участников, как это возможно в других группах.

1 лайк

Автоматические группы, кажется, обрабатываются иначе. Я видел в коде некоторые проверки, требующие прав администратора, хотя это могут быть и остатки…

@hugh / @martin, не подскажете ли вы, кто может управлять автоматическими группами?

Эта тема была автоматически закрыта через 16 часов. Новые ответы больше не допускаются.

Я думаю, что настройка сайта, упомянутая Моэном, действительно это позволяет. Вот соответствующая проверка Guardian:

Однако на стороне клиента мы отправляем противоречивые сообщения :sweat_smile:

Вот последующий PR для изменения проверки на canManageGroup, а также для обновления версии canManageGroup в модели пользователя, чтобы она стала более логичной (единственное место, где она используется, — это контроллер группы):

5 лайков