Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(xo-web/new/SR/NFS): disable "No selected value" path option #6137

Merged
merged 9 commits into from
Mar 23, 2022
Merged
2 changes: 2 additions & 0 deletions CHANGELOG.unreleased.md
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@
- [Rolling Pool Update] Don't fail if `load-balancer` plugin is missing (Starter and Enterprise plans)
- [Backup/Restore] Fix missing backups on Backblaze
- [Templates] Fix "incorrect state" error when trying to delete a default template [#6124](https://github.com/vatesfr/xen-orchestra/issues/6124) (PR [#6119](https://github.com/vatesfr/xen-orchestra/pull/6119))
- [New SR] Fix "SR_BACKEND_FAILURE_103" error when selecting "No selected value" for the path [#5991](https://github.com/vatesfr/xen-orchestra/issues/5991) (PR [#6137](https://github.com/vatesfr/xen-orchestra/pull/6137))

### Packages to release

Expand All @@ -43,3 +44,4 @@
- @xen-orchestra/backups minor
- @xen-orchestra/proxy minor
- xo-server patch
- xo-web patch
6 changes: 4 additions & 2 deletions packages/xo-web/src/xo-app/new/sr/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -688,15 +688,17 @@ export default class New extends Component {
<label htmlFor='selectSrPath'>{_('newSrPath')}</label>
<select
className='form-control'
defaultValue={null}
defaultValue=''
id='selectSrPath'
onChange={event => {
this._handleSrPathSelection(event.target.value)
}}
ref='path'
required
>
<option value={null}>{formatMessage(messages.noSelectedValue)}</option>
<option disabled value=''>
{formatMessage(messages.noSelectedValue)}
</option>
{map(paths, (item, key) => (
<option key={key} value={item.path}>
{item.path}
Expand Down