Added command for removing any todo

.gitignore

@@ -1,3 +1,2 @@
 dist
 .DS_Store
-node_modules

server/command.go

@@ -330,26 +330,18 @@ func (p *Plugin) runPopCommand(args []string, extra *model.CommandArgs) (bool, e
 }
 
 func (p *Plugin) runRemoveCommand(args []string, extra *model.CommandArgs) (bool, error) {
-	message := strings.Join(args, " ")
-	listID := MyListKey
-	issueList, err := p.listManager.GetIssueList(extra.UserId, listID)
-	if err != nil {
-		return false, err
-	}
-
-	var issueID string
-	for _, issue := range issueList {
-		if issue.Message == message {
-			issueID = issue.ID
-		}
+	if args[0] != "--todo" {
+		p.postCommandResponse(extra, "Invalid command provided")
+		return false, nil
 	}
 
-	if issueID == "" {
-		p.postCommandResponse(extra, "Can't find the todo in list")
+	todoID := args[1]
+	if todoID == "" {
+		p.postCommandResponse(extra, "Empty todoID provided")
 		return false, nil
 	}
 
-	issue, foreignID, _, _, err := p.listManager.RemoveIssue(extra.UserId, issueID)
+	issue, foreignID, _, _, err := p.listManager.RemoveIssue(extra.UserId, todoID)
 	if err != nil {
 		return false, err
 	}
@@ -497,8 +489,8 @@ func getAutocompleteData() *model.AutocompleteData {
 	pop := model.NewAutocompleteData("pop", "", "Removes the Todo issue at the top of the list")
 	todo.AddCommand(pop)
 
-	remove := model.NewAutocompleteData("remove", "", "Removes the given Todo")
-	remove.AddTextArgument("E.g. be average", "[message]", "")
+	remove := model.NewAutocompleteData("remove", "--todo id", "Removes the given Todo")
+	remove.AddNamedDynamicListArgument("todo", "--todo todoID", getAutocompletePath(AutocompletePathRemoveTodoSuggestions), true)
 	todo.AddCommand(remove)
 
 	send := model.NewAutocompleteData("send", "[user] [todo]", "Sends a Todo to a specified user")
@@ -527,3 +519,7 @@ func getAutocompleteData() *model.AutocompleteData {
 	todo.AddCommand(help)
 	return todo
 }
+
+func getAutocompletePath(path string) string {
+	return "plugins/" + manifest.Id + "/autocomplete" + path
+}

server/constants.go

@@ -0,0 +1,6 @@
+package main
+
+const (
+	AutocompletePath                      = "/autocomplete"
+	AutocompletePathRemoveTodoSuggestions = "/getRemoveTodoSuggestions"
+)

server/plugin.go

@@ -127,6 +127,8 @@ func (p *Plugin) ServeHTTP(c *plugin.Context, w http.ResponseWriter, r *http.Req
 		p.handleEdit(w, r)
 	case "/change_assignment":
 		p.handleChangeAssignment(w, r)
+	case AutocompletePath + AutocompletePathRemoveTodoSuggestions:
+		p.getRemoveTodoSuggestions(w, r)
 	default:
 		http.NotFound(w, r)
 	}
@@ -669,3 +671,40 @@ func (p *Plugin) handleErrorWithCode(w http.ResponseWriter, code int, errTitle s
 	})
 	_, _ = w.Write(b)
 }
+
+func (p *Plugin) getRemoveTodoSuggestions(w http.ResponseWriter, r *http.Request) {
+	userID := r.Header.Get("Mattermost-User-ID")
+	if userID == "" {
+		http.Error(w, "Not authorized", http.StatusUnauthorized)
+		return
+	}
+	listID := MyListKey
+	issues, err := p.listManager.GetIssueList(userID, listID)
+	if err != nil {
+		p.API.LogError("Unable to get issues for user err=" + err.Error())
+		p.handleErrorWithCode(w, http.StatusInternalServerError, "Unable to get issues for user", err)
+		return
+	}
+
+	out := []model.AutocompleteListItem{}
+	for _, issue := range issues {
+		s := model.AutocompleteListItem{
+			Item:     issue.ID,
+			Hint:     issue.Message,
+			HelpText: issue.Description,
+		}
+		out = append(out, s)
+	}
+
+	outJSON, err := json.Marshal(out)
+	if err != nil {
+		p.API.LogError("Unable marhsal issues list to json err=" + err.Error())
+		p.handleErrorWithCode(w, http.StatusInternalServerError, "Unable marhsal issues list to json", err)
+		return
+	}
+
+	_, err = w.Write(outJSON)
+	if err != nil {
+		p.API.LogError("Unable to write json response err=" + err.Error())
+	}
+}