-
Notifications
You must be signed in to change notification settings - Fork 21
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
Add bind param data for autofill params #84
Conversation
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.
looks ok to me
1a847a5
to
39b4b6c
Compare
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.
Verified 👍
- Client ID correctly filled and public client type selected
- Service Instance ID correctly filled and bearer client type selected
@sedroche Is that value there the service Id? Can we change that bind param description to something like |
@@ -19,7 +19,13 @@ plans: | |||
- name: default | |||
description: Deploy keycloak | |||
free: True | |||
metadata: {} | |||
metadata: | |||
mobileclient_bind_parameters_data: |
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.
Can we delete this line here
Line 62 in 39b4b6c
default: public |
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.
That is still needed for bindings that are not created through our custom UI. This only takes effect through our integrations UI.
@sedroche Yeah something along those lines since the value there is not the service name anymore. 👍 |
39b4b6c
to
be8b38c
Compare
Ticket: https://issues.jboss.org/browse/AEROGEAR-2413
Adding new property on metadata that will be used to auto-populate binding params form.
Setting a type of
path
will retrieve the value from the resource.Setting a type of
default
will set that valueVerification
Use this UI branch -> aerogear-attic/origin-web-console#38
Client binding
Client Id should be filled and keycloak client type of
public
should be selected.Service binding
Service Id should be filled and keycloak client type of
bearer
should be selected.