couple of syntax and unmarshaling fixes. closes PagerDuty#232

Erog38 · Jul 16, 2020 · 4c106de · 4c106de
1 parent ab036b7
commit 4c106de
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 8 deletions.
diff --git a/incident.go b/incident.go
@@ -392,8 +392,8 @@ type ListIncidentLogEntriesOptions struct {
 	Includes   []string `url:"include,omitempty,brackets"`
 	IsOverview bool     `url:"is_overview,omitempty"`
 	TimeZone   string   `url:"time_zone,omitempty"`
-	Since      string   `url:since,omitempty`
-	Until      string   `url:until,omitempty`
+	Since      string   `url:"since,omitempty"`
+	Until      string   `url:"until,omitempty"`
 }
 
 // ListIncidentLogEntries lists existing log entries for the specified incident.
@@ -439,10 +439,10 @@ type ResponderRequestTargets struct {
 
 // ResponderRequestOptions defines the input options for the Create Responder function.
 type ResponderRequestOptions struct {
-	From        string                   `json:"-"`
-	Message     string                   `json:"message"`
-	RequesterID string                   `json:"requester_id"`
-	Targets     []ResponderRequestTarget `json:"responder_request_targets"`
+	From        string                    `json:"-"`
+	Message     string                    `json:"message"`
+	RequesterID string                    `json:"requester_id"`
+	Targets     []ResponderRequestTargets `json:"responder_request_targets"`
 }
 
 // ResponderRequest contains the API structure for an incident responder request.

diff --git a/incident_test.go b/incident_test.go
@@ -573,7 +573,7 @@ func TestIncident_ResponderRequest(t *testing.T) {
 		From:        from,
 		Message:     "help",
 		RequesterID: "PL1JMK5",
-		Targets:     []ResponderRequestTarget{r},
+		Targets:     []ResponderRequestTargets{{r}},
 	}
 
 	user := User{}

diff --git a/webhook.go b/webhook.go
@@ -27,6 +27,7 @@ type IncidentDetails struct {
 	Urgency              string            `json:"urgency"`
 	ResolveReason        *string           `json:"resolve_reason"`
 	AlertCounts          AlertCounts       `json:"alert_counts"`
+	Alerts               []WebhookAlert    `json:"alerts"`
 	Metadata             interface{}       `json:"metadata"`
 	Description          string            `json:"description"`
 }
@@ -40,11 +41,16 @@ type WebhookPayloadMessages struct {
 type WebhookPayload struct {
 	ID         string          `json:"id"`
 	Event      string          `json:"event"`
-	CreatedOn  time.Time       `json:"created_on`
+	CreatedOn  time.Time       `json:"created_on"`
 	Incident   IncidentDetails `json:"incident"`
 	LogEntries []LogEntry      `json:"log_entries"`
 }
 
+// WebhookAlert describes the alert of the incident
+type WebhookAlert struct {
+	AlertKey string `json:"alert_key"`
+}
+
 // DecodeWebhook decodes a webhook from a response object.
 func DecodeWebhook(r io.Reader) (*WebhookPayloadMessages, error) {
 	var payload WebhookPayloadMessages