Integration forms and config template
Review Request #7136 — Created March 30, 2015 and discarded
This provides the integration forms class to manage the configuration of a integration in the configure page. The config template allows integration to display additional information in the configure integration page.
This form provides some preset form fields that are common for an integration. Integration could further customize the form for its own configuration by subclassing the integration forms.
The config template allows integration to provide a more detailed description, with extra content to simplify the process of setting up an integration.
Manual testing with a dummy integration.
Pass all unit test.
| Description | From | Last Updated |
|---|---|---|
|
Sets -> Set |
JY jyuen | |
|
Loads -> Load |
JY jyuen | |
|
Saves -> Save |
JY jyuen | |
|
Col: 80 E501 line too long (90 > 79 characters) |
 reviewbot
|
|
|
Creates -> Create |
JY jyuen | |
|
Col: 9 E123 closing bracket does not match indentation of opening bracket's line |
reviewbot
|
|
|
This should use super |
 david
|
|
|
This can be combined: try: return self.config.integration.default_configurations[key] except KeyError: raise KeyError(...) |
david
|
|
|
Add another blank line here. |
david
|
|
|
Add a trailing comma here. |
david
|
|
|
Only one blank line here. |
david
|
|
|
HTML should be single-space indented. |
david
|
|
|
This should still be in a blocktrans. You can use blocktrans with to define the variable. |
david
|
|
|
This should still be in a blocktrans. You can use blocktrans with to define the variable. |
david
|
- Change Summary:
-
Fix issues from review
- Testing Done:
-
+ Manual testing with a dummy integration.
- Commit:
-
f1656f2503ade52ca9c328d139d6af06754eb4f1c8d3d067d37fff56f94be3dea245fbe0968e227a
-
Tool: Pyflakes Processed Files: reviewboard/integrations/forms.py reviewboard/integrations/views.py reviewboard/integrations/integration.py reviewboard/integrations/configurations.py Ignored Files: reviewboard/templates/integrations/configure_integration.html Tool: PEP8 Style Checker Processed Files: reviewboard/integrations/forms.py reviewboard/integrations/views.py reviewboard/integrations/integration.py reviewboard/integrations/configurations.py Ignored Files: reviewboard/templates/integrations/configure_integration.html
- Summary:
-
[WIP] Integration forms and config templateIntegration forms and config template
- Description:
-
~ This provides the integration forms class to manage the configuration of a integration in the configure page.
~ This provides the integration forms class to manage the configuration of a integration in the configure page. The config template allows integration to display additional information in the configure integration page.
- Testing Done:
-
~ Manual testing with a dummy integration.
~ Manual testing with a dummy integration.
+ Pass all unit test. - Commit:
-
c8d3d067d37fff56f94be3dea245fbe0968e227a487740d6f8af370734331b9cda8066a1ff915d9d
- Diff:
-
Revision 3 (+174 -11)
- Added Files:
-
Tool: Pyflakes Processed Files: reviewboard/integrations/forms.py reviewboard/integrations/views.py reviewboard/integrations/integration.py reviewboard/integrations/configurations.py Ignored Files: reviewboard/templates/integrations/configure_integration.html reviewboard/static/rb/css/pages/integration.less reviewboard/templates/integrations/config_template.html Tool: PEP8 Style Checker Processed Files: reviewboard/integrations/forms.py reviewboard/integrations/views.py reviewboard/integrations/integration.py reviewboard/integrations/configurations.py Ignored Files: reviewboard/templates/integrations/configure_integration.html reviewboard/static/rb/css/pages/integration.less reviewboard/templates/integrations/config_template.html
- Description:
-
This provides the integration forms class to manage the configuration of a integration in the configure page. The config template allows integration to display additional information in the configure integration page.
+ + This form provides some preset form fields that are common for an integration. Integration could further customize the form for its own configuration by subclassing the integration forms.
+ + The config template allows integration to provide a more detailed description, with extra content to simplify the process of setting up an integration.
- Commit:
-
487740d6f8af370734331b9cda8066a1ff915d9dcb4d3a3dd5b652578cc06b5f055753d5e34bc3ba
- Diff:
-
Revision 4 (+175 -12)
-
Tool: Pyflakes Processed Files: reviewboard/integrations/forms.py reviewboard/integrations/views.py reviewboard/integrations/integration.py reviewboard/integrations/configurations.py Ignored Files: reviewboard/templates/integrations/configure_integration.html reviewboard/static/rb/css/pages/integration.less reviewboard/templates/integrations/config_template.html Tool: PEP8 Style Checker Processed Files: reviewboard/integrations/forms.py reviewboard/integrations/views.py reviewboard/integrations/integration.py reviewboard/integrations/configurations.py Ignored Files: reviewboard/templates/integrations/configure_integration.html reviewboard/static/rb/css/pages/integration.less reviewboard/templates/integrations/config_template.html
- Commit:
-
cb4d3a3dd5b652578cc06b5f055753d5e34bc3ba1ba3e404029cc412a0924904e07495fd05b1978a
- Diff:
-
Revision 5 (+208 -13)
-
Tool: PEP8 Style Checker Processed Files: reviewboard/integrations/forms.py reviewboard/integrations/views.py reviewboard/integrations/integration.py reviewboard/integrations/configurations.py Ignored Files: reviewboard/templates/integrations/configure_integration.html reviewboard/static/rb/css/pages/integration.less reviewboard/templates/integrations/config_template.html Tool: Pyflakes Processed Files: reviewboard/integrations/forms.py reviewboard/integrations/views.py reviewboard/integrations/integration.py reviewboard/integrations/configurations.py Ignored Files: reviewboard/templates/integrations/configure_integration.html reviewboard/static/rb/css/pages/integration.less reviewboard/templates/integrations/config_template.html -
- Commit:
-
1ba3e404029cc412a0924904e07495fd05b1978a2d7653cb3753a2492b32fffc7aa07a68df8127ee
- Diff:
-
Revision 6 (+207 -13)
-
Tool: PEP8 Style Checker Processed Files: reviewboard/integrations/forms.py reviewboard/integrations/views.py reviewboard/integrations/integration.py reviewboard/integrations/configurations.py Ignored Files: reviewboard/templates/integrations/configure_integration.html reviewboard/static/rb/css/pages/integration.less reviewboard/templates/integrations/config_template.html Tool: Pyflakes Processed Files: reviewboard/integrations/forms.py reviewboard/integrations/views.py reviewboard/integrations/integration.py reviewboard/integrations/configurations.py Ignored Files: reviewboard/templates/integrations/configure_integration.html reviewboard/static/rb/css/pages/integration.less reviewboard/templates/integrations/config_template.html
Tool: PEP8 Style Checker Processed Files: reviewboard/webapi/tests/test_configured_integration.py reviewboard/integrations/forms.py reviewboard/integrations/views.py reviewboard/integrations/configurations.py reviewboard/integrations/manager.py reviewboard/integrations/integration.py Tool: Pyflakes Processed Files: reviewboard/webapi/tests/test_configured_integration.py reviewboard/integrations/forms.py reviewboard/integrations/views.py reviewboard/integrations/configurations.py reviewboard/integrations/manager.py reviewboard/integrations/integration.py