Hi all,
This patch implements #1072 (a child of #459), showing all Instances in
a Pool.
A few notes:
- The mockups show Assembly and Deployment in the table as being links.
I added conditional support for making them links when an instance has
an assembly or deployable, though currently none do. The links go to the
ostensibly-correct place, but neither seems to be implemented yet. (For
example, Assemblies#show doesn't even exist yet.) These should come into
existence in the coming days, though.
- I did not use a sortable_table_header, even though they'd have been
nice. Pools already use sortable_table_header, so we ended up with two
on the page. Sorting did not work properly and sometimes threw
exceptions, depending on what you sorted. Additionally, the wrong tab is
loaded after a sort because the sort goes back to 'show'. I started to
try to make it work, but it turned into a great time sink for what I
think is a comparatively unimportant feature. This problem also exists
in other places, such as "Matching Provider Hardware Profiles," so I
think we should try to find a better solution as part of revamping the
UI.
-- Matt