rusackas commented on PR #35044: URL: https://github.com/apache/superset/pull/35044#issuecomment-5553514132
@winterka thanks for the patience on this one, and sorry it dragged. Going back to the original commit, the change itself was tiny and right: the `client_id` placeholder was wrong and the example wanted a black-style cleanup. The catch is that the whole Keycloak / Flask-OIDC section got removed from the docs when we bumped to Flask-AppBuilder 5 (#33055), since that Flask-OpenID based path no longer works there, so the text this fixes isn't in the tree anymore and the merge from `master` is now re-adding the deleted section wholesale. Closing this out rather than course-correcting it. If you're running Keycloak on a current release and have a working Authlib-based setup, a short doc PR for that would be very welcome. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
