-
Notifications
You must be signed in to change notification settings - Fork 255
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
feat(xo-web,xo-server): implement ISO import #6180
Conversation
3b0b036
to
25fb509
Compare
23a8b03
to
d152470
Compare
7eb01eb
to
e54b968
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't forget to mention xo-server
and xo-web
in the commit message's context: feat(xo-server,xo-web/import): ...
@@ -1584,7 +1584,7 @@ const messages = { | |||
// ---- Disk import --- | |||
diskImportFailed: 'Disk import failed', | |||
diskImportSuccess: 'Disk import success', | |||
dropDisksFiles: 'Drop VMDK or VHD files here to import disks.', | |||
dropDisksFiles: 'Drop ISO, VMDK or VHD files here to import disks.', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Other languages need to be updated too.
|
||
return ( | ||
<ReactDropzone | ||
activeClassName={styles.activeDropzone} | ||
className={styles.dropzone} | ||
multiple={multiple} | ||
onDrop={onDrop} | ||
accept={accept} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Keep the props sorted.
@@ -108,12 +108,16 @@ const DiskImport = decorate([ | |||
<Row> | |||
<LabelCol>{_('importToSr')}</LabelCol> | |||
<InputCol> | |||
<SelectSr onChange={effects.onChangeSr} required value={sr} /> | |||
<SelectSr onChange={effects.onChangeSr} required value={sr} allowIsoSr /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why don't you pass predicate={isSrWritableOrIso}
instead of handling a rare case in the component itself?
4331d10
to
0279dbd
Compare
7c59a93
to
0279dbd
Compare
it would be really nice to have the import possible via URL also. Like it is already possible with virtual machines |
@prononext Now available in the REST API: https://xen-orchestra.com/blog/xen-orchestra-5-73/#%F0%9F%93%A1-rest-api-iso-import |
Check list
Fixes #007
orSee xoa-support#42
)CHANGELOG.unreleased.md
:${name} v${new version}
)cron/parse.spec.js
)xo-server
API changes, the corresponding test has been added to/updated onxo-server-test
Process
WiP:
(Work in Progress) if not ready to be mergedFrom the Four Agreements: