-
Notifications
You must be signed in to change notification settings - Fork 37
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: [WD-14036] Duplicate instance (lxc copy) #849
feat: [WD-14036] Duplicate instance (lxc copy) #849
Conversation
ac36cc3
to
e21e0f2
Compare
So I'm getting an error where it says it can't find the instance I'm duplicating in the project I've selected. Is it possible that the project dropdown is being used to look for the source instance, rather than specifying the destination? CleanShot.2024-08-15.at.20.58.31.mp4 |
Some observations
|
e21e0f2
to
bf45e63
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.
Thanks for the adjustments, this looks much cleaner already. Two further suggestions below.
bf45e63
to
0aba060
Compare
Alrighty, that's all been covered, @edlerd , please let me know what you think :) |
Hey, Piper,
|
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.
LGTM
383a48c
to
667b622
Compare
@Kxiru ,
|
667b622
to
f3c6e0f
Compare
f3c6e0f
to
73040f8
Compare
73040f8
to
8ffba4a
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.
One last comment on the hook for the deduplication, and there is the open discussion on the CSS change, then this should be good from my side.
@Kxiru just a few minor comments from my end and also good to go, great work! :) |
8ffba4a
to
567376c
Compare
168aaed
to
1b5f68b
Compare
Signed-off-by: Nkeiruka <[email protected]>
1b5f68b
to
064f454
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.
LGTM, thanks for adding the duplication!
Done
QA
Screenshots