opt: trigger tag code

tsundata · Mar 20, 2021 · a2c3ecb · a2c3ecb
1 parent 75279c1
commit a2c3ecb
Show file tree

Hide file tree

Showing 2 changed files with 57 additions and 42 deletions.
diff --git a/internal/app/message/trigger/tags/issue.go b/internal/app/message/trigger/tags/issue.go
@@ -26,24 +26,30 @@ func (t *Issue) Handle(ctx *ctx.Context, text string) {
 		return
 	}
 
-	// data
+	// get user
 	client := github.NewGithub("", "", "", accessToken)
 	user, err := client.GetUser()
 	if err != nil {
 		return
 	}
-	if *user.Login != "" {
-		issue, err := client.CreateIssue(*user.Login, "assistant", github.Issue{Title: &text})
-		if err != nil {
-			ctx.Logger.Error(err)
-			return
-		}
-		if *issue.ID > 0 {
-			_, err = ctx.MsgClient.Send(context.Background(), &pb.MessageRequest{Text: fmt.Sprintf("Created Issue #%d %s", *issue.Number, *issue.HTMLURL)})
-			if err != nil {
-				ctx.Logger.Error(err)
-				return
-			}
-		}
+	if *user.Login == "" {
+		return
+	}
+
+	// create issue
+	issue, err := client.CreateIssue(*user.Login, "assistant", github.Issue{Title: &text})
+	if err != nil {
+		ctx.Logger.Error(err)
+		return
+	}
+	if *issue.ID == 0 {
+		return
+	}
+
+	// send message
+	_, err = ctx.MsgClient.Send(context.Background(), &pb.MessageRequest{Text: fmt.Sprintf("Created Issue #%d %s", *issue.Number, *issue.HTMLURL)})
+	if err != nil {
+		ctx.Logger.Error(err)
+		return
 	}
 }
diff --git a/internal/app/message/trigger/tags/todo.go b/internal/app/message/trigger/tags/todo.go
@@ -26,41 +26,50 @@ func (t *Todo) Handle(ctx *ctx.Context, text string) {
 		return
 	}
 
-	// data
+	// get user
 	client := github.NewGithub("", "", "", accessToken)
 	user, err := client.GetUser()
 	if err != nil {
 		return
 	}
-	if *user.Login != "" {
-		projects, err := client.GetUserProjects(*user.Login)
-		if err != nil {
-			ctx.Logger.Error(err)
-			return
-		}
+	if *user.Login == "" {
+		return
+	}
+
+	// get projects
+	projects, err := client.GetUserProjects(*user.Login)
+	if err != nil {
+		ctx.Logger.Error(err)
+		return
+	}
+	if len(*projects) == 0 {
+		return
+	}
 
-		if len(*projects) > 0 {
-			columns, err := client.GetProjectColumns(*(*projects)[0].ID)
-			if err != nil {
-				ctx.Logger.Error(err)
-				return
-			}
+	// get columns
+	columns, err := client.GetProjectColumns(*(*projects)[0].ID)
+	if err != nil {
+		ctx.Logger.Error(err)
+		return
+	}
+	if len(*columns) == 0 {
+		return
+	}
 
-			if len(*columns) > 0 {
-				card, err := client.CreateCard(*(*columns)[0].ID, github.ProjectCard{Note: &text})
-				if err != nil {
-					ctx.Logger.Error(err)
-					return
-				}
-				if *card.ID > 0 {
-					_, err = ctx.MsgClient.Send(context.Background(), &pb.MessageRequest{Text: fmt.Sprintf("Created Todo Card #%d", *card.ID)})
-					if err != nil {
-						ctx.Logger.Error(err)
-						return
-					}
-				}
-			}
-		}
+	// create card
+	card, err := client.CreateCard(*(*columns)[0].ID, github.ProjectCard{Note: &text})
+	if err != nil {
+		ctx.Logger.Error(err)
+		return
+	}
+	if *card.ID == 0 {
+		return
+	}
 
+	// send message
+	_, err = ctx.MsgClient.Send(context.Background(), &pb.MessageRequest{Text: fmt.Sprintf("Created Todo Card #%d", *card.ID)})
+	if err != nil {
+		ctx.Logger.Error(err)
+		return
 	}
 }