fix(xo-web/metadata-restore-modal): add check to hide pool select whe…

…n restoring xo config backup
vatesfr · Jan 8, 2024 · 5f9a85c · 5f9a85c
1 parent bb19afc
commit 5f9a85c
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/CHANGELOG.unreleased.md b/CHANGELOG.unreleased.md
@@ -17,6 +17,7 @@
 - [Settings/Logs] Fix `sr.getAllUnhealthyVdiChainsLength: not enough permissions` error with non-admin users (PR [#7265](https://github.com/vatesfr/xen-orchestra/pull/7265))
 - [Settings/Logs] Fix `proxy.getAll: not enough permissions` error with non-admin users (PR [#7249](https://github.com/vatesfr/xen-orchestra/pull/7249))
 - [Replication/Health Check] Fix `healthCheckVm.add_tag is not a function` error [Forum#69156](https://xcp-ng.org/forum/post/69156)
+- [Backup/Restore] Fix unnecessary pool selector in XO config backup restore modal [Forum#8130](https://xcp-ng.org/forum/topic/8130/xo-configbackup-restore) (PR [#7287](https://github.com/vatesfr/xen-orchestra/pull/7287))
 
 ### Packages to release
 

diff --git a/packages/xo-web/src/xo-app/backup/restore/restore-metadata-backups-modal-body.js b/packages/xo-web/src/xo-app/backup/restore/restore-metadata-backups-modal-body.js
@@ -55,9 +55,11 @@ export default class RestoreMetadataBackupModalBody extends Component {
             />
           </Col>
         </Row>
-        <Row className='mt-1'>
-          <SelectPool onChange={this.linkState('pool')} required value={this.state.pool} />
-        </Row>
+        {this.props.type !== 'XO' && (
+          <Row className='mt-1'>
+            <SelectPool onChange={this.linkState('pool')} required value={this.state.pool} />
+          </Row>
+        )}
         {this.props.type !== 'XO' && <SingleLineRow>{restorationWarning}</SingleLineRow>}
       </Container>
     )