-
Notifications
You must be signed in to change notification settings - Fork 241
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
Don't show "Invitation sent" on shared calendars #4983
Comments
The related issue about the not working invitations is this: nextcloud/server#26668 I am not really into the calendar app's source code, but AFAIK most of the app is a CalDAV client running in the web-browser. This means that it might need some support from the Nextcloud DAV backend in order to identify the shared calendars (ok, you can also look at the ...shared_by... string in their URI). In the draft pull-request nextcloud/server#36756 there is also a commit which implements the SharingSupport interface in the backend. One effect is that there are new fields |
That's not what we do. There's a Using Sabre's new fields from nextcloud/server#36756 would be used exactly the same (apart for eventual future invitee status). |
So then this bug could easily be fixed as the calendar app already knows which calendars are shared and which not? |
Absolutely. |
It happened again that someone was misled by Invitation sent when in fact nothing was done by the backend. I suggest we try to reflect the RSVP/invitation status as accurate as possible according to https://icalendar.org/CalDAV-Scheduling-RFC-6638/3-2-9-schedule-status-values.html. E.g. show when an invitation should have been sent (from the user's perspective) but wasn't. Even catch this and show a little warning toast when the user saves their changes. |
Is your feature request related to a problem? Please describe.
Since invitations don't work for shared calendars atm, remove the misleading "Invitation sent" when adding a participant.
Describe the solution you'd like
No response
Describe alternatives you've considered
No response
Additional context
No response
The text was updated successfully, but these errors were encountered: