mgubaidullin commented on code in PR #992:
URL: https://github.com/apache/camel-karavan/pull/992#discussion_r1400560596
##########
karavan-web/karavan-app/src/main/webui/src/api/ProjectService.ts:
##########
@@ -258,7 +258,6 @@ export class ProjectService {
public static copyProject(sourceProject: string, project: Project) {
KaravanApi.copyProject(sourceProject, project, res => {
if (res.status === 200 || res.status === 201) {
- EventBus.sendAlert( 'Success', 'Project copied', 'success');
Review Comment:
We can have UI validation because there might me project created by another
user that is not yet sync with the current one, or not authorized to have
access to. So, verification should be server-side. If copied than server
returns 200 and we show notification, otherwise server return 409 and show
error notofication
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
