Merge pull request #17 from checkly/kjda/alert-channels

removing id from createAlertChannel

checkly.go

@@ -445,7 +445,6 @@ func (c *Client) DeleteEnvironmentVariable(key string) error {
 // the newly-created alert channel, or an error.
 func (c *Client) CreateAlertChannel(ac AlertChannel) (*AlertChannel, error) {
 	payload := map[string]interface{}{
-		"id":     ac.ID,
 		"type":   ac.Type,
 		"config": ac.GetConfig(),
 	}
@@ -457,7 +456,7 @@ func (c *Client) CreateAlertChannel(ac AlertChannel) (*AlertChannel, error) {
 	if err != nil {
 		return nil, err
 	}
-	if status != http.StatusOK {
+	if status != http.StatusOK && status != http.StatusCreated {
 		return nil, fmt.Errorf("unexpected response status: %d, res: %q, payload: %v", status, res, string(data))
 	}
 	result := map[string]interface{}{}

checkly_test.go

@@ -815,7 +815,6 @@ func validateAlertChannel(t *testing.T, body []byte) {
 		t.Fatalf("decoding error for data %q: %v", body, err)
 	}
 	ac := checkly.AlertChannel{}
-	ac.ID = int64(response["id"].(float64))
 	ac.Type = response["type"].(string)
 	cfgJSON, err := json.Marshal(response["config"])
 	if err != nil {