feat: when bulk importing, ignore already invited people

[Pritam-sd-dev]

What does this PR do?

Fixes #13825
Before:
https://www.loom.com/share/2478041272b24f6aabed035c06f0c7e5?sid=18a4848a-1b1a-49c5-8ed9-ff7fc44af281

After:
https://www.loom.com/share/285d8bd3141640d98db157495c140171?sid=41e32490-af3f-49e1-b14f-5cf502cc058c

Type of change

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)

How should this be tested?

Checklist

• I haven't checked if my changes generate no new warnings
• I haven't added tests that prove my fix is effective or that my feature works
• I haven't checked if new and existing unit tests pass locally with my changes

[vercel]

@Pritam-sd-dev is attempting to deploy a commit to the cal Team on Vercel.

A member of the Team first needs to authorize it.

[github-actions]

Thank you for following the naming conventions! 🙏 Feel free to join our discord and post your PR link.

[graphite-app]

Graphite Automations

A Graphite automation took an action on this PR • (02/26/24)

1 label was added based on Keith Williams's automation, 'Add community label'

[github-actions]

📦 Next.js Bundle Analysis for @calcom/web

This analysis was generated by the Next.js Bundle Analysis action. 🤖

This PR introduced no changes to the JavaScript bundle! 🙌

[github-actions]

This PR is being marked as stale due to inactivity.

[PeerRich]

looks solid, anything missing?

[emrysal]

Hi @Pritam-sd-dev - I'd like see this fixed in the getUsersToInvite function, instead of throwing an error what we should do there is a .filter() function that runs the check eligibility code. This should be enough to fix the issue.

Also, you can consider splitting up the check eligibility code into two seperate functions (one for illegal actions, a guard and one for filtering to prevent duplicate invites).

[github-actions]

This PR is being marked as stale due to inactivity.

[CarinaWolli]

Hi @Pritam-sd-dev - I'd like see this fixed in the getUsersToInvite function, instead of throwing an error what we should do there is a .filter() function that runs the check eligibility code. This should be enough to fix the issue.

Also, you can consider splitting up the check eligibility code into two seperate functions (one for illegal actions, a guard and one for filtering to prevent duplicate invites).

@Pritam-sd-dev will you be able to take a look at that? Putting it in draft for now until this is addressed 🙏

[Amit91848]

This has already been fixed #13139

cal.com/packages/trpc/server/routers/viewer/teams/inviteMember/inviteMember.handler.ts

Lines 78 to 86 in df95ec4

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

[CarinaWolli]

This has already been fixed #13139