-
Notifications
You must be signed in to change notification settings - Fork 356
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
docs: Apply minor edits #8215
docs: Apply minor edits #8215
Conversation
✅ Deploy Preview for determined-ui canceled.
|
Currently we support two types of webhooks: ``Slack`` and ``Default``. A payload for a ``Default`` | ||
webhook will contain information about the event itself, the trigger for the event, and the entity | ||
that triggered the event. The shape of ``event_data`` is determined by ``event_type``. Below is an | ||
For webhooks, you have two options: ``Slack`` and ``Default``. A payload for a ``Default`` webhook |
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.
I don't think I like this better; it seems important to mention "type" as the answer to "options for what?". Perhaps "Determined supports two types of webhooks...".
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.
yes i like that
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.
edit applied
9e592d3
to
3056473
Compare
3056473
to
e08e400
Compare
Fix typos, apply style guide.