flake8
passed.
JSHint
passed.
Add documentation on using IntegrationHook.
Review Request #9374 — Created Nov. 14, 2017 and submitted
This adds a doc page on IntegrationHook. It touches upon the purpose of
integrations, linking to the topic page on integrations and the guide on
writing them. It then provides a full example of an integration and
usage of a hook that results in an HTTP POST when review requests are
published.
Built and read the docs. Checked for errors, broken links, and spelling
issues.
-
-
docs/manual/extending/extensions/hooks/integration-hook.rst (Diff revision 1) This is a single long, complicated sentence that attempts to completely describe a complicated subject. How about breaking it up into a few sentences:
These are similar to extensions, but are intended for connecting to third-party services or tools. Each integration can have multiple configurations, making them ideal for cases where different groups or teams may have different requirements. For example, one group may want notifications posted to their chat service while another may not.
-
docs/manual/extending/extensions/hooks/integration-hook.rst (Diff revision 1) Indentation is a little wacky here.
Change Summary:
- Rewrote the introductory paragraph to be much more clear using David's suggested wording.
- Fixed some indentation in the example.
Commit: |
|
||||
|---|---|---|---|---|---|
Diff: |
Revision 2 (+98) |
Checks run (2 succeeded)
flake8
passed.
JSHint
passed.