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

Improvement: ページを削除できるユーザーを限定したい #714

Closed
ghost opened this issue Nov 26, 2018 · 4 comments
Closed
Assignees

Comments

@ghost
Copy link

ghost commented Nov 26, 2018

Environment

Host

item version
OS Ubuntu 16.04
GROWI 3.2.8
node.js 8.12.0
npm 6.4.1
Using Docker yes
Using growi-docker-compose yes

(Accessing https://{GROWI_HOST}/admin helps you to fill in above versions)

Client

item version
OS Windows 7
browser Chrome 70.0.3538.110

What is the expected result? (期待される動作)

  • ページの削除を管理者およびページを作成したユーザーのみが行える

Note

現状、ページの削除は登録ユーザーであれば任意に行うことができると思いますが、これを管理者およびページを作成したユーザーのみに限定するオプションがあるとよいと感じたため、リクエストさせていただきました。ご検討よろしくお願いいたします。

@yuki-takei yuki-takei changed the title ページを削除できるユーザーを限定したい Improvement: ページを削除できるユーザーを限定したい Nov 29, 2018
@yuki-takei
Copy link
Member

@k84rc ご提案ありがとうございます。

確かに、Wiki への参加が自由に行える状態で誰でも完全削除できるのは権限として強すぎるかもしれませんね。

  • ページの完全削除を行えるユーザーを以下から選択
    • 誰でも可能 (default)
    • 管理者とページの作者のみ可能
    • 管理者のみ可能

というオプションを管理画面に付けようかなと思います。
こちらで要求満たせそうでしょうか?

@yuki-takei
Copy link
Member

因みに実装時期ですが、現在 Access Control List 周りの大幅なリファクタを imprv/refactor-acl ブランチで行っている最中なので、こちらの着手はそれが終わってからになると思います。すみません。

@ghost
Copy link
Author

ghost commented Nov 30, 2018

@yuki-takei

* ページの完全削除を行えるユーザーを以下から選択
  
  * 誰でも可能 (default)
  * 管理者とページの作者のみ可能
  * 管理者のみ可能

こちらで問題ありません。よろしくお願いいたします。

@yuki-takei
Copy link
Member

fixed by #1020, #1075

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

No branches or pull requests

2 participants