-
Notifications
You must be signed in to change notification settings - Fork 232
Fix runtime error on deploy image page #1888
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
[test] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. However, thinking about this last night (after approving the 3.6 changes), Initializing $scope.forms in the controller rather than the link function would probably fix the errors as well. Sorry for the delayed thoughts.
@jeff-phillips-18 Ah, I think you're probably right. I'll try it this morning and update the PR. I'm inclined to leave the enterprise-3.6 fix alone since I know it works and I've done quite a bit of testing. @benjaminapetersen FYI, see Jeff's comment above. |
The ng-form was not correctly initialized, resulting in a runtime error.
fb088aa
to
2560a9f
Compare
@jeff-phillips-18 Your suggestion worked 👍 Thanks, this looks much cleaner. |
Evaluated for origin web console test up to 2560a9f |
Origin Web Console Test Results: SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_request_origin_web_console/31/) (Base Commit: f62c98f) (PR Branch Commit: 2560a9f) |
@jeff-phillips-18 another look for my updates? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
@jeff-phillips-18 Thanks! [merge][severity: blocker] |
Evaluated for origin web console merge up to 2560a9f |
Origin Web Console Merge Results: SUCCESS (https://ci.openshift.redhat.com/jenkins/job/merge_pull_request_origin_web_console/60/) (Base Commit: 4bcfb4f) (PR Branch Commit: 2560a9f) (Extended Tests: blocker) |
The ng-form was not correctly initialized, resulting in a runtime error.
Cherry picked from branch enterprise-3.6 (#1887)
@rhamilto