The proposal to merge lp:~dimitern/maas/bug-994887-nodes-listing-does-not-support-sorting into lp:maas has been updated.
Description changed to: Implemented sorting for /MAAS/nodes/ list - by hostname or status (asc/desc). As advised, I pulled only what was needed from the project django-sortable and I'm using it in the views/templates. Since django-sortable is not in main archive, we cannot use it, but it's only a tiny package, so I extracted what was needed and added it to maasserver (also did some style formatting so it'll conform to our code). There are a couple of very minor UI changes, which I was advised also to request a separate review from Huw Wilkins. The added elements are 2 up/down arrows appearing on the right of the two table columns - MAC and Status. Here are screenshots how it looks in all states: http://people.canonical.com/~dimitern/maas-nodes-sort-none.png http://people.canonical.com/~dimitern/maas-nodes-sort-mac-asc.png http://people.canonical.com/~dimitern/maas-nodes-sort-mac-desc.png http://people.canonical.com/~dimitern/maas-nodes-sort-status-asc.png http://people.canonical.com/~dimitern/maas-nodes-sort-status-desc.png For more details, see: https://code.launchpad.net/~dimitern/maas/bug-994887-nodes-listing-does-not-support-sorting/+merge/130600 -- https://code.launchpad.net/~dimitern/maas/bug-994887-nodes-listing-does-not-support-sorting/+merge/130600 Your team Launchpad code reviewers is requested to review the proposed merge of lp:~dimitern/maas/bug-994887-nodes-listing-does-not-support-sorting into lp:maas. _______________________________________________ Mailing list: https://launchpad.net/~launchpad-reviewers Post to : [email protected] Unsubscribe : https://launchpad.net/~launchpad-reviewers More help : https://help.launchpad.net/ListHelp
