- Issue created by @JeroenT
- @jeroent opened merge request.
- Status changed to Needs review
over 1 year ago 8:10am 9 June 2023 - 🇧🇪Belgium JeroenT 🇧🇪
The MR fixes 3 things:
- The config schema as still the config schema for the old version. I updated this so it works with the provider config entities.
Before:
After
Still 1 error left, but the error is about the redirect_url field, which is provided by openid_connect and shouldn't be saved in config. So we should fix that in openid_connect.
- Hook
keycloak_form_openid_connect_admin_settings_alter()
was no longer called since openid_connect switched the providers to config entities. This resulted in config that was saved in the wrong place and validation was no longer running. I was able to remove the logic from this hooks and move it to the provider plugin. - Form states were no longer working and all the fields were always shown. The structure of the form was changed, so I had to update the selectors.
- The config schema as still the config schema for the old version. I updated this so it works with the provider config entities.
- Status changed to RTBC
over 1 year ago 4:16pm 9 June 2023 - 🇧🇪Belgium BramDriesen Belgium 🇧🇪
Looks great! Quickly tested this and the form indeed saves correctly now. Didn't notice that before.
-
BramDriesen →
committed f64e2ace on 2.2.x authored by
JeroenT →
Issue #3365802 by JeroenT: Keycloak config form stopped working...
-
BramDriesen →
committed f64e2ace on 2.2.x authored by
JeroenT →
- Status changed to Fixed
over 1 year ago 4:18pm 9 June 2023 - 🇧🇪Belgium BramDriesen Belgium 🇧🇪
Also wondering what module or tool you used in your screenshots 😉
- 🇧🇪Belgium JeroenT 🇧🇪
@BramDriesen,
Automatically closed - issue fixed for 2 weeks with no activity.