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

[CAL-3422] [CAL-3230] [CAL-3193] when bulk importing, ignore already invited people #13825

Closed
1 of 2 tasks
PeerRich opened this issue Feb 22, 2024 · 2 comments
Closed
1 of 2 tasks
Labels
community Created by Linear-GitHub Sync enterprise area: enterprise, audit log, organisation, SAML, SSO ✨ feature New feature or request Medium priority Created by Linear-GitHub Sync organizations area: organizations, orgs
Milestone

Comments

@PeerRich
Copy link
Member

PeerRich commented Feb 22, 2024

  • remove max. 100 people for admins impersonating and inside orgs
  • don'throw an error and hard-stops when user exists already

its making bulk imports incredibly frustrating if multiple people already joined on their own

ideally we dont stop the import


<sub>[CAL-3193](https://linear.app/calcom/issue/CAL-3193/when-bulk-importing-ignore-already-invited-people)</sub>

<sub>[CAL-3198](https://linear.app/calcom/issue/CAL-3198/[cal-3193]-when-bulk-importing-ignore-already-invited-people)</sub>

<sub>[CAL-3230](https://linear.app/calcom/issue/CAL-3230/[cal-3193]-when-bulk-importing-ignore-already-invited-people)</sub>

From SyncLinear.com | CAL-3422

@PeerRich PeerRich changed the title when bulk importing, ignore already invited people [CAL-3193] when bulk importing, ignore already invited people Feb 22, 2024
@maige-app maige-app bot added ✨ feature New feature or request 🚨 needs approval This feature request has not been reviewed yet by the Product Team and needs approval beforehand labels Feb 22, 2024
@PeerRich PeerRich added enterprise area: enterprise, audit log, organisation, SAML, SSO organizations area: organizations, orgs and removed 🚨 needs approval This feature request has not been reviewed yet by the Product Team and needs approval beforehand labels Feb 22, 2024
@keithwillcode keithwillcode added the Medium priority Created by Linear-GitHub Sync label Feb 22, 2024
@keithwillcode keithwillcode added this to the v4.0 milestone Feb 22, 2024
@keithwillcode keithwillcode changed the title [CAL-3193] when bulk importing, ignore already invited people [CAL-3198] [CAL-3193] when bulk importing, ignore already invited people Feb 22, 2024
@Pritam-sd-dev
Copy link
Contributor

hi, team
started working on it.

@keithwillcode keithwillcode changed the title [CAL-3198] [CAL-3193] when bulk importing, ignore already invited people [CAL-3193] when bulk importing, ignore already invited people Feb 27, 2024
@keithwillcode keithwillcode modified the milestones: v4.0, v3.9 Feb 27, 2024
@keithwillcode keithwillcode added the community Created by Linear-GitHub Sync label Feb 27, 2024
@keithwillcode keithwillcode changed the title [CAL-3193] when bulk importing, ignore already invited people [CAL-3230] [CAL-3193] when bulk importing, ignore already invited people Feb 27, 2024
@CarinaWolli CarinaWolli modified the milestones: v3.9, v4.0 Mar 18, 2024
@keithwillcode keithwillcode modified the milestones: v4.0, v4.1 Apr 15, 2024
@keithwillcode keithwillcode changed the title [CAL-3230] [CAL-3193] when bulk importing, ignore already invited people [CAL-3422] [CAL-3230] [CAL-3193] when bulk importing, ignore already invited people Apr 15, 2024
@Amit91848
Copy link
Contributor

This has already been fixed #13139

const existingUsersWithMemberships = await getUsersToInvite({
usernamesOrEmails: usernameOrEmailsToInvite,
isInvitedToOrg: isOrg,
team,
});
const existingUsersWithMembershipsThatNeedToBeInvited = existingUsersWithMemberships.filter(
(invitee) => invitee.canBeInvited
);

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community Created by Linear-GitHub Sync enterprise area: enterprise, audit log, organisation, SAML, SSO ✨ feature New feature or request Medium priority Created by Linear-GitHub Sync organizations area: organizations, orgs
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants