Skip to content

ui: show vpn users tab in networks section - #7277

Merged
shwstppr merged 3 commits into
apache:4.17from
shapeblue:ui-show-vpn-users
Apr 11, 2023
Merged

shwstppr merged 3 commits into
apache:4.17from
shapeblue:ui-show-vpn-users

Conversation

@shwstppr

@shwstppr shwstppr commented Feb 22, 2023 •

Copy link
Copy Markdown
Contributor

Description

Fixes #7248

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):

How Has This Been Tested?

Fixes apache#7248

Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
@kiranchavala

Copy link
Copy Markdown
Member

@blueorangutan ui

@blueorangutan

Copy link
Copy Markdown

@kiranchavala 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: https://qa.cloudstack.cloud/simulator/pr/7277 (QA-JID-54)

@kiranchavala

Copy link
Copy Markdown
Member

@shwstppr LGTM

vpn-user

@shwstppr
shwstppr marked this pull request as ready for review April 3, 2023 10:17
@shwstppr

shwstppr commented Apr 3, 2023

Copy link
Copy Markdown
Contributor Author

@blueorangutan ui

@shwstppr
shwstppr requested review from weizhouapache and yadvr April 3, 2023 10:17

@weizhouapache weizhouapache left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

code lgtm

@DaanHoogland

Copy link
Copy Markdown
Contributor

@blueorangutan ui

@blueorangutan

Copy link
Copy Markdown

@DaanHoogland 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: https://qa.cloudstack.cloud/simulator/pr/7277 (QA-JID-75)

@DaanHoogland

Copy link
Copy Markdown
Contributor

I think something is wrong with the qa env, I can't log on to verify (deal with that some other time)

@shwstppr shwstppr closed this Apr 11, 2023
@shwstppr shwstppr reopened this Apr 11, 2023
@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.

@kiranchavala kiranchavala left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM , tested the changes and its displaying the vpn users correctly

@shwstppr
shwstppr merged commit 55e35c6 into apache:4.17 Apr 11, 2023
@shwstppr shwstppr mentioned this pull request Apr 11, 2023
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.

5 participants