-
Notifications
You must be signed in to change notification settings - Fork 220
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
feat: Add activityEvent for ReadOnly user access actions #7615
feat: Add activityEvent for ReadOnly user access actions #7615
Conversation
@@ -1009,6 +1009,8 @@ | |||
"ADMIN_USERS_DEACTIVATE": "Deactivate user", | |||
"ADMIN_USERS_GIVE_ADMIN": "Give admin access", | |||
"ADMIN_USERS_REMOVE_ADMIN": "Remove admin access", | |||
"ADMIN_USERS_GIVE_READ_ONLY": "Give read only access", | |||
"ADMIN_USERS_REMOVE_READ_ONLY": "Remove read only access", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
今更って感じは否めないが、give/remove って変な感じがする。。
再考をお願いしたい。API エンドポイントもそうなってると思うので、最終的には変えたい。後続タスクでOK。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@@ -1017,6 +1017,8 @@ | |||
"ADMIN_USERS_DEACTIVATE": "ユーザーを停止する", | |||
"ADMIN_USERS_GIVE_ADMIN": "管理者にする", | |||
"ADMIN_USERS_REMOVE_ADMIN": "管理者から外す", | |||
"ADMIN_USERS_GIVE_READ_ONLY": "閲覧ユーザーにする", | |||
"ADMIN_USERS_REMOVE_READ_ONLY": "閲覧ユーザーから外す", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
英語「Read Only」日本語「閲覧のみ」にしてアクセス権としました。
https://wsgrowi.slack.com/archives/C0111HG81GB/p1683537245035329
task: https://redmine.weseek.co.jp/issues/121247
Screenshot