Skip to content
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

Open
miaulalala opened this issue Feb 13, 2023 · 5 comments
Open

Don't show "Invitation sent" on shared calendars #4983

miaulalala opened this issue Feb 13, 2023 · 5 comments
Assignees
Labels
1. to develop Accepted and waiting to be taken care of bug enhancement New feature request

Comments

@miaulalala
Copy link
Contributor

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

@miaulalala miaulalala added enhancement New feature request 0. to triage Pending approval or rejection labels Feb 13, 2023
@rotdrop
Copy link
Contributor

rotdrop commented Feb 17, 2023

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 {DAV:}share-resource-uri and {DAV:}share-access which might be helpful when trying to identify shard calendars (other than by looking at their URI and searching for "shared_by").

@tcitworld
Copy link
Member

tcitworld commented Feb 21, 2023

One effect is that there are new fields {DAV:}share-resource-uri and {DAV:}share-access which might be helpful when trying to identify shard calendars (other than by looking at their URI and searching for "shared_by").

That's not what we do. There's a shares property for DAV collections using the property exposed by the current sharing backend.

Using Sabre's new fields from nextcloud/server#36756 would be used exactly the same (apart for eventual future invitee status).

@rotdrop
Copy link
Contributor

rotdrop commented Feb 21, 2023

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?

@tcitworld
Copy link
Member

Absolutely.

@ChristophWurst ChristophWurst added 1. to develop Accepted and waiting to be taken care of and removed 0. to triage Pending approval or rejection labels Jun 28, 2023
@ChristophWurst
Copy link
Member

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. to develop Accepted and waiting to be taken care of bug enhancement New feature request
Projects
Status: 📄 To do
Status: 👓 Design review
Development

No branches or pull requests

5 participants