Merge ~caleb-ellis/maas:filter-count into maas:master
Status: | Merged |
---|---|
Approved by: | Caleb Ellis |
Approved revision: | 9378c4e0e38fb1be145e6a0c21a73145fad91d30 |
Merge reported by: | MAAS Lander |
Merged at revision: | not available |
Proposed branch: | ~caleb-ellis/maas:filter-count |
Merge into: | maas:master |
Diff against target: |
38 lines (+6/-3) 2 files modified
src/maasserver/static/js/angular/directives/machines_table.js (+4/-2) src/maasserver/static/partials/nodes-list.html (+2/-1) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Anthony Dillon | Approve | ||
MAAS Maintainers | Pending | ||
Review via email: mp+367480@code.launchpad.net |
Commit message
LP: #1829225 - Fix incorrect filter counts in machine list
Description of the change
## Done
- Fixed incorrect filter counts in machine list. This was occurring due to not properly clearing the metadata when loading the machine list, so the counts were getting added to every time you navigated to the machine list. This fix introduces two-way binding for the machine metadata, which is not ideal but ensures that the metadata is always correct.
## QA
- Go to machine list
- Check that the filter dropdown counts are correct
- Navigate to a different page then navigate back to the machine list
- Check that the machines still load in batches
- Check that the filter dropdowns are still correct
LGTM +1