-
-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
Move iMIP to its own interface #34893
Conversation
Should the IManager's methods also be extracted? Same issue here, all implementations of IManager now need to support iMIP. |
IManager is a consumed/injected service. The implementation is done by the server. This is fine. The breaking change only occurs when you add methods to interfaces that are implemented by apps. |
4590e18
to
f166d4b
Compare
Documentation PR here: nextcloud/documentation#9269 |
and clean up the code a bit. Added: - author Removed: - unused imports Changed: - moved iMIP handling to new interface - pointed code to new implementation Signed-off-by: Anna Larch <anna@nextcloud.com>
e5405d0
to
aebcc31
Compare
Drone failure unrelated |
Drone failure was related, the dumped autoloader needs an update ;) |
PR to fix in #34954 |
and clean up the code a bit.
The old implementation forced every writable calendar to support iMIP.
Added:
Removed:
Changed:
Documentation: nextcloud/documentation#9269