-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
upgrade sequence ... #23913
Comments
TODO:
|
/me subscribes CC @VicDeo |
This does only work for internal users. LDAP users are not counted because the check for upgrade happens before loading the authentication apps. 🙈 |
I know - but we cannot load all auth apps before performing this check - test if ldap is enabled and assume this is a big installation? |
What about user_shibboleth? I guess we need to do some tricks here before we have the general user table (but that is 9.2). I'm fine with hard coding those two apps. |
@VicDeo @DeepDiver1975 tick more boxes in #23913 (comment) ? |
@VicDeo please update |
moved to backlog or close ? |
closing as obsolete |
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
current sequence
The text was updated successfully, but these errors were encountered: