search

OI-wiki / feedback-sys

OI Wiki 划词评论反馈系统
https://oi-wiki.org/#comment-16
Apache License 2.0
11 stars 3 forks source link

Feat: delete & edit comment for administrators #55

Closed shaokeyibb closed 2 months ago

shaokeyibb commented 2 months ago

合并前操作：

  1. 设置 GH_ORG_ADMINISTRATOR_TEAM secret，格式为 {ORG_NAME}/{TEAM_SLUG}
  2. 为上述组织安装 GitHub App

此 PR 的其他更改

  1. 正确处理当用户在 GitHub 授权页面点击 “Cancel” 时的响应
  2. 修复了一个导致 JWT 校验无效的问题
  3. fix #58
  4. fix #57
  5. fix #50
  6. fix #49
Enter-tainer commented 2 months ago

话说githubapp需要有额外的权限吗，还是只要install了就行

shaokeyibb commented 2 months ago

话说githubapp需要有额外的权限吗，还是只要install了就行

要的，就是那个 member 的 read 权限

Enter-tainer commented 2 months ago

现在是不是还没法install😂试了下报了个 400 bad request。我猜是这个merge并部署之后才可以

shaokeyibb commented 2 months ago

现在是不是还没法install😂试了下报了个 400 bad request。我猜是这个merge并部署之后才可以

啊那个无所谓，后台不会拿安装信息做额外的事儿，这个安装只是为了在用户登录的时候有权拿到用户的组织信息罢了

Enter-tainer commented 2 months ago

研究了一下github权限机制，发现好像还有点复杂。。我们现在在用的权限系统实际上是 Team 机制。（而不是admin）。admin的权限也可能有点太大了，可能更合理的做法是用 team，专门建一个team，允许team里面的人来管理评论。

https://docs.github.com/en/rest/teams/members?apiVersion=2022-11-28#get-team-membership-for-a-user

所以可能这里要改一改，改成 user 是否是属于某个 org 的 某个 team

Enter-tainer commented 2 months ago

env我已经加好了，先merge一版吧

shaokeyibb commented 2 months ago

env我已经加好了，先merge一版吧

稍等下，我这边打算把头像加上（

Enter-tainer commented 2 months ago

我先merge一把吧，不然等会review不动了

Enter-tainer commented 2 months ago

应该不会影响你本地的开发，你到时候rebase一下master就好