Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

ui: Fix async poll job#5221

Merged
DaanHoogland merged 1 commit intoapache:mainapache/cloudstack:mainfrom
shapeblue:fix-async-job-mainshapeblue/cloudstack:fix-async-job-mainCopy head branch name to clipboard
Jul 21, 2021
Merged

ui: Fix async poll job#5221
DaanHoogland merged 1 commit intoapache:mainapache/cloudstack:mainfrom
shapeblue:fix-async-job-mainshapeblue/cloudstack:fix-async-job-mainCopy head branch name to clipboard

Conversation

@davidjumani
Copy link
Contributor

@davidjumani davidjumani commented Jul 20, 2021

Description

Continuation of #4782 after merging with main

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

  • Major
  • Minor

@davidjumani davidjumani requested a review from utchoang July 20, 2021 05:26
@davidjumani
Copy link
Contributor Author

davidjumani commented Jul 20, 2021

@blueorangutan ui

@blueorangutan
Copy link

blueorangutan commented Jul 20, 2021

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

@blueorangutan
Copy link

blueorangutan commented Jul 20, 2021

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

@davidjumani davidjumani added this to the 4.16.0.0 milestone Jul 20, 2021
utchoang
utchoang approved these changes Jul 20, 2021
View reviewed changes
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

@DaanHoogland DaanHoogland merged commit 84e52c9 into apache:main Jul 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@DaanHoogland DaanHoogland DaanHoogland approved these changes

@utchoang utchoang utchoang approved these changes

Assignees

No one assigned

Labels

component:UI

Projects

None yet

Milestone

4.16.0.0

Development

Successfully merging this pull request may close these issues.

4 participants

@davidjumani @blueorangutan @DaanHoogland @utchoang
Morty Proxy This is a proxified and sanitized view of the page, visit original site.