Add support in Review Board for service integrations.
Review Request #7949 — Created Feb. 7, 2016 and submitted
This makes use of the new Djblets Integrations framework, allowing extensions to register service integrations for Review Board. These integrations work sort of like extensions, in that they can listen for events and act upon them, but are primarily designed to interact with third-party services. Integrations can contain any number of admin-provided configurations. This would allow, for example, a Slack integration to post notifications about review requests to different Slack channels or even teams, depending on the review groups or repositories.
Modified the Slack extension to use integrations, and configured it
for a couple different channels. Saw that it posted to both channels
when updating review requests.Checked that the styles for the pages were correct.
Description | From | Last Updated |
---|---|---|
'django_reset' imported but unused |
reviewbot | |
'from settings_local import *' used; unable to detect undefined names |
reviewbot | |
'PIPELINE_CSS' imported but unused |
reviewbot | |
'PIPELINE_JS' imported but unused |
reviewbot | |
'django_reset' imported but unused |
reviewbot | |
'from settings_local import *' used; unable to detect undefined names |
reviewbot | |
'PIPELINE_CSS' imported but unused |
reviewbot | |
'PIPELINE_JS' imported but unused |
reviewbot | |
Given that this is in Review Board, can we make the language about "any supporting application" more specific? |
david | |
Docstring? |
david | |
'django_reset' imported but unused |
reviewbot | |
'from settings_local import *' used; unable to detect undefined names |
reviewbot | |
'PIPELINE_JS' imported but unused |
reviewbot | |
'PIPELINE_CSS' imported but unused |
reviewbot | |
'django_reset' imported but unused |
reviewbot | |
'from settings_local import *' used; unable to detect undefined names |
reviewbot |
-
Tool: Pyflakes Processed Files: reviewboard/integrations/base.py reviewboard/integrations/urls.py reviewboard/settings.py reviewboard/integrations/forms.py reviewboard/integrations/models.py reviewboard/staticbundles.py reviewboard/extensions/hooks.py reviewboard/integrations/__init__.py reviewboard/admin/urls.py reviewboard/integrations/views.py docs/manual/conf.py Ignored Files: reviewboard/static/rb/css/pages/admin.less docs/manual/extending/coderef/index.rst reviewboard/static/rb/css/pages/admin-integrations.less reviewboard/templates/admin/base_site.html Tool: PEP8 Style Checker Processed Files: reviewboard/integrations/base.py reviewboard/integrations/urls.py reviewboard/settings.py reviewboard/integrations/forms.py reviewboard/integrations/models.py reviewboard/staticbundles.py reviewboard/extensions/hooks.py reviewboard/integrations/__init__.py reviewboard/admin/urls.py reviewboard/integrations/views.py docs/manual/conf.py Ignored Files: reviewboard/static/rb/css/pages/admin.less docs/manual/extending/coderef/index.rst reviewboard/static/rb/css/pages/admin-integrations.less reviewboard/templates/admin/base_site.html
-
-
-
-
- Change Summary:
-
- Fixed some typos in the docs for
Integration
. - Turned off function signatures in the
autosummary
.
- Fixed some typos in the docs for
-
Tool: Pyflakes Processed Files: reviewboard/integrations/base.py reviewboard/integrations/urls.py reviewboard/settings.py reviewboard/integrations/forms.py reviewboard/integrations/models.py reviewboard/staticbundles.py reviewboard/extensions/hooks.py reviewboard/integrations/__init__.py reviewboard/admin/urls.py reviewboard/integrations/views.py docs/manual/conf.py Ignored Files: reviewboard/static/rb/css/pages/admin.less docs/manual/extending/coderef/index.rst reviewboard/static/rb/css/pages/admin-integrations.less reviewboard/templates/admin/base_site.html Tool: PEP8 Style Checker Processed Files: reviewboard/integrations/base.py reviewboard/integrations/urls.py reviewboard/settings.py reviewboard/integrations/forms.py reviewboard/integrations/models.py reviewboard/staticbundles.py reviewboard/extensions/hooks.py reviewboard/integrations/__init__.py reviewboard/admin/urls.py reviewboard/integrations/views.py docs/manual/conf.py Ignored Files: reviewboard/static/rb/css/pages/admin.less docs/manual/extending/coderef/index.rst reviewboard/static/rb/css/pages/admin-integrations.less reviewboard/templates/admin/base_site.html
-
-
-
-
- Change Summary:
-
- Fixed the
IntegrationHook
docstring to be more specific to Review Board. - Added a missing docstring.
- Fixed the
-
Tool: Pyflakes Processed Files: reviewboard/integrations/base.py reviewboard/integrations/urls.py reviewboard/settings.py reviewboard/integrations/forms.py reviewboard/integrations/models.py reviewboard/staticbundles.py reviewboard/extensions/hooks.py reviewboard/integrations/__init__.py reviewboard/admin/urls.py reviewboard/integrations/views.py docs/manual/conf.py Ignored Files: reviewboard/static/rb/css/pages/admin.less docs/manual/extending/coderef/index.rst reviewboard/static/rb/css/pages/admin-integrations.less reviewboard/templates/admin/base_site.html Tool: PEP8 Style Checker Processed Files: reviewboard/integrations/base.py reviewboard/integrations/urls.py reviewboard/settings.py reviewboard/integrations/forms.py reviewboard/integrations/models.py reviewboard/staticbundles.py reviewboard/extensions/hooks.py reviewboard/integrations/__init__.py reviewboard/admin/urls.py reviewboard/integrations/views.py docs/manual/conf.py Ignored Files: reviewboard/static/rb/css/pages/admin.less docs/manual/extending/coderef/index.rst reviewboard/static/rb/css/pages/admin-integrations.less reviewboard/templates/admin/base_site.html
-
-