Fix up the integrations pages for modern Django and Djblets changes.
Review Request #11013 — Created May 7, 2020 and submitted
This makes a handful of changes to the Admin UI for managing
integrations, fixing up some interactions, and leveraging new work in
the Djblets Config Forms UI.The main integrations configuration template has changed to hard-code
fewer things (making it easier to integrate into the Review Board admin
UI), moving down some of the Django-specific parts into Djblets's base
admin UI template. Some state has also been injected into the template
from the view, providing information necessary for both Django and
Review Board to properly render the Save/Delete buttons for the page
(andThe JavaScript backing the integrations list now makes use of the new
state management and display code for Config Forms list items. This
reduces the work that the template must do, and gets rid of some
now-unnecessary CSS.It also fixes some display issues with modalbox usage when confirming
deletion of an integration configuration.
All Python and JavaScript unit tests pass.
Manually tested the saving and deletion functionality for an integration
configuration on the configuration page and deletion on the integration
list page.
Summary | ID |
---|---|
8cf49c8d6050549ea6a9977bb7391f853774fd5d |
Description | From | Last Updated |
---|---|---|
E999 SyntaxError: invalid syntax |
reviewbot |
- Commits:
-
Summary ID e9ff48f66fcb23841a29dc595f66b50404abd479 8cf49c8d6050549ea6a9977bb7391f853774fd5d - Diff:
-
Revision 2 (+172 -180)