Skip to content

ui: fix add management ip range form#5427

Merged
yadvr merged 1 commit into
apache:4.15from
shapeblue:fix-ui-addmanagementiprange
Sep 10, 2021
Merged

ui: fix add management ip range form#5427
yadvr merged 1 commit into
apache:4.15from
shapeblue:fix-ui-addmanagementiprange

Conversation

@shwstppr
Copy link
Copy Markdown
Contributor

@shwstppr shwstppr commented Sep 9, 2021

Description

Add management IP range form shows the same pod multiple times when the pod has multiple IP ranges.
This PR fixes the behaviour.

Types of changes

  • Breaking change (fix or feature that would cause existing functionality to change)
  • New feature (non-breaking change which adds functionality)
  • Bug fix (non-breaking change which fixes an issue)
  • Enhancement (improves an existing feature and functionality)
  • Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity

Feature/Enhancement Scale

  • Major
  • Minor

Bug Severity

  • BLOCKER
  • Critical
  • Major
  • Minor
  • Trivial

Screenshots (if appropriate):

Before:
Screenshot from 2021-09-09 15-04-47

After:
Screenshot from 2021-09-09 15-03-59

How Has This Been Tested?

@shwstppr
Copy link
Copy Markdown
Contributor Author

shwstppr commented Sep 9, 2021

@blueorangutan ui

@blueorangutan
Copy link
Copy Markdown

@shwstppr 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/5427 (SL-JID-621)

Copy link
Copy Markdown

@utchoang utchoang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@shwstppr shwstppr changed the base branch from main to 4.15 September 10, 2021 13:00
@shwstppr shwstppr changed the base branch from 4.15 to main September 10, 2021 13:01
Add management IP range form shows same pod multiple times when the pods has multiple IP ranges.
This PR fixes the behaviour.

Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
@shwstppr shwstppr force-pushed the fix-ui-addmanagementiprange branch from 0957eab to 56092fd Compare September 10, 2021 13:04
@shwstppr shwstppr changed the base branch from main to 4.15 September 10, 2021 13:04
@yadvr
Copy link
Copy Markdown
Member

yadvr commented Sep 10, 2021

@blueorangutan ui

1 similar comment
@shwstppr
Copy link
Copy Markdown
Contributor Author

@blueorangutan ui

@blueorangutan
Copy link
Copy Markdown

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

Copy link
Copy Markdown
Member

@weizhouapache weizhouapache left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

code lgtm

this seems to be a blocker issue

@blueorangutan
Copy link
Copy Markdown

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

@yadvr yadvr added this to the 4.15.2.0 milestone Sep 10, 2021
@yadvr yadvr merged commit 8baf384 into apache:4.15 Sep 10, 2021
@yadvr
Copy link
Copy Markdown
Member

yadvr commented Sep 10, 2021

Tested LGTM

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants