-
Notifications
You must be signed in to change notification settings - Fork 33
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
Support NC25 - NC27 + bug fixing. #191
Comments
Maybe consider making a proper fork? Ideally of course someone from the Nextcloud team would add you as a contributor to this repo. Thanks for updating this very useful app :) |
Well, I think they added me some time ago to their Nextcloud organization in github, at least in the Github UI it looks like I could merge the PR by myself. I'll have a look later, still in progress of moving to another residence ... |
Appears the pull-requests for this have been merged, but no new release is available. |
@rotdrop Sorry for this basic questions: is there a way to install your fork ? Manually by FTP ? |
I just updated my NC23 to NC27 and installed this plugin via git (https://github.com/nextcloud/user_sql.git) and it works! Thanks a lot. I hope the project is going to be continued. 👍 |
I just test-updated our NC25 instance to NC26.0.12 after a updated mail_roundcube app is being available. I ran into an issue using user_sql. The admin interface is working as before but user login is not possible. In the nextcloud.log file there are errors as the following: "App user_sql threw an error during app.php load: Interface "OCP\Group\Backend\ISearchableGroupBackend" not found" Any chance to get this working on NC26 upwards? |
I tried update our NC25 instance again to latest NC26. User_sql was freshly cloned from this GIT repo. The error persists. I than upgraded to NC 27.1.2. I didn´t had to loose anything. This time the SQL auth plugin works and does not throw errors. As NC26 goes EOL this month I will not investigate any further. NC27 seems to work. All in all sorry for the noise and keep up the good work. |
I guess this could be closed as #190 was merged... |
See PR #190.
The text was updated successfully, but these errors were encountered: