fix(xo-web/backup): cleanup settings correctly when deselecting healt…

…h check (#6515) Fix #6501
vatesfr · Nov 8, 2022 · 9e87a88 · 9e87a88
1 parent 12e98bf
commit 9e87a88
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 0 deletions.
diff --git a/CHANGELOG.unreleased.md b/CHANGELOG.unreleased.md
@@ -20,6 +20,7 @@
 - [Pool] Add tooltip on "no XCP-ng Pro support" warning icon (PR [#6505](https://github.com/vatesfr/xen-orchestra/pull/6505))
 - [Backup] Respect HTTP proxy setting when connecting to XCP-ng/XenServer pools
 - [Dashboard/Health] Fix `an error has occurred` in case of unknown default SR (PR [#6508](https://github.com/vatesfr/xen-orchestra/pull/6508))
+- [Backup] Really disable Healthcheck when unchecking settings [#6501](https://github.com/vatesfr/xen-orchestra/issues/6501) (PR [#6515](https://github.com/vatesfr/xen-orchestra/pull/6515))
 
 ### Packages to release
 

diff --git a/packages/xo-web/src/xo-app/backup/new/new-schedule.js b/packages/xo-web/src/xo-app/backup/new/new-schedule.js
@@ -80,6 +80,7 @@ const New = decorate([
       toggleHealthCheck({ setSchedule }, { target: { checked } }) {
         setSchedule({
           healthCheckVmsWithTags: checked ? [] : undefined,
+          healthCheckSr: checked ? this.state.healthCheckSr : undefined,
         })
       },
       setHealthCheckSr({ setSchedule }, sr) {