-
Notifications
You must be signed in to change notification settings - Fork 36
[WIP] Feature/vm wizard #7
Conversation
|
@iceteabottle thanks for resuming this. One request from last time I tested your component/form, can you add the device mixin to hide the infocard on mobile? |
Add disk size selection for custom disk offering ids.
Change translation.
Done. |
Respect zone selection on loading isos. Fetch isos again on zone id change.
Add tag for self uploaded isos.
|
Nice @iceteabottle - do you think instead we could have an explicit tab of featured templates (use the star icon), rest looks great! |
Thanks. Maybe I could pin the featured above instead of breaking the OS tabs logic. What do you think, @rhtyd ? |
|
@iceteabottle that's good idea as well, so basically show features templates? However, per the old CloudStack templates have a sort order so I don't know if that would break anything esp. in env with large number of templates. cc @PaulAngus @svenvogel ? |
|
Hi @svenvogel in the second screenshot, the order is used to determine the sort order of a resource. CloudStack stores the order in |
|
@rhtyd I wrapped each inner wizard components into a collapsable item to improve the UX. |
|
@utchoang please pick this one next, in consultation with @iceteabottle and @svenvogel. Let me know if you've any questions or need any help. |
|
@rhtyd Today I will add the last major form part "ssh key pairs selection" to finish the main layout stuff. Can you please merge this PR tomorrow? Then @utchoang can continue my work in his own fork. |
|
@utchoang can you resume here please? |
@utchoang please see my checklist #7 (comment) @rhtyd please merge this PR so @utchoang can finish this feature. Thanks for the cooperation in this project. I'm sorry that I have to discontinue my work here. |
|
@iceteabottle Thank you so much. |
|
@rhtyd can you merge that we can proceed? |
|
@utchoang @svenvogel I'll squash merge this per your request, I've not tested it. Just checked and added license header to some files. |
|
@utchoang you can continue on it now, raise a new PR. When you do checkout latest master, ensure it's same as origin master by doing: |
compute: work-in-progress VM deployment wizard (#7)
This implements a work-in-progress VM deployment wizard. Co-authored-by: Rohit Yadav <rohit@apache.org>







Uh oh!
There was an error while loading. Please reload this page.