calendar icon indicating copy to clipboard operation
calendar copied to clipboard

Allow shared calendars as appointment conflict calendars

Open ChristophWurst opened this issue 3 years ago • 16 comments

Is your feature request related to a problem? Please describe.

As a user I might have calendars shared to me by other users, e.g. a company-wide calendar.

Describe the solution you'd like

It would be nice if I could select that shared calendar for appointment conflicts as well. Then company-wide events would also block appointment slots.

Describe alternatives you've considered

N/a

Additional context

  • [ ] https://github.com/nextcloud/server/issues/30068

ChristophWurst avatar Dec 02 '21 17:12 ChristophWurst

in general, being able to chose which calendars are treated as conflict calendars for appointments would be useful!

violoncelloCH avatar Dec 15 '21 13:12 violoncelloCH

in general, being able to chose which calendars are treated as conflict calendars for appointments would be useful!

Coming with v3.1 https://github.com/nextcloud/calendar/pull/3770

ChristophWurst avatar Dec 15 '21 14:12 ChristophWurst

I was about to make a feature request similar to this, but am going to comment here instead.

In our case most often Appointments are to be used for arranging meetings with an external party (who can't see shared calendars). For this reason it would be most useful to have an Appointments URL that can be a combination of the public Appointment calendars of several users.

For example, if several people in the company are trying to arrange an appointment with some external party you would ideally send them one link which provides the appointments available that work for all three employees. This would be really handy.

This seems different from what you propose because one has to set up a new Appointments set for every combination of users to be present at a meeting.

reos-rcrozier avatar Mar 04 '22 15:03 reos-rcrozier

would this also apply to subscription calendars as well? Or just calendars shared within nextcloud?

Blechlawine avatar May 08 '22 22:05 Blechlawine

@Blechlawine if you're replying to me, I was only thinking of calendars shared within Nextcloud, but actually I mean the Appointments sets shared within Nextcloud,

My idea is users can set up their own appointments sets, just as one can now, and I could somehow select from them to make the combined appointments set for several people.

reos-rcrozier avatar May 08 '22 22:05 reos-rcrozier

@reos-rcrozier I was replying to op, but what your're describing sounds very useful as well

Blechlawine avatar May 08 '22 23:05 Blechlawine

I'm sure there are cases where subscriptions as conflict source make sense :+1:

I just don't know if the change is in scope for this ticket when someone picks it up. We can re-evaluate afterwards and create a another ticket if not.

ChristophWurst avatar May 09 '22 07:05 ChristophWurst

We have an organization wide calendar with team meetings which is shared by an organization-admin account. That should be definitely be taken into account when checking for conflicts to offer free time slots.

rasos avatar Feb 07 '23 11:02 rasos

It seems like the server isse was fixed in NC 24.0.8 and 25.0.2 - does this mean that we can revisit this issue now? See https://github.com/nextcloud/server/issues/30068

pjrobertson avatar Apr 19 '24 01:04 pjrobertson

https://github.com/nextcloud/server/issues/30068 was closed by a script

ChristophWurst avatar Apr 19 '24 08:04 ChristophWurst

You're right, I missed that. I was trying to investigate this today. It seems like it should actually be possible right now though, without changing the server.

Right now the calendars are taken from selectableConflictCalendars() here. This in turn gets its base calendars to filter from the store getter ownSortedCalendars().

Could we not just add a new getter/get the base calendars and then make sure not to filter out the .filter(calendar => !calendar.isSharedWithMe). I'll submit a PR with this thinking

pjrobertson avatar Apr 19 '24 08:04 pjrobertson

Try but I don't think it will work. The backend logic will fail to access those calendars and therefore conflicts are not detected.

ChristophWurst avatar Apr 19 '24 09:04 ChristophWurst

I put out a PR #5934, but I think I understand what you're saying now. I'm still struggling to set up a testing environment that includes my local calendar app, so haven't been able to test if this actually works or not

pjrobertson avatar Apr 19 '24 09:04 pjrobertson

You can find some resources for dev envs at https://nextcloud.com/developer

ChristophWurst avatar Apr 19 '24 09:04 ChristophWurst

Yeah, I've been through that multiple times but there's little info on testing an existing app, and when I tried it it was throwing all sorts of errors. I unfortunately don't have time to debug for half a day :(

pjrobertson avatar Apr 21 '24 03:04 pjrobertson

+1 🙏

viktorix avatar Aug 05 '24 15:08 viktorix