Skip to content

Commit

Permalink
Merge pull request #42 from Scalingo/fix/41/notifiers_webhook_url
Browse files Browse the repository at this point in the history
fix(notifiers) Correctly store the webhook url when reading
  • Loading branch information
john-scalingo authored Jul 7, 2022
2 parents aff7c59 + 1dde295 commit 17a6436
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions scalingo/resource_scalingo_notifier.go
Original file line number Diff line number Diff line change
Expand Up @@ -236,6 +236,12 @@ func setFromScNotifier(d *schema.ResourceData, c scalingo.API, notifier *scaling

typeData := notifier.Specialize().TypeDataMap()
for key, value := range typeData {
// GoScalingo calls the Webhook URL type data: "webhook url"
// However in this provider, we call it "webhook_url"
// This handles the translation
if key == "webhook url" {
key = "webhook_url"
}
values[key] = value
}

Expand Down

0 comments on commit 17a6436

Please sign in to comment.