Merge lp://qastaging/~ricgard/maas/edit-and-save-subnet-details into lp://qastaging/~maas-committers/maas/trunk
Proposed by
Richard McCartney
Status: | Merged |
---|---|
Approved by: | Richard McCartney |
Approved revision: | no longer in the source branch. |
Merged at revision: | 6097 |
Proposed branch: | lp://qastaging/~ricgard/maas/edit-and-save-subnet-details |
Merge into: | lp://qastaging/~maas-committers/maas/trunk |
Diff against target: |
220 lines (+114/-35) 3 files modified
src/maasserver/static/js/angular/controllers/subnet_details.js (+22/-0) src/maasserver/static/js/angular/controllers/tests/test_subnet_details.js (+20/-0) src/maasserver/static/partials/subnet-details.html (+72/-35) |
To merge this branch: | bzr merge lp://qastaging/~ricgard/maas/edit-and-save-subnet-details |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Mike Pontillo (community) | Approve | ||
Richard McCartney (community) | Needs Resubmitting | ||
Andres Rodriguez (community) | Needs Information | ||
Blake Rouse | Pending | ||
Review via email:
|
Commit message
Remove auto save on blur for the Subnet details summary row. Applied static content when not in edit mode.
Description of the change
- Removes the auto saving on blur for the Subnet details summary row.
- Applied and updates the correct static content for the subnet when not in edit mode
---
TO TEST
- Visit a subnet details page, amend any content and save. Content should have updated accordingly.
To post a comment you must log in.
What about validation and surfacing errors? I would imagine that these forms do not have validation in the backend atm.