feat: add contract sign check

Author: benjamin-747

comment.yaml

@@ -24,52 +24,54 @@ task:
 
 command:
   noPermission: >
-    操作失败，您当前没有权限修改该任务。
+    操作失败：您当前没有权限修改该任务。
   invalidTaskState: >
-    操作失败，当前任务的状态不允许进行该操作。
+    操作失败：当前任务的状态不允许进行该操作。
   unsupportStuCommand: >
-    "不支持的命令. 当前仅支持 /request-assign, /request-complete, 和 /request-release命令。"
+    不支持的命令：当前仅支持 /request-assign, /request-complete, 和 /request-release命令。
   unsupportMentorCommand: >
-    "不支持的命令. 当前仅支持 /intern-disapprove, /intern-approve, /intern-fail, /intern-done 和 /intern-close命令。"
+    不支持的命令：当前仅支持 /intern-disapprove, /intern-approve, /intern-fail, /intern-done 和 /intern-close命令。
 
 requestAssign:
   success: >
-    认领成功，等待导师审核，请及时联系导师进行审核。
+    认领成功：等待导师审核，请及时联系导师进行审核。
   waitingInfoReview: >
-    认领失败，需要先到 https://summer-ospp.ac.cn/r2cnLogin 进行学生注册并通过审核后才能认领任务。
+    认领失败：请先在 https://summer-ospp.ac.cn/r2cnLogin 进行学生注册并通过审核后才能认领任务。
+  waitingContract: >
+    认领失败：合同未签署，如果已经确认签署请在 R2CN Discord(https://discord.gg/WRp4TKv6rh) 的 #signup-student 频道联系管理员更新合同签署状态。
   existTask: >
-    认领失败，你已经存在认领中的任务了。
+    认领失败：你已经存在认领中的任务了。
   claimByOther: >
-    认领失败，已经有同学申请了此任务，请选择其他任务。
+    认领失败：已经有同学申请了此任务，请选择其他任务。
   alreadyClaim: >
-    认领失败，该任务已经被你认领，请不要重复认领！
+    认领失败：该任务已经被你认领，请不要重复认领！
 
 internDisapprove:
   success: >
-    拒绝认领成功, 任务释放为开放状态。
+    拒绝认领成功：任务释放为开放状态。
 
 internApprove:
   success: >
-    认领审核成功，已将该任务分配给学生。
+    认领审核成功：已将该任务分配给学生。
 
 requestComplete:
   success: >
-    请求完成成功, 等待导师进行完成审核。
+    请求完成成功：等待导师进行完成审核。
   noRelatedPR: >
-    请求完成失败, 至少需要一个关联的PR。
+    请求完成失败：至少需要一个关联的PR。
 
 requestRelease:
   success: >
-    请求释放成功，当前任务重新开放。
+    请求释放成功：当前任务重新开放。
 
 internFail:
   success: >
-    学生任务失败，当前任务重新开放。
+    学生任务失败：当前任务重新开放。
 
 internDone:
   success: >
-    操作成功，任务关闭并发放积分给学生。
+    操作成功：任务关闭并发放积分给学生。
 
 internClose:
   success: >
-    操作成功，任务关闭不再追踪。
+    操作成功：任务关闭不再追踪。

src/common.ts

@@ -61,6 +61,7 @@ interface CommandComment {
 interface RequestAssign {
     success: string,
     waitingInfoReview: string,
+    waitingContract: string,
     existTask: string,
     claimByOther: string,
     alreadyClaim: string,
@@ -98,7 +99,6 @@ interface InternClose {
 export interface CommandRequest {
     github_issue_id: number,
     login: string
-    github_id: number
     student_name?: string
 }
 

src/index.ts

@@ -89,7 +89,7 @@ export default (app: Probot) => {
                 // Update Task Score
                 await Task.updateTaskScore(context.payload.issue, score);
                 await context.octokit.issues.createComment(context.issue({
-                    body: config.comment.task.successUpdate + score
+                    body: `${config.comment.task.successUpdate.trim()}: ${score}`
                 }));
 
             }

src/mentor.ts

@@ -34,7 +34,6 @@ export async function handle_mentor_cmd(context: Context, config: Config, payloa
     const req = {
         github_issue_id: task.github_issue_id,
         login: user.login,
-        github_id: user.id
     };
     switch (command) {
         case "/intern-disapprove":

src/student.ts

@@ -25,7 +25,6 @@ export async function handle_stu_cmd(context: Context, config: Config, payload:
     const req = {
         github_issue_id: task.github_issue_id,
         login: user.login,
-        github_id: user.id
     };
 
     switch (command) {
@@ -38,11 +37,17 @@ export async function handle_stu_cmd(context: Context, config: Config, payload:
                 return setResponse(config.comment.command.invalidTaskState);
             }
 
+            // 学生身份校验
             const verify = await verifyStudentIdentity(user.login);
             if (!verify.success) {
                 return setResponse(config.comment.requestAssign.waitingInfoReview);
             }
 
+            // 合同签署校验
+            // if (!verify.contract_deadline) {
+            //     return setResponse(config.comment.requestAssign.waitingContract);
+            // }
+
             if (task.student_github_login === user.login) {
                 return setResponse(config.comment.requestAssign.alreadyClaim);
             }
@@ -54,7 +59,6 @@ export async function handle_stu_cmd(context: Context, config: Config, payload:
             if (await requestAssign({
                 github_issue_id: task.github_issue_id,
                 login: user.login,
-                github_id: user.id,
                 student_name: verify.student_name,
             })) {
                 return setResponse(config.comment.requestAssign.success, true);
@@ -108,6 +112,7 @@ interface UserReq {
 interface VerifyStuRes {
     success: true,
     student_name?: string
+    contract_deadline?: string,
 }
 
 async function verifyStudentIdentity(login: string) {