Skip to content

Fix problem where UI appears unresponsive while waiting for websocket update#1959

Merged
openshift-bot merged 1 commit intoopenshift:masterfrom
benjaminapetersen:bpeterse/issue/1875-resume-rollouts-stalled-ui
Oct 6, 2017
Merged

Fix problem where UI appears unresponsive while waiting for websocket update#1959
openshift-bot merged 1 commit intoopenshift:masterfrom
benjaminapetersen:bpeterse/issue/1875-resume-rollouts-stalled-ui

Conversation

@benjaminapetersen
Copy link
Contributor

Fix #1875
@spadgett

(aside, i think the dist having something to the effect of return !0 for return true is kinda funny... optimize out those 2 characters 😄 )

@spadgett
Copy link
Member

We also have the issue on the deployment config and deployment pages

Copy link
Member

@spadgett spadgett left a comment

Choose a reason for hiding this comment

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

Needs an update to the deployment config and deployment pages as well.

@benjaminapetersen benjaminapetersen force-pushed the bpeterse/issue/1875-resume-rollouts-stalled-ui branch from 2a81b7f to 2623975 Compare September 28, 2017 19:36
@benjaminapetersen
Copy link
Contributor Author

Looking at these.

@benjaminapetersen benjaminapetersen force-pushed the bpeterse/issue/1875-resume-rollouts-stalled-ui branch from 2623975 to 620ab1e Compare September 29, 2017 14:15
@benjaminapetersen
Copy link
Contributor Author

Updated.
Interested in reducing the code duplication & reusing alerts here, but prob not in scope for a bug fix. Something like:

  <alerts 
    ng-if="deployment.spec.paused && !updatingPausedState"
    alerts="pausedAlert"></alerts>

@benjaminapetersen
Copy link
Contributor Author

This can go today as well, if we can wake up the merge queue...

@spadgett
Copy link
Member

spadgett commented Oct 6, 2017

[merge]

@openshift-bot
Copy link

Evaluated for origin web console merge up to 620ab1e

@openshift-bot
Copy link

openshift-bot commented Oct 6, 2017

Origin Web Console Merge Results: SUCCESS (https://ci.openshift.redhat.com/jenkins/job/merge_pull_request_origin_web_console/322/) (Base Commit: ec0a953) (PR Branch Commit: 620ab1e)

@openshift-bot openshift-bot merged commit 2e0fbe8 into openshift:master Oct 6, 2017
@benjaminapetersen benjaminapetersen deleted the bpeterse/issue/1875-resume-rollouts-stalled-ui branch October 9, 2017 14:22
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.

3 participants