mvolf commented on code in PR #992:
URL: https://github.com/apache/camel-karavan/pull/992#discussion_r1407313969
##########
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:
@mgubaidullin My plan was to fix this quickly by just throwing the exception
on back-end and handle it in the UI (as agreed in our previous comments) and
after that implement the generic validation framework which I would apply to
create/copy project actions initially, but could be used in the whole project.
I will have the PR for the quick fix at the beginning of the next week,
since I'm busy this week.
Let me know if this is fine with you. If it is, please create a task for the
generic validation mentioned above and assign it to me.
--
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]
