Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Halo 2.10 版本中关联S3文件的全选复选框消失 #82

Closed
longjuan opened this issue Oct 7, 2023 · 5 comments · Fixed by #83
Closed

Halo 2.10 版本中关联S3文件的全选复选框消失 #82

longjuan opened this issue Oct 7, 2023 · 5 comments · Fixed by #83

Comments

@longjuan
Copy link
Member

longjuan commented Oct 7, 2023

复选框消失:
image
正常状态:
image

经排查,与 halo-dev/halo#4598 有关。

@longjuan
Copy link
Member Author

longjuan commented Oct 7, 2023

Hi,@ruibaby

<Transition v-else appear name="fade">
<ul
class="box-border h-full w-full divide-y divide-gray-100"
role="list"
>
<li v-for="(file, index) in s3Objects.objects" :key="index">
<VEntity :is-selected="checkSelection(file)">
<template
#checkbox
>
<input
v-model="selectedFiles"
:value="file.key || ''"
class="h-4 w-4 rounded border-gray-300 text-indigo-600"
name="post-checkbox"
:disabled="file.isLinked"
type="checkbox"
/>
</template>
<template #start>

有空帮忙看下,没看出哪里的问题,在 halo-dev/halo#4598 之后就有问题。

@ruibaby
Copy link
Member

ruibaby commented Oct 7, 2023

应该是和这行有关:

v-permission="['system:users:aaa']"

system:users:aaa 似乎没有在任何的 role template 中定义,如果这里不需要 UI 权限判断,移除这个指令即可。

halo-dev/halo#4598 之后才暴露这个问题是因为以前超管角色的 uiPermissions 是 *(代表拥有所有权限),但这个 PR 可能因为 PAT 的特性需要,会将角色拥有的所有角色模板的 ui permissions 列出提供给前端用于判断。

这个改动是必要的,已经暴露出了一些遗留问题,比如 halo-sigs/plugin-photos#24 (comment)

@longjuan
Copy link
Member Author

longjuan commented Oct 7, 2023

应该是和这行有关:

v-permission="['system:users:aaa']"

system:users:aaa 似乎没有在任何的 role template 中定义,如果这里不需要 UI 权限判断,移除这个指令即可。

halo-dev/halo#4598 之后才暴露这个问题是因为以前超管角色的 uiPermissions 是 *(代表拥有所有权限),但这个 PR 可能因为 PAT 的特性需要,会将角色拥有的所有角色模板的 ui permissions 列出提供给前端用于判断。

这个改动是必要的,已经暴露出了一些遗留问题,比如 halo-sigs/plugin-photos#24 (comment)

感谢,确实是这个问题,当时抄代码的时候随手改了一个没注意。
之前考虑加上权限模板,但是页面移到插件详情页以后,没有插件管理权限就无法进入相应页面(插件查看权限也不行),有空了我提个issuse。

@ruibaby
Copy link
Member

ruibaby commented Oct 7, 2023

但是页面移到插件详情页以后,没有插件管理权限就无法进入相应页面(插件查看权限也不行)

现在有查看权限应该可以,应该是 2.9 之后 Console 部分的插件扩展加载就不需要插件查看权限了,因为已经把 Console 部分的扩展合并成了 bundle.js,所有登录用户都可以访问。

但这个关联功能既然放在了插件详情页面,是不是可以考虑直接依赖插件的查看权限?

cc @JohnNiang

如果考虑到这个关联功能要开放给没有插件查看权限的人使用的话,可能放在插件详情的选项卡就不是很合适了。

@JohnNiang
Copy link
Member

IMO,关联 S3 文件的功能入口如果能放置在下图中红色框处,会不会更好呢?

image

如果当前用户有附件管理权限,则显示“关联 S3 文件”按钮,并进入对应的页面。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants