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

Accept/decline/tentative actions shown for (read-only) shared calendar #4097

Closed
ChristophWurst opened this issue Mar 22, 2022 · 1 comment · Fixed by #4101
Closed

Accept/decline/tentative actions shown for (read-only) shared calendar #4097

ChristophWurst opened this issue Mar 22, 2022 · 1 comment · Fixed by #4101
Assignees
Labels
3. to review Waiting for reviews bug nextcloud-gmbh Tickets with importance for Nextcloud Gmbh

Comments

@ChristophWurst
Copy link
Member

ChristophWurst commented Mar 22, 2022

Steps to reproduce

  1. As user A create a calendar as user A
  2. As user A Share the calendar read-only with user B
  3. As user B click on the event

Expected behavior

There is nothing to click or change for user B.

Actual behaviour

User B can click Accept/Decline/Tentative on the read-only event.

Bildschirmfoto von 2022-03-22 13-51-52

In contrast I can update participation on the copy in my personal calendar.

If the calendar is shared with write permissions, the update works, but I wonder if we shouldn't hide the buttons as well. This might be debatable.

Calendar app version

3.2.2

CalDAV-clients used

No response

Browser

No response

Client operating system

No response

Server operating system

No response

Web server

No response

Database engine version

No response

PHP engine version

No response

Nextcloud version

No response

Updated from an older installed version or fresh install

No response

List of activated apps

No response

Nextcloud configuration

No response

Web server error log

No response

Log file

No response

Browser log

No response

Additional info

This is an edge case of the new feature #144. cc @st3iny

@ChristophWurst ChristophWurst added bug 0. to triage Pending approval or rejection 1. to develop Accepted and waiting to be taken care of and removed 0. to triage Pending approval or rejection labels Mar 22, 2022
@ChristophWurst ChristophWurst moved this to 📄 To do (5-20 entries) in 💌 📅 👥 Groupware team Mar 22, 2022
@ChristophWurst ChristophWurst added the nextcloud-gmbh Tickets with importance for Nextcloud Gmbh label Mar 22, 2022
@st3iny st3iny self-assigned this Mar 24, 2022
@st3iny st3iny added 2. developing Work in progress and removed 1. to develop Accepted and waiting to be taken care of labels Mar 24, 2022
@st3iny
Copy link
Member

st3iny commented Mar 24, 2022

If the calendar is shared with write permissions, the update works, but I wonder if we shouldn't hide the buttons as well.

I tested this edge case and it works fine. The response is propagated correctly to both events if either of the two is updated.

@st3iny st3iny added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Mar 24, 2022
@ChristophWurst ChristophWurst moved this from 📄 To do (5-20 entries) to 🏗️ In progress in 💌 📅 👥 Groupware team Mar 24, 2022
Repository owner moved this from 🏗️ In progress to ☑️ Done in 💌 📅 👥 Groupware team Mar 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews bug nextcloud-gmbh Tickets with importance for Nextcloud Gmbh
Projects
Development

Successfully merging a pull request may close this issue.

2 participants