Должно ли это быть видно всем?
Похоже, это происходит на всех автоматических страницах групп. В качестве примера для скриншота я выбрал «модераторов», но также затрагиваются сотрудники, администраторы и группы уровней доверия.
На мобильных устройствах, если вы можете отправлять сообщения в группу, битый текст сдвигает кнопку отправки сообщения.
Примечание… Добавлен тег приоритета, поэтому мы рассмотрим это в ближайшие недели.
Я думаю, что это довольно легко исправить.
В данный момент всплывающая подсказка видна всем, но в ней используется строка для раздела администратора:
По-моему, эта информация актуальна только для тех, кто может добавлять участников группы. К ним могут относиться администраторы, владельцы групп и, в зависимости от настройки «Модераторы управляют категориями и группами», модераторы. Я полагаю, что автоматические группы не могут иметь владельцев, поэтому любой, кому разрешено управлять автоматической группой, должен иметь доступ к строке раздела администратора, а для остальных её можно скрыть.
+ {{#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}}
Не стесняйся делать PR, @Moin ![]()
Я не могу написать тесты ![]()
Без проблем, я тогда заберу его ![]()
Спасибо, это будет исправлено после слияния
Отсутствовала проверка на то, является ли текущий пользователь администратором, поскольку управлять автоматическими группами могут только администраторы.
Интересно. Я думал, что модераторы тоже могут это делать, если включена опция «Модераторы управляют категориями и группами». Для них вкладка «Управление» видна.
Именно поэтому я считал полезным объяснить, почему они не могут добавлять участников, как это возможно в других группах.
Автоматические группы, кажется, обрабатываются иначе. Я видел в коде некоторые проверки, требующие прав администратора, хотя это могут быть и остатки…
@hugh / @martin, не подскажете ли вы, кто может управлять автоматическими группами?
Эта тема была автоматически закрыта через 16 часов. Новые ответы больше не допускаются.
Я думаю, что настройка сайта, упомянутая Моэном, действительно это позволяет. Вот соответствующая проверка Guardian:
Однако на стороне клиента мы отправляем противоречивые сообщения ![]()
Вот последующий PR для изменения проверки на canManageGroup, а также для обновления версии canManageGroup в модели пользователя, чтобы она стала более логичной (единственное место, где она используется, — это контроллер группы):




