fix(xo-web/health): make "Too many snapshots" table sortable by numbe…

…r of snaphots (#6255) See zammad#6439
vatesfr · May 31, 2022 · ae0eb9e · ae0eb9e
1 parent 0521266
commit ae0eb9e
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/CHANGELOG.unreleased.md b/CHANGELOG.unreleased.md
@@ -14,6 +14,7 @@
 > Users must be able to say: “I had this issue, happy to know it's fixed”
 
 - [load-balancer] Fix density mode failing to shutdown hosts (PR [#6253](https://github.com/vatesfr/xen-orchestra/pull/6253))
+- [Health] Make "Too many snapshots" table sortable by number of snapshots (PR [#6255](https://github.com/vatesfr/xen-orchestra/pull/6255))
 
 ### Packages to release
 
@@ -31,5 +32,6 @@
 
 - @xen-orchestra/xapi minor
 - xo-server minor
+- xo-web patch
 
 <!--packages-end-->
diff --git a/packages/xo-web/src/xo-app/dashboard/health/index.js b/packages/xo-web/src/xo-app/dashboard/health/index.js
@@ -397,6 +397,7 @@ const TOO_MANY_SNAPSHOT_COLUMNS = [
     default: true,
     name: _('numberOfSnapshots'),
     itemRenderer: vm => vm.snapshots.length,
+    sortCriteria: vm => vm.snapshots.length,
     sortOrder: 'desc',
   },
 ]