Admin.groups.manage.membership.automatic 未被替换

用户体验
1

The image shows a screenshot of a computer or mobile device displaying a webpage, likely a social media platform. (Captioned by AI)
The image shows a screenshot of a computer or mobile device displaying a webpage, likely a social media platform. (Captioned by AI)1774×433 115 KB

这应该对所有人可见吗？

2 个赞
2

我在管理员群组页面上也看到了它。

image
image1170×1319 82.3 KB

3

我认为它出现在所有自动分组页面上。我选择了“版主”作为截图的示例，但工作人员、管理员和信任级别组也受到影响。

1 个赞
4

在移动设备上，如果您可以给群组发送消息，则损坏的文本会移动消息按钮。

_20240920_141814
_20240920_141814720×522 43.7 KB

1 个赞
5

注意… 在这里添加了一个优先级标签，我们将在未来几周内进行查看。

1 个赞
6

我认为修复起来相当容易。

目前，工具提示对所有人可见，但它为管理员部分使用了一个字符串：

image
image1078×160 9.36 KB

我认为这些信息仅与可以添加组成员的人相关。这可能包括管理员、组所有者以及（取决于 moderators manage categories and groups 设置）版主。我相信自动组不能有所有者，因此任何被允许管理自动组的人也应该能够访问管理员部分的字符串，而其他人则可以隐藏它。

+       {{#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 个赞
7

随时欢迎你提交 PR，@Moin :wink:

8

我无法编写测试 :frowning:

9

没关系，我拿了 :wink:

1 个赞
11
12

谢谢，一旦合并就会修复

13

“它缺少一个检查以确保当前用户是管理员，因为只有管理员才能管理自动组。”

有意思。我以为版主也可以，如果启用了“版主管理类别和组”。“管理”选项卡对他们来说是可见的。

Screenshot_20241206_151909_Firefox
Screenshot_20241206_151909_Firefox1737×1079 171 KB

这就是为什么我认为解释他们为什么不能像在其他组那样添加成员是有帮助的。

1 个赞
14

我认为自动组的处理方式有所不同，我在代码中看到了一些需要管理员权限的检查，不过那可能是遗留的……

@hugh / @martin 你们知道谁可以管理自动组吗？

15

此主题已在 16 小时后自动关闭。不再允许回复。

16

我认为 Moin 提到的站点设置允许这样做，这是相关的 guardian 检查：

但在客户端，我们发送了相互矛盾的消息 :sweat_smile:

这是一个后续的 PR，用于更改检查 canManageGroup，并更新 user model 版本中的 canManageGroup 以使其更合理（唯一使用该版本的地方来自此 group controller）：

5 个赞