Skip to content

UI: Update AddAccountOrUserToProject to show a drop down for account & user - #9072

Merged
kiranchavala merged 1 commit into
apache:4.19from
shapeblue:ui-update-add-account-to-project
Jun 11, 2024
Merged

kiranchavala merged 1 commit into
apache:4.19from
shapeblue:ui-update-add-account-to-project

Conversation

@vishesh92

@vishesh92 vishesh92 commented May 10, 2024 •

Copy link
Copy Markdown
Member

Description

This PR fixes #7216

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)
  • build/CI

Feature/Enhancement Scale or Bug Severity

Feature/Enhancement Scale

  • Major
  • Minor

Screenshots (if appropriate):

image

image

How Has This Been Tested?

How did you try to break this feature and the system with this change?

@vishesh92
vishesh92 force-pushed the ui-update-add-account-to-project branch from 4bb70bf to db68ab3 Compare May 10, 2024 07:48
@vishesh92 vishesh92 changed the title UI: Update AddAccountOrUserToProject to show a drop down for account … UI: Update AddAccountOrUserToProject to show a drop down for account & user May 10, 2024
@vishesh92

Copy link
Copy Markdown
Member Author

@blueorangutan ui

@blueorangutan

Copy link
Copy Markdown

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

@codecov

codecov Bot commented May 10, 2024 •

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 14.96%. Comparing base (ea9a0f4) to head (db68ab3).
Report is 23 commits behind head on 4.19.

Additional details and impacted files
@@             Coverage Diff              @@
##               4.19    #9072      +/-   ##
============================================
- Coverage     14.96%   14.96%   -0.01%     
+ Complexity    10995    10992       -3     
============================================
  Files          5373     5373              
  Lines        469024   469031       +7     
  Branches      58818    60697    +1879     
============================================
- Hits          70197    70187      -10     
- Misses       391056   391074      +18     
+ Partials       7771     7770       -1     
Flag Coverage Δ
uitests 4.31% <ø> (-0.01%) ⬇️
unittests 15.67% <ø> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@blueorangutan

Copy link
Copy Markdown

UI build: ✔️
Live QA URL: https://qa.cloudstack.cloud/simulator/pr/9072 (QA-JID-339)

@DaanHoogland DaanHoogland left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

clgtm, tested in QA

@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 UI build

Addaccount Adduser

@kiranchavala
kiranchavala merged commit 3321971 into apache:4.19 Jun 11, 2024
@DaanHoogland
DaanHoogland deleted the ui-update-add-account-to-project branch June 11, 2024 06:06
dhslove pushed a commit to ablecloud-team/ablestack-cloud that referenced this pull request Jun 17, 2024
@DaanHoogland DaanHoogland added this to the 4.19.1.0 milestone Sep 6, 2024
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.

Ability to select account and user in the drop down when adding to the project

4 participants