-
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
Calendar: Remove valarms from objects in read-only shared calendars #24377
Comments
Hello, we managed to solve this issue for owncloud 9.0 calendar. We have tested it a bit but it still needs testing. I don't know how to contribute code to the apps, so I just drag and drop the file here. This is a modification of the the file apps/dav/lib/caldav/caldavbackend.php caldavbackend.php_oc90-no-Alarm-For-ReadOnly-Shared-Calender.txt Thanks to Katinka for initial code! |
@vmuth I've tested your solution a bit today on OC 9.0.7 but am running into problems, have you gotten any further with this yourself by any chance? |
Hi @aartsr, sorry for the late reply. Which problems do you have with this code? How different is the caldavbackend.php from your original? There should only be differences in line 536, 586 and between 1396 - 1425. |
Hi @vmuth thank you for your reply. I've used the exact code in the txt-file you provided above in combination with an OC 9.0.7 test install at Hostiso, which uses a private smtp server for e-mail notifications (tested to work). I created an event in calendar X which is shared with user a (read-only) and user b (edit rights). Behaviour observed in the OC web interface:
Behaviour observed in both iPhone calendar and Outlook/Caldav Synchronizer:
I'm guessing the above behaviour is not what you're experiencing? May I ask what OC version seems to be working for you? Any tips are welcome. |
@georgehrke
Back in the old calendar app there was an open pull request made by you that removed the alarm from events in read-only sharedy calendars.
You postponed this PR due the calendar on OC9 uses another caldav implementation.
Is something like that already possible in OC9 or is it planned?
Old PR is: owncloud-archive/calendar#668
Old issue: https://github.com/owncloudarchive/calendar/issues/547
The text was updated successfully, but these errors were encountered: