Merge pull request #18 from x-qdo/bugfix/multi_assignee

add list of assignees to task model
x-qdo · Apr 2, 2021 · e295b70 · e295b70
2 parents 9189a8c + 36added
commit e295b70
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 7 deletions.
diff --git a/pkg/handler/clickup.go b/pkg/handler/clickup.go
@@ -101,7 +101,7 @@ func (h *clickUpWebhooks) doAction(event *clickup.WebhookEvent, tenant string) e
 		return errors.Wrap(err, "ClickUp webhook: slackChannel is not defined")
 	}
 
-	changes = generateTaskChangesByEvent(event)
+	changes = generateTaskChangesByEvent(event, task)
 	err = h.publisher.ClickUpTaskUpdated(changes, slackChannel)
 	if err != nil {
 		return errors.Wrap(err, "ClickUp webhook: can't trigger changes event")
@@ -110,7 +110,7 @@ func (h *clickUpWebhooks) doAction(event *clickup.WebhookEvent, tenant string) e
 	return nil
 }
 
-func generateTaskChangesByEvent(event *clickup.WebhookEvent) model.TaskChanges {
+func generateTaskChangesByEvent(event *clickup.WebhookEvent, task *clickup.Task) model.TaskChanges {
 	changes := model.TaskChanges{
 		Type:      string(event.Type),
 		ClickupID: event.TaskID,
@@ -129,11 +129,7 @@ func generateTaskChangesByEvent(event *clickup.WebhookEvent) model.TaskChanges {
 				value = after["priority"]
 			}
 		case clickup.TaskAssigneeUpdated:
-			if after, ok := historyItem.After.(map[string]interface{}); ok {
-				value = after["username"]
-			} else {
-				value = nil
-			}
+			value = task.Assignees
 		}
 		changes.AddChange(historyItem.Field, value)
 		changes.Username = historyItem.User.Username

diff --git a/pkg/provider/clickup/model.go b/pkg/provider/clickup/model.go
@@ -35,6 +35,7 @@ type Task struct {
 	Archived     bool          `json:"archived"`
 	TeamID       string        `json:"team_id"`
 	CustomFields []CustomField `json:"custom_fields,omitempty"`
+	Assignees    []User        `json:"assignees"`
 
 	List struct {
 		ID string `json:"id"`