Integration admin page
Review Request #7106 — Created March 23, 2015 and discarded
The integration admin page provides a single interface to manage all the configured integrations. Admmin user can add, delete, change or toggle the state of the integration through this interface. On this page, the admin user will get to see all the configured integrations within the user's permission settings. Each of these integrations will be group according to their integration type, with a simple description and possible actions.
All of the interaction are achieved through the backbone model, which relies on the integration WebApiResource.
Unit test done.
Manual testing on the integration page.
- Toggle of state
- Delete of integration
- Multiple integration class
- Single integration class
| Description | From | Last Updated |
|---|---|---|
|
"Respresents" Also, maybe capitalize integration |
CR cristocrat | |
|
"... an integration class" |
CR cristocrat | |
|
maybe get rid of the first "the" |
CR cristocrat | |
|
maybe reword to: "... ConfiguredIntegration object, which provides enabling or disabling of the instance." |
CR cristocrat | |
|
provides (with an s) |
CR cristocrat | |
|
provides (with an s) |
CR cristocrat | |
|
Maybe capitalize integration |
CR cristocrat | |
|
"It will also provide a link to create a new instance of this" |
CR cristocrat | |
|
"This will load a configured..." |
CR cristocrat | |
|
"...objects that belong to..." |
CR cristocrat | |
|
"Displays a configured integration in a Manage Integration list." |
CR cristocrat | |
|
will with two l's "...deleting of the instance." |
CR cristocrat | |
|
"This destroys..." "... by calling HTTP DLETE..." |
CR cristocrat | |
|
destroy typo |
CR cristocrat | |
|
This should use RB.BaseResource, which handles URLs, parsing, etc. |
 david
|
|
|
I doubt we want to append another period--all the error strings in webapi/errors.py have text that ends in a period. |
david
|
|
|
I doubt we want to append another period--all the error strings in webapi/errors.py have text that ends in a period. |
david
|
|
|
This should use RB.BaseResource, which handles URLs, parsing, etc. |
david
|
|
|
The "Add" text should be passed in as a variable, and run through gettext(). |
david
|
|
|
This text should be passed in and run through gettext(). |
david
|
|
|
This text should be run through gettext(). |
david
|
|
|
We shouldn't add another period. |
david
|
XU
- Diff:
-
Revision 2 (+602)
-
Tool: PEP8 Style Checker Processed Files: reviewboard/staticbundles.py Ignored Files: reviewboard/static/rb/css/pages/integration.less reviewboard/templates/integrations/configure_integration.html reviewboard/templates/integrations/integration_list.html reviewboard/templates/admin/base_site.html reviewboard/static/rb/js/integrations/models/integrationManagerModel.js reviewboard/static/rb/js/integrations/views/integrationManagerView.js Tool: Pyflakes Processed Files: reviewboard/staticbundles.py Ignored Files: reviewboard/static/rb/css/pages/integration.less reviewboard/templates/integrations/configure_integration.html reviewboard/templates/integrations/integration_list.html reviewboard/templates/admin/base_site.html reviewboard/static/rb/js/integrations/models/integrationManagerModel.js reviewboard/static/rb/js/integrations/views/integrationManagerView.js
XU
- Diff:
-
Revision 3 (+602)
-
Tool: PEP8 Style Checker Processed Files: reviewboard/staticbundles.py Ignored Files: reviewboard/static/rb/css/pages/integration.less reviewboard/templates/integrations/configure_integration.html reviewboard/templates/integrations/integration_list.html reviewboard/templates/admin/base_site.html reviewboard/static/rb/js/integrations/models/integrationManagerModel.js reviewboard/static/rb/js/integrations/views/integrationManagerView.js Tool: Pyflakes Processed Files: reviewboard/staticbundles.py Ignored Files: reviewboard/static/rb/css/pages/integration.less reviewboard/templates/integrations/configure_integration.html reviewboard/templates/integrations/integration_list.html reviewboard/templates/admin/base_site.html reviewboard/static/rb/js/integrations/models/integrationManagerModel.js reviewboard/static/rb/js/integrations/views/integrationManagerView.js
XU
- Change Summary:
-
Update the description
- Description:
-
~ This provides the backbone model and admin page to manage all the integration.
~ The integration admin page provides a single interface to manage all the configured integrations. Admmin user can add, delete, change or toggle the state of the integration through this interface. On this page, the admin user will get to see all the configured integrations within the user's permission settings. Each of these integrations will be group according to their integration type, with a simple description and possible actions.
~ Update: March 23, 2015
~ All of the interaction are achieved through the backbone model, which relies on the integration WebApiResource.
- - - Added the latest design for the admin page.
XU
- Change Summary:
-
Fix comments for various methods
- Diff:
-
Revision 4 (+602)
-
Tool: Pyflakes Processed Files: reviewboard/staticbundles.py Ignored Files: reviewboard/static/rb/css/pages/integration.less reviewboard/templates/integrations/configure_integration.html reviewboard/templates/integrations/integration_list.html reviewboard/templates/admin/base_site.html reviewboard/static/rb/js/integrations/models/integrationManagerModel.js reviewboard/static/rb/js/integrations/views/integrationManagerView.js Tool: PEP8 Style Checker Processed Files: reviewboard/staticbundles.py Ignored Files: reviewboard/static/rb/css/pages/integration.less reviewboard/templates/integrations/configure_integration.html reviewboard/templates/integrations/integration_list.html reviewboard/templates/admin/base_site.html reviewboard/static/rb/js/integrations/models/integrationManagerModel.js reviewboard/static/rb/js/integrations/views/integrationManagerView.js
XU
- Diff:
-
Revision 5 (+602)
-
Tool: Pyflakes Processed Files: reviewboard/staticbundles.py Ignored Files: reviewboard/static/rb/css/pages/integration.less reviewboard/templates/integrations/configure_integration.html reviewboard/templates/integrations/integration_list.html reviewboard/templates/admin/base_site.html reviewboard/static/rb/js/integrations/models/integrationManagerModel.js reviewboard/static/rb/js/integrations/views/integrationManagerView.js Tool: PEP8 Style Checker Processed Files: reviewboard/staticbundles.py Ignored Files: reviewboard/static/rb/css/pages/integration.less reviewboard/templates/integrations/configure_integration.html reviewboard/templates/integrations/integration_list.html reviewboard/templates/admin/base_site.html reviewboard/static/rb/js/integrations/models/integrationManagerModel.js reviewboard/static/rb/js/integrations/views/integrationManagerView.js
XU
- Diff:
-
Revision 6 (+602)
-
Tool: PEP8 Style Checker Processed Files: reviewboard/staticbundles.py Ignored Files: reviewboard/static/rb/css/pages/integration.less reviewboard/templates/integrations/configure_integration.html reviewboard/templates/integrations/integration_list.html reviewboard/templates/admin/base_site.html reviewboard/static/rb/js/integrations/models/integrationManagerModel.js reviewboard/static/rb/js/integrations/views/integrationManagerView.js Tool: Pyflakes Processed Files: reviewboard/staticbundles.py Ignored Files: reviewboard/static/rb/css/pages/integration.less reviewboard/templates/integrations/configure_integration.html reviewboard/templates/integrations/integration_list.html reviewboard/templates/admin/base_site.html reviewboard/static/rb/js/integrations/models/integrationManagerModel.js reviewboard/static/rb/js/integrations/views/integrationManagerView.js
XU
- Change Summary:
-
Clean up css
- Diff:
-
Revision 7 (+598)
-
Tool: Pyflakes Processed Files: reviewboard/staticbundles.py Ignored Files: reviewboard/static/rb/css/pages/integration.less reviewboard/templates/integrations/configure_integration.html reviewboard/templates/integrations/integration_list.html reviewboard/templates/admin/base_site.html reviewboard/static/rb/js/integrations/models/integrationManagerModel.js reviewboard/static/rb/js/integrations/views/integrationManagerView.js Tool: PEP8 Style Checker Processed Files: reviewboard/staticbundles.py Ignored Files: reviewboard/static/rb/css/pages/integration.less reviewboard/templates/integrations/configure_integration.html reviewboard/templates/integrations/integration_list.html reviewboard/templates/admin/base_site.html reviewboard/static/rb/js/integrations/models/integrationManagerModel.js reviewboard/static/rb/js/integrations/views/integrationManagerView.js
Tool: Pyflakes Processed Files: reviewboard/staticbundles.py Ignored Files: reviewboard/static/rb/css/pages/integration.less reviewboard/templates/integrations/configure_integration.html reviewboard/templates/integrations/integration_list.html reviewboard/templates/admin/base_site.html reviewboard/static/rb/js/integrations/models/integrationManagerModel.js reviewboard/static/rb/js/integrations/views/integrationManagerView.js Tool: PEP8 Style Checker Processed Files: reviewboard/staticbundles.py Ignored Files: reviewboard/static/rb/css/pages/integration.less reviewboard/templates/integrations/configure_integration.html reviewboard/templates/integrations/integration_list.html reviewboard/templates/admin/base_site.html reviewboard/static/rb/js/integrations/models/integrationManagerModel.js reviewboard/static/rb/js/integrations/views/integrationManagerView.js