-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
CHE-4670: make copy file operation more informative #4926
Conversation
@@ -544,7 +544,9 @@ private NewProjectConfigDto asDto(MutableProjectConfig config) { | |||
checkArgument(!source.getLocation().isRoot(), "Workspace root is not allowed to be copied"); |
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.
translate this statement with using return promises.reject(...)
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.
In this class we have many validations like this:
checkArgument(expression, errorMessage)
So, I think we should change many of them but not in this issue.
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.
then ok
Build success. https://ci.codenvycorp.com/job/che-pullrequests-build/2518/ |
@vparfonov For this kind of PR, you need to notify PM. and request approval Could we change the messages to:
|
What does this PR do?
Makes copy file operation more informative.
If a file with its name already exists the window will be shown:
What issues does this PR fix or reference?
#4670
Changelog
Make copy file operation more informative