Skip to content
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

[fix-#3843][api]When update workflow definition,first verify the name of the workflow definition, save the data #3930

Closed
wants to merge 1 commit into from

Conversation

zhuangchong
Copy link
Contributor

What is the purpose of the pull request

fix #3843
1.When update workflow definition,first verify the name of the workflow definition, save the data.
2.Be consistent with the 1.3.3-release branch code

Brief change log

udp.vue

@zhuangchong
Copy link
Contributor Author

@break60
done. the code is merged to 1.3.3-release, I update to the dev branch to keep the code consistent.

@sonarcloud
Copy link

sonarcloud bot commented Oct 16, 2020

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@break60
Copy link
Contributor

break60 commented Oct 20, 2020

+1

@CalvinKirs CalvinKirs added the UI ui and front end related label Oct 25, 2020
@zhuangchong
Copy link
Contributor Author

Branch 1.3.3 will be merged into dev, so close this pr.

@zhuangchong zhuangchong closed this Nov 6, 2020
@zhuangchong zhuangchong deleted the dev-fix#3789 branch April 22, 2022 05:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
UI ui and front end related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Improvement][api] When the update workflow definition name already exists, the prompt is not friendly
3 participants