Skip to content

UI fix message.add.vpn.customer.gateway.failed when catched error - #6489

Closed
hooklee2000 wants to merge 1 commit into
apache:mainfrom
hooklee2000:fix-message.add.vpn.customer.gateway.failed-when-catched-error
Closed

hooklee2000 wants to merge 1 commit into
apache:mainfrom
hooklee2000:fix-message.add.vpn.customer.gateway.failed-when-catched-error

Conversation

@hooklee2000

@hooklee2000 hooklee2000 commented Jun 23, 2022

Copy link
Copy Markdown
Contributor

Description
Types of changes

As 4.17.0.0,when catched error, we changed message.success.add.vpn.customer.gateway to
message.add.vpn.customer.gateway.failed.

Bug fix (non-breaking change which fixes an issue)

Feature/Enhancement Scale or Bug Severity
Bug Severity

Minor

when catched error,  we changed message.success.add.vpn.customer.gateway to
message.add.vpn.customer.gateway.failed.
@boring-cyborg

boring-cyborg Bot commented Jun 23, 2022

Copy link
Copy Markdown

Congratulations on your first Pull Request and welcome to the Apache CloudStack community! If you have any issues or are unsure about any anything please check our Contribution Guide (https://github.com/apache/cloudstack/blob/main/CONTRIBUTING.md)
Here are some useful points:

@acs-robot

Copy link
Copy Markdown

Found UI changes, kicking a new UI QA build
@blueorangutan ui

@blueorangutan

Copy link
Copy Markdown

@acs-robot a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

1 similar comment
@blueorangutan

Copy link
Copy Markdown

@acs-robot a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

@blueorangutan

Copy link
Copy Markdown

UI build: ✔️
Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/6489 (SL-JID-1817)

@yadvr

yadvr commented Jun 23, 2022

Copy link
Copy Markdown
Member

@hooklee2000 can you change the base branch of the PR to 4.17 and rebase your PR branch to the 4.17 branch?

@hooklee2000

Copy link
Copy Markdown
Contributor Author

Closed by a new pr #6493

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants