-
Notifications
You must be signed in to change notification settings - Fork 231
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
Set form definition for bind parameters when creating a binding. #2215
Conversation
[merge] |
Evaluated for origin web console merge up to 8c1b12b |
@@ -159,6 +159,7 @@ | |||
ctrl.serviceClassName = instance.spec.serviceClassName; | |||
ctrl.plan = BindingService.getPlanForInstance(instance, ctrl.serviceClass); | |||
ctrl.parameterSchema = _.get(ctrl.plan, 'serviceInstanceCredentialCreateParameterSchema'); | |||
ctrl.parameterFormDefinition = _.get(ctrl.plan, 'externalMetadata.schemas.service_binding.create.openshift_form_definition'); |
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.
We will need to update this when the API changes come ('**spec.**externalMetadata.schemas.service_binding...)
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.
We will need to update this when the API changes come
Updated in #2155
Origin Web Console Merge Results: SUCCESS (https://ci.openshift.redhat.com/jenkins/job/merge_pull_request_origin_web_console/315/) (Base Commit: 9b3fc29) (PR Branch Commit: 8c1b12b) |
Form definition for bind parameters were not being set, rendering input fields out of order.