-
Notifications
You must be signed in to change notification settings - Fork 393
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Document webhook_notifications
for databricks_job
resource
#1842
Conversation
02fd671
to
6f94ec7
Compare
docs/resources/job.md
Outdated
|
||
### webhook Confiuration Block | ||
|
||
* `id` - ID of the destination that is notified when an event defined in `webhook_notifications` is triggered. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
there is a discrepancy in terminology - here the destination
word is used, and in other places: system notification ID
- it's better to standartize
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changed, ty!
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## master #1842 +/- ##
=======================================
Coverage 90.14% 90.15%
=======================================
Files 142 142
Lines 11358 11363 +5
=======================================
+ Hits 10239 10244 +5
Misses 711 711
Partials 408 408
|
webhook_notifications
for databricks_job
resource
Document
webhook_notifications
field. Webhooks are configured in TF as follows:OpenAPI spec added here.