Merge ~deadlight/maas:remove-repo into maas:master
Proposed by
Karl Williams
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Blake Rouse | ||||
Approved revision: | eb563d1987b41b2900b16db26e6daee75cd0d809 | ||||
Merge reported by: | MAAS Lander | ||||
Merged at revision: | not available | ||||
Proposed branch: | ~deadlight/maas:remove-repo | ||||
Merge into: | maas:master | ||||
Diff against target: |
49 lines (+11/-9) 1 file modified
src/maasserver/static/partials/settings.html (+11/-9) |
||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Blake Rouse (community) | Approve | ||
Anthony Dillon | Approve | ||
MAAS Lander | Approve | ||
Review via email: mp+337938@code.launchpad.net |
Commit message
Fixes LP #1749867 - Allow removal of custom repositories
Make the confirmation buttons appear on /settings/
Description of the change
Make the confirmation buttons appear on /settings/
QA:
- Go to /settings/
- Add a repo other than the default ones
- Click the "remove" button on this new row
- See that there is a confirmation form
- Click remove
- See that the repo is removed
To post a comment you must log in.
UNIT TESTS
-b remove-repo lp:~deadlight/maas/+git/maas into -b master lp:~maas-committers/maas
STATUS: SUCCESS 900b16db26e6dae e75cd0d809
COMMIT: eb563d1987b41b2