[1.8b5] Commissioning script contents is shown under other settings
Bug #1453730 reported by
Andres Rodriguez
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| MAAS |
Fix Released
|
Critical
|
Blake Rouse | ||
Bug Description
When accessing the settings page, Commissioning scripts contents is being shown under other setting options. However, when I click F5, the contents is shown correctly (see both attached screenshots).
That being said, the contents of the commissioning scripts should not be shown by default. By default, it should be collapsed and we should allow accessing the contents on a drop down. The reason behind this is that if we were to have multiple commissioning scripts, or very large, it would not look appealing.
Related branches
lp://qastaging/~blake-rouse/maas/fix-1453730
- Gavin Panella (community): Approve
- Raphaël Badin (community): Approve
-
Diff: 443 lines (+404/-1)6 files modifiedsrc/maasserver/context_processors.py (+1/-0)
src/maasserver/static/css/maas-styles.css (+1/-1)
src/maasserver/static/js/os_distro_select.js (+155/-0)
src/maasserver/static/js/tests/test_os_distro_select.html (+38/-0)
src/maasserver/static/js/tests/test_os_distro_select.js (+198/-0)
src/maasserver/static/scss/maas/components/_tables.scss (+11/-0)
Revision history for this message
| Andres Rodriguez (andreserl) wrote | #1 |
Revision history for this message
| Andres Rodriguez (andreserl) wrote | #2 |
| no longer affects: | maas (Ubuntu) |
| Changed in maas: | |
| milestone: | none → 1.8.0 |
| importance: | Undecided → High |
| summary: |
- Commissioning script contents is shown under page settings + Commissioning script contents is shown under other settings |
| summary: |
- Commissioning script contents is shown under other settings + [1.8b5] Commissioning script contents is shown under other settings |
| tags: | added: ui |
| Changed in maas: | |
| importance: | High → Critical |
| description: | updated |
| Changed in maas: | |
| status: | New → Triaged |
| Changed in maas: | |
| assignee: | nobody → Richard McCartney (ricgard) |
Revision history for this message
| Blake Rouse (blake-rouse) wrote | #3 |
| Changed in maas: | |
| assignee: | Richard McCartney (ricgard) → Blake Rouse (blake-rouse) |
| Changed in maas: | |
| status: | Triaged → Fix Committed |
| Changed in maas: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
This is actually a JS issue, from me removing to much of the YUI code. I will need to add it back in.