Skip to content

Conversation

@shwstppr
Copy link
Contributor

@shwstppr shwstppr commented Jun 1, 2021

Description

Refactor code getting API params for APIs in UI forms.
Added a new util plugin in plugins.js

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?

@shwstppr
Copy link
Contributor Author

shwstppr commented Jun 1, 2021

Based on testing and feasibility either of this PR or #5063 can be merged.

@blueorangutan ui

@shwstppr shwstppr force-pushed the refactor-ui-forms-getapiparams branch from 0abb5de to b4a71c7 Compare June 1, 2021 20:11
@apache apache deleted a comment from blueorangutan Jun 1, 2021
@apache apache deleted a comment from blueorangutan Jun 1, 2021
Refactor code getting api params for APIs in UI forms.
Added a new util plugin in plugins.js

Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
@shwstppr shwstppr force-pushed the refactor-ui-forms-getapiparams branch from b4a71c7 to 0978f4f Compare June 1, 2021 20:35
@apache apache deleted a comment from blueorangutan Jun 1, 2021
@apache apache deleted a comment from blueorangutan Jun 1, 2021
@shwstppr
Copy link
Contributor Author

shwstppr commented Jun 1, 2021

@blueorangutan ui

@blueorangutan
Copy link

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

@blueorangutan
Copy link

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

Copy link

@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 mentioned this pull request Jun 2, 2021
12 tasks
Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
@shwstppr
Copy link
Contributor Author

shwstppr commented Jun 2, 2021

@blueorangutan ui

@blueorangutan
Copy link

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

shwstppr added 2 commits June 18, 2021 18:10
Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
@shwstppr
Copy link
Contributor Author

@blueorangutan ui

@blueorangutan
Copy link

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

@shwstppr shwstppr removed the type:bug label Jun 18, 2021
@blueorangutan
Copy link

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

@rohityadavcloud
Copy link
Member

@shwstppr is this ready for review/testing/merge?

@shwstppr
Copy link
Contributor Author

@rhtyd yes, this is ready

@rohityadavcloud rohityadavcloud added this to the 4.15.2.0 milestone Jul 15, 2021
@rohityadavcloud rohityadavcloud merged commit 0406e27 into apache:4.15 Jul 15, 2021
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