-
Notifications
You must be signed in to change notification settings - Fork 48
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
Incompatible with NC14 #215
Comments
fixed next release. |
@daita Is the patch public already? I'd like to do some fixes on the admin settings, but the app is broken on master atm. |
I was just heading to the admin settings thing, but if you want to take care of it, please do. I will release a branch with some other thing later today. |
Yes I'll sure I'll to that once master so ready for 14 👍 |
Error install circles: |
With the new version 0.15.0 the problem has been solved. |
Sorry! I apologize for my previous message. It can be installed but it still does not work correctly... |
Can you confirm you have this line in lib/ShareByCircleProvider.php: circles/lib/ShareByCircleProvider.php Line 718 in a1361a7
|
If I confirm that it is. |
He deleted and reinstalled the application and now it allows me to add mienbros... |
Fixed in 934b4e1 and elsewhere. |
We needed to add some functions to the share provider interface.
For circles this means we should add those functions (even if they don't do anything for now). Else php does 💥
The text was updated successfully, but these errors were encountered: