Allow to select contact groups as attendees

[tcitworld]

demo.webm

A significant limitation is that if a contact has multiple addresses, all of them will be added (as shown in the above video). Unfortunately once attendees are added, their email addresses are currently not shown, so you can't know which attendee item to remove.

I'm not sure what's the best way to proceed here and I'd like your opinion on that (as well as @nextcloud/designers ):

• only add the first email
• show the email in the attendee list (always or when a contact is added with several emails), so that the user can remove the unwanted emails
• add a special handling modal form to decide which email to pick (please don't say that :sweat_smile: ) ?
• change the way the multiselect works to select actual contacts, not contacts emails, and then pick the email used in the attendee list. This means we also need to allow contacts being added twice if the user wants to sent invites to both emails.

(Also the message « Group contains X contacts » kinda lies, it's about the attendees emails, not the attendees contacts.

Closes #167

[codecov[bot]]

Codecov Report

Attention: Patch coverage is 0% with 24 lines in your changes are missing coverage. Please review.

Project coverage is 23.96%. Comparing base (6b4845a) to head (2f96d9e). Report is 2 commits behind head on main.

Files	Patch %	Lines
.../components/Editor/Invitees/InviteesListSearch.vue	0.00%	17 Missing :warning:
...rc/components/Editor/Invitees/InviteesListItem.vue	0.00%	7 Missing :warning:

Additional details and impacted files

@@             Coverage Diff              @@
##               main    #4742      +/-   ##
============================================
+ Coverage     23.92%   23.96%   +0.03%     
- Complexity      455      460       +5     
============================================
  Files           247      247              
  Lines         11517    11557      +40     
  Branches       1934     1944      +10     
============================================
+ Hits           2756     2770      +14     
- Misses         8761     8787      +26     

Flag	Coverage Δ
javascript	15.45% <0.00%> (-0.05%)	:arrow_down:
php	58.69% <ø> (+0.22%)	:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

[ChristophWurst]

• show the email in the attendee list (always or when a contact is added with several emails), so that the user can remove the unwanted emails

This was also discussed in https://github.com/nextcloud/calendar/issues/3853#issuecomment-1308523996. I think it would make sense and adds value in general.

[nimishavijay]

show the email in the attendee list (always or when a contact is added with several emails), so that the user can remove the unwanted emails

This sounds good to me as well.

There should be something to indicate that there are multiple emails associated with one contact, especially when the group has many contacts. So we could possibly:

1. show the emails only for the contacts with multiple emails (and not for every contact)
2. show all the emails and sort the contacts with multiple emails up top
3. show all emails and add some visual indication to show that the contact has multiple emails, like combining the 2 list items

I would go with 2) because it's simple and doesn't require any scrolling to see that there are multiple email contacts cc @jancborchardt

[tcitworld]

For now I have added the suggestion from @ChristophWurst as it's the easy part. However I'm not sure it's needed if we show the emails in the attendee item subline as per 2. .

In any case we really need to remove the « invitation sent » label until the event is saved and the invitations are really sent, as adding the email next to it will even more increase confusion.

[miaulalala]

I fundamentally disagree with hiding the email address in the three dot menu. I think it should be visible immediately without user interaction as it is relevant information and belongs to the same sort of information as a Display Name is.

[ekrahmer]

from the user's perspective it is very simple: use the preferred email address EMAIL;PREF=1;TYPE=OTHER:[email protected] EMAIL;TYPE=HOME:[email protected] This (RFC6350) PREF parameter is part of the NC contact database and can be handled by CardDAV apps like Cardbook. Ask the contact app developers to make it accessible in the NC web interface.

(I'm waiting since long to be able to invite the Tenor voice section of our choit to extra rehersals including theit accept/decline replies.....)

[jancborchardt]

Talked a bit about this with @nimishavijay, we were going through the comments here and concluded:

• It would be annoying to receive mails to all your email addresses. Already now we have the situation that when receiving an invite you just accept it, and it would just be more annoying notifications.
• It would also be strange to clutter up the attendee list with all the mails of one person. You want to invite people, not emails. If you want to invite a person with a specific mail, you can still check through the list and invite the person via that mail

What we can do is:

• Using the "preferred" mail address as suggested by @ekrahmer is a good option
  • If that doesn’t exist, simply only pick the first one. In a server like e.g. a work Nextcloud, it is likely the first (or only) mail of a person which is their relevant one for a work event.
• Showing the emails directly in the list as mentioned by @miaulalala is also a good point. We can show them in the subline like so: "Invitation sent · [email protected]"

[small1]

Any status on this one? :) I just got a question about this feature and went looking for a ticket for it :)

[miaulalala]

@tcitworld are you happy to implement the changes Jan suggested? I can take over otherwise, excellent work btw.

[VLGM]

Hi,

I'm a newbie but when this will be released ? How can I use this, because I absolutely need this function form teams planning.

Regards

[Ekinoxies]

Hello all, great work ! Do you know when this feature will be implemented ? :D

[JMoVS]

has there been any movement on this?

[Durkar]

It would be great, if someone will contiue the work. :)

[Upperholme]

The lack of any way to easily add multiple email addresses as invitees to an event is crippling. I've tried pasting in a comma separated list of addresses (doesn't work), pasting in from Excel or similar (doesn't work), adding contacts to an address book and/or group in the Contacts app and trying to name the book or group in the invitees field. Any or all of these approaches would allow me to use Nextcloud calendar. This particular thread is 18 months old. I've come across others that stretch back to 2015 or beyond.

[miaulalala]

Added the email as text next to the attendees:

I think I broke the lovely https://github.com/nextcloud/calendar/pull/5396