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

Add contact groups as attendees for an event [$50] #167

Open
tcitworld opened this issue Nov 7, 2016 · 32 comments · May be fixed by #4742
Open

Add contact groups as attendees for an event [$50] #167

tcitworld opened this issue Nov 7, 2016 · 32 comments · May be fixed by #4742
Labels
1. to develop Accepted and waiting to be taken care of enhancement New feature request Feature: Editor needs research

Comments

@tcitworld
Copy link
Member

tcitworld commented Nov 7, 2016

The participant event module should suggest contacts groups from the contacts app.

Related issues

@tcitworld tcitworld added 1. to develop Accepted and waiting to be taken care of enhancement New feature request Feature: Editor labels Nov 7, 2016
@tcitworld tcitworld changed the title Add contact groups as participants for an event Add contact groups as attendees for an event Nov 7, 2016
@tcitworld
Copy link
Member Author

Maybe linked to #163

@georgehrke georgehrke added this to the 1.7.0 milestone Nov 29, 2016
@georgehrke
Copy link
Member

I'm actually not sure if we can query for contact groups via the contacts php api.

Also, how to show if some users of a group have no e-mail address

@tcitworld
Copy link
Member Author

Also, how to show if some users of a group have no e-mail address

Just don't show them. :-)
I'll look if the groups are available.

@maksemuz
Copy link

I'd offer the following:
1 - use access groups, especially synced LDAP groups, not groups from contacts. You can parse LDAP groups with standard LDAP tools.
2 - if some participant account has no email, you can just add event to its calendar but do not send invitation by email.

@pwFoo
Copy link

pwFoo commented Jul 20, 2019

Should groups expand to members / contacts and add the "person" to the event or just added as group of persons?

Expand the groups to users / contacts / persons would be flexible (add / remove single persons from event).
Save group to event would add / remove users / contacts if groups modified (maybe not wanted for fix planned events).

@georgehrke
Copy link
Member

Expand the groups to users / contacts / persons would be flexible (add / remove single persons from event).

This would be my choice for a first implementation. That should be fairly easy and doesn't require modifications to the Nextcloud server itself.

@dieter-wilhelm
Copy link

dieter-wilhelm commented Dec 27, 2019

Added 50 € on Bountysource for this indespensible calendar feature. But for the sake of privacy there should be an option to hide the list of invitees!

Thanks for the calendar app.

@flozi00
Copy link

flozi00 commented Feb 18, 2020

Expand the groups to users / contacts / persons would be flexible (add / remove single persons from event).

This would be my choice for a first implementation. That should be fairly easy and doesn't require modifications to the Nextcloud server itself.

Any updates about implementing this feature as temporary solution ?

@georgehrke
Copy link
Member

Nope, as always: This is a free-software project. Pull-requests are very much appreciated.

@flozi00
Copy link

flozi00 commented Feb 18, 2020

Where can I find the relevant code part ?

@georgehrke
Copy link
Member

To implement this, you would have to alter the ContactsController:
https://github.com/nextcloud/calendar/blob/master/lib/Controller/ContactController.php
Preferably one endpoint that allows the lookup of groups and another endpoint that returns chunks of contacts in this group.

We have an InviteesListSearch component.

In the search method (findAttendees), start looking for groups. When the user selects a group, start loading the contacts of that group in the addAttendee method. Please load them in chunks of 50 to 100.

@flozi00
Copy link

flozi00 commented Feb 20, 2020

My PHP knowledge is too basic for that and never worked with vue before.
I tried it two days without success, can you implement this feature please?

@jospoortvliet jospoortvliet changed the title Add contact groups as attendees for an event Add contact groups as attendees for an event [$50] Mar 25, 2020
@jakobroehrl
Copy link
Contributor

What do you think about adding the talk room participants as attendees?
Similar function as sharing a file to a talk room

@wiswedel
Copy link
Contributor

This discussion and the related issues that have been closed for being duplicates takes some wild directions in my opinion.

Some are talking about contact groups (from the Contacts app), some are talking about User groups (from the user management / LDAP). Even Talk rooms have been brought up.
Additionally, Circles should not be forgotten since they're also a way of grouping people plus you can already share a calendar with a circle, but not an event.

tl;dr: We have the request to share with

  1. Contact groups (from address books in the Contacts app)
  2. User groups (from the user management, including LDAP groups)
  3. Circles
  4. Talk rooms (i.e. their members)

If you, @tcitworld (as issue initiator) agree with me, would you mind changing the issue title?

@micheledimeo
Copy link

@m4us1ne , I felt that my offer was too low and then bountysource is at the centre of too much controversy for my liking.
Since this issue was included in the "approved improvements" of calendar, I'm waiting for version 2.2 to use this feature.
In the meantime I'm using a calendar external to NC.

@tcitworld tcitworld changed the title Add contact groups as attendees for an event [$50] Add contact groups as attendees for an event Mar 19, 2021
@tcitworld
Copy link
Member Author

tl;dr: We have the request to share with
1. Contact groups (from address books in the Contacts app)
2. User groups (from the user management, including LDAP groups)
3. Circles
4. Talk rooms (i.e. their members)

If you, @tcitworld (as issue initiator) agree with me, would you mind changing the issue title?

Done and splitted the issue into several (see the updated first message).

@ChristophWurst ChristophWurst modified the milestones: 2.2.0, 2.3.0 Mar 24, 2021
@DJaeger DJaeger changed the title Add contact groups as attendees for an event Add contact groups as attendees for an event [$50] Apr 1, 2021
@DJaeger
Copy link

DJaeger commented Apr 1, 2021

@pierreozoux
Copy link
Member

pierreozoux commented Apr 2, 2021

Looks like bountysource is no longer the default.
https://help.nextcloud.com/t/dont-use-bountysource-anymore/84943

@tcitworld do you plan to develop this feature?
If yes, you could as well donate to https://framasoft.org/en/ (he is part of this association)

@julian70400
Copy link

julian70400 commented Apr 20, 2021

Hello, any news ?

This feature is absoluelty essential and is a basic for every groupware. How can we help ? What is actually blocking this feature ?

@jpubb
Copy link

jpubb commented May 19, 2021

I'm actually not sure if we can query for contact groups via the contacts php api.

Also, how to show if some users of a group have no e-mail address

This is what people want to use...

image

People are simply trying to replicate MS Exchange Distribution Lists functionality here (used for decades). That is:

System Account users > groups of users > email/calendar

Every big enterprise with MS Exchange has some kind of lists set up that email/calendaring can access so they can email communications to _DL_ALL_INTERNAL_EMPLOYEEES and _DL_ALL_EXTERNAL_CONTRACTORS

Nobody wants to have to onesie twosie manually add recips, especially when you're emailing "all". The nextcloud calendar app already ignores atendee results where the nextcloud user does not have an email field populated, right? So we're at least able to get a list of users and see their email field. Ok, that's a start.

Why not just do something like:

Option 1:

  • Let user select a nextcloud group(s) as attendee
  • Upon selection, ValidateGroup() and foreach user in group, verify each user in the group has an email address, or throw error that "$CurrentGroupMemberNAme does not have an email address associated with their account. Terminating."

Keep it simple. Admins will know what to do from there (add an email address for the user, or remove the user from the group)

Option 2:

  • Let user select a nextcloud group(s) as attendee
  • Upon selection, ExpandGroup() and foreach user in group who has an email add that individual attendee in the TO or CC field.

Option 3:
Worst option, but I think people will take what they can get here because manually adding people sucks to the point of nothing nice to say. If you're saying you have no access to read Nextcloud groups. Then would it be possible for this option 3 to just create a Calendar Distribution List - CDL within Calendar app that can be maintained? Worst option, because everyone who wants "groups for calendar" already has these department/group lists created once in Nextcloud User Groups. But if it's the only option, we'll all be "happy" to maintain 2 lists I'm sure.

@jpubb
Copy link

jpubb commented May 19, 2021

I'd offer the following:
1 - use access groups, especially synced LDAP groups, not groups from contacts. You can parse LDAP groups with standard LDAP tools.
> 2 - if some participant account has no email, you can just add event to its calendar but do not send invitation by email.

That's a good idea. I'd also include a notification for the user so they know it was added. But that's all extra code. If I were the app author/maintainer, I'd just "no soup for you" if someone in the group doesn't have an email address for version 1 of this. "Email adress for group member $username is missing. Stopping."

@jpubb
Copy link

jpubb commented May 19, 2021

Nope, as always: This is a free-software project. Pull-requests are very much appreciated.

Btw, if anyone is interested in DIY instant bounties but you don't know how to code, you can always post the project for a fixed price at some place like Upwork, and make sending a pull request to the maintainer (in addition to sending the patch & instructions to you immediately) as part of the deliverable.

@barbuslex

This comment has been minimized.

@ChristophWurst ChristophWurst modified the milestones: v2.4.0, v2.4.1 Nov 25, 2021
@tcitworld tcitworld removed this from the v2.4.1 milestone Dec 17, 2021
@tcitworld tcitworld removed the bounty label Dec 17, 2021
@msharp1

This comment was marked as off-topic.

@tcitworld

This comment was marked as off-topic.

@DFC1990

This comment was marked as duplicate.

@mightynico

This comment was marked as duplicate.

@julian70400

This comment was marked as duplicate.

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 enhancement New feature request Feature: Editor needs research
Projects
None yet
Development

Successfully merging a pull request may close this issue.