- {org.metadata?.isOrganizationConfigured ? (
+ {org.organizationSettings?.isOrganizationConfigured ? (
{t("dns_configured")}
) : (
{t("dns_missing")}
@@ -136,7 +136,7 @@ function AdminOrgTable() {
{
updateMutation.mutate({
id: org.id,
- metadata: {
+ organizationSettings: {
isOrganizationConfigured: true,
},
});
diff --git a/packages/features/ee/teams/api/upgrade.ts b/packages/features/ee/teams/api/upgrade.ts
index aebdbfd691..db37c77b09 100644
--- a/packages/features/ee/teams/api/upgrade.ts
+++ b/packages/features/ee/teams/api/upgrade.ts
@@ -87,7 +87,7 @@ async function handler(req: NextApiRequest, res: NextApiResponse) {
if (!session) return { message: "Team upgraded successfully" };
- const redirectUrl = metadata?.data?.isOrganization
+ const redirectUrl = team?.isOrganization
? `${WEBAPP_URL}/settings/organizations/profile?upgraded=true`
: `${WEBAPP_URL}/settings/teams/${team.id}/profile?upgraded=true`;
diff --git a/packages/features/ee/teams/components/MemberListItem.tsx b/packages/features/ee/teams/components/MemberListItem.tsx
index 714cfe9115..c8de6d6369 100644
--- a/packages/features/ee/teams/components/MemberListItem.tsx
+++ b/packages/features/ee/teams/components/MemberListItem.tsx
@@ -5,7 +5,6 @@ import { useState } from "react";
import { useLocale } from "@calcom/lib/hooks/useLocale";
import { MembershipRole } from "@calcom/prisma/enums";
-import { teamMetadataSchema } from "@calcom/prisma/zod-utils";
import type { RouterOutputs } from "@calcom/trpc/react";
import { trpc } from "@calcom/trpc/react";
import useMeQuery from "@calcom/trpc/react/hooks/useMeQuery";
@@ -47,9 +46,7 @@ const useCurrentUserId = () => {
};
const checkIsOrg = (team: Props["team"]) => {
- const metadata = teamMetadataSchema.safeParse(team.metadata);
- if (metadata.success && metadata.data?.isOrganization) return true;
- return false;
+ return team.isOrganization;
};
export default function MemberListItem(props: Props) {
diff --git a/packages/features/filters/components/TeamsFilter.tsx b/packages/features/filters/components/TeamsFilter.tsx
index 1c34ce09b1..44dfe75bb6 100644
--- a/packages/features/filters/components/TeamsFilter.tsx
+++ b/packages/features/filters/components/TeamsFilter.tsx
@@ -6,7 +6,6 @@ import { classNames } from "@calcom/lib";
import { getPlaceholderAvatar } from "@calcom/lib/defaultAvatarImage";
import { useLocale } from "@calcom/lib/hooks/useLocale";
import { useTypedQuery } from "@calcom/lib/hooks/useTypedQuery";
-import { teamMetadataSchema } from "@calcom/prisma/zod-utils";
import { trpc } from "@calcom/trpc/react";
import type { RouterOutputs } from "@calcom/trpc/react";
import { AnimatedPopover, Avatar, Divider, Tooltip, VerticalDivider } from "@calcom/ui";
@@ -88,7 +87,7 @@ export const TeamsFilter = ({
/>
{teams
- ?.filter((team) => !teamMetadataSchema.parse(team.metadata)?.isOrganization)
+ ?.filter((team) => !team?.isOrganization)
.map((team) => (
}) {
+ return team.isOrganization;
}
export function getEntityPermissionLevel(
diff --git a/packages/lib/server/queries/teams/index.ts b/packages/lib/server/queries/teams/index.ts
index c910c8cea1..ba4ae2967b 100644
--- a/packages/lib/server/queries/teams/index.ts
+++ b/packages/lib/server/queries/teams/index.ts
@@ -83,6 +83,7 @@ export async function getTeamWithMembers(args: {
id: true,
name: true,
slug: true,
+ isOrganization: true,
...(!!includeTeamLogo ? { logo: true } : {}),
bio: true,
hideBranding: true,
diff --git a/packages/lib/server/repository/team.test.ts b/packages/lib/server/repository/team.test.ts
index c709b138a7..8e2e71885d 100644
--- a/packages/lib/server/repository/team.test.ts
+++ b/packages/lib/server/repository/team.test.ts
@@ -30,9 +30,7 @@ describe("getOrg", () => {
id: 101,
name: "Test Team",
slug: "test-slug",
- metadata: {
- isOrganization: true,
- },
+ isOrganization: true,
},
]);
@@ -52,18 +50,16 @@ describe("getOrg", () => {
expect(firstFindManyCallArguments[0]).toEqual({
where: {
slug: "test-slug",
- metadata: {
- path: ["isOrganization"],
- equals: true,
- },
+ isOrganization: true,
},
select: {
id: true,
slug: true,
metadata: true,
+ isOrganization: true,
},
});
- expect(org?.metadata?.isOrganization).toBe(true);
+ expect(org?.isOrganization).toBe(true);
});
it("should not return an org result if metadata.isOrganization isn't true", async () => {
@@ -93,15 +89,13 @@ describe("getOrg", () => {
expect(firstFindManyCallArguments[0]).toEqual({
where: {
slug: "test-slug",
- metadata: {
- path: ["isOrganization"],
- equals: true,
- },
+ isOrganization: true,
},
select: {
id: true,
slug: true,
metadata: true,
+ isOrganization: true,
},
});
expect(org).toBe(null);
@@ -171,6 +165,7 @@ describe("getTeam", () => {
slug: true,
name: true,
metadata: true,
+ isOrganization: true,
},
});
expect(team).not.toBeNull();
@@ -185,9 +180,7 @@ describe("getTeam", () => {
id: 101,
name: "Test Team",
slug: "test-slug",
- metadata: {
- isOrganization: true,
- },
+ isOrganization: true,
},
]);
@@ -214,6 +207,7 @@ describe("getTeam", () => {
slug: true,
name: true,
metadata: true,
+ isOrganization: true,
},
});
expect(team).toBe(null);
@@ -260,10 +254,7 @@ describe("getTeam", () => {
},
},
],
- metadata: {
- path: ["isOrganization"],
- equals: true,
- },
+ isOrganization: true,
},
},
select: {
@@ -271,6 +262,7 @@ describe("getTeam", () => {
name: true,
slug: true,
metadata: true,
+ isOrganization: true,
},
});
});
@@ -294,10 +286,7 @@ describe("getTeam", () => {
where: {
slug: "test-team",
parent: {
- metadata: {
- path: ["isOrganization"],
- equals: true,
- },
+ isOrganization: true,
OR: [
{
slug: "test-org",
@@ -316,6 +305,7 @@ describe("getTeam", () => {
slug: true,
name: true,
metadata: true,
+ isOrganization: true,
},
});
});
diff --git a/packages/lib/server/repository/team.ts b/packages/lib/server/repository/team.ts
index 26d63de555..5780c31102 100644
--- a/packages/lib/server/repository/team.ts
+++ b/packages/lib/server/repository/team.ts
@@ -7,8 +7,9 @@ import prisma from "@calcom/prisma";
import { teamMetadataSchema } from "@calcom/prisma/zod-utils";
type TeamGetPayloadWithParsedMetadata =
- | (Omit, "metadata"> & {
+ | (Omit, "metadata" | "isOrganization"> & {
metadata: z.infer;
+ isOrganization: boolean;
})
| null;
@@ -49,7 +50,8 @@ async function getTeamOrOrg({
teamSelect = {
...teamSelect,
metadata: true,
- };
+ isOrganization: true,
+ } satisfies TeamSelect;
if (lookupBy.havingMemberWithId) where.members = { some: { userId: lookupBy.havingMemberWithId } };
if ("id" in lookupBy) {
@@ -63,10 +65,7 @@ async function getTeamOrOrg({
// Note that an organization and a team that doesn't belong to an organization, both have parentId null
// If the organization has null slug(but requestedSlug is 'test') and the team also has slug 'test', we can't distinguish them without explicitly checking the metadata.isOrganization
// Note that, this isn't possible now to have same requestedSlug as the slug of a team not part of an organization. This is legacy teams handling mostly. But it is still safer to be sure that you are fetching an Organization only in case of isOrgView
- where.metadata = {
- path: ["isOrganization"],
- equals: true,
- };
+ where.isOrganization = true;
// We must fetch only the team here.
} else {
if (forOrgWithSlug) {
@@ -83,21 +82,32 @@ async function getTeamOrOrg({
// teamSelect extends Prisma.TeamSelect but still teams doesn't contain a valid team as per TypeScript and thus it doesn't consider it having team.metadata, team.id and other fields
// This is the reason below code is using a lot of assertions.
+ const select = teamSelect;
+
const teams = await prisma.team.findMany({
where,
select: teamSelect,
});
const teamsWithParsedMetadata = teams
- .map((team) => ({
- ...team,
- // Using Type assertion here because we know that the metadata is present and Prisma and TypeScript aren't playing well together
- metadata: teamMetadataSchema.parse((team as { metadata: z.infer }).metadata),
- }))
+ .map((team) => {
+ const teamX = team;
+
+ // eslint-disable-next-line @typescript-eslint/ban-ts-comment
+ // @ts-ignore ts types are way too complciated for this now
+ const parsedMetadata = teamMetadataSchema.parse(team.metadata ?? {});
+ return {
+ ...team,
+ // eslint-disable-next-line @typescript-eslint/ban-ts-comment
+ // @ts-ignore It does exist
+ isOrganization: team.isOrganization as boolean,
+ metadata: parsedMetadata,
+ };
+ })
// In cases where there are many teams with the same slug, we need to find out the one and only one that matches our criteria
.filter((team) => {
// We need an org if isOrgView otherwise we need a team
- return isOrg ? team.metadata?.isOrganization : !team.metadata?.isOrganization;
+ return isOrg ? team.isOrganization : !team.isOrganization;
});
if (teamsWithParsedMetadata.length > 1) {
diff --git a/packages/lib/validateUsername.ts b/packages/lib/validateUsername.ts
index 56d0a10de9..10e03ede45 100644
--- a/packages/lib/validateUsername.ts
+++ b/packages/lib/validateUsername.ts
@@ -1,6 +1,5 @@
import { getOrgUsernameFromEmail } from "@calcom/features/auth/signup/utils/getOrgUsernameFromEmail";
import prisma from "@calcom/prisma";
-import { teamMetadataSchema } from "@calcom/prisma/zod-utils";
export const getUsernameForOrgMember = async ({
email,
@@ -94,10 +93,12 @@ export const validateAndGetCorrectedUsernameInTeam = async (
},
select: {
metadata: true,
+ isOrganization: true,
parentId: true,
+ organizationSettings: true,
parent: {
select: {
- metadata: true,
+ organizationSettings: true,
},
},
},
@@ -107,17 +108,15 @@ export const validateAndGetCorrectedUsernameInTeam = async (
teamId,
team,
});
- const teamData = { ...team, metadata: teamMetadataSchema.parse(team?.metadata) };
- const organization = teamData.metadata?.isOrganization ? teamData : teamData.parent;
+ const organization = team?.isOrganization ? team : team?.parent;
if (organization) {
- const orgMetadata = teamMetadataSchema.parse(organization.metadata);
// Organization context -> org-context username check
- const orgId = teamData.parentId || teamId;
+ const orgId = team?.parentId || teamId;
return validateAndGetCorrectedUsernameAndEmail({
username,
email,
organizationId: orgId,
- orgAutoAcceptEmail: orgMetadata?.orgAutoAcceptEmail || "",
+ orgAutoAcceptEmail: organization?.organizationSettings?.orgAutoAcceptEmail || "",
isSignup,
});
} else {
diff --git a/packages/prisma/migrations/20231215054122_organization_metadata/migration.sql b/packages/prisma/migrations/20231215054122_organization_metadata/migration.sql
new file mode 100644
index 0000000000..59a5f73aec
--- /dev/null
+++ b/packages/prisma/migrations/20231215054122_organization_metadata/migration.sql
@@ -0,0 +1,43 @@
+ALTER TABLE "Team" ADD COLUMN "isOrganization" BOOLEAN NOT NULL DEFAULT false;
+
+-- CreateTable
+CREATE TABLE "OrganizationSettings" (
+ "id" SERIAL NOT NULL,
+ "organizationId" INTEGER NOT NULL,
+ "isOrganizationConfigured" BOOLEAN NOT NULL DEFAULT false,
+ "isOrganizationVerified" BOOLEAN NOT NULL DEFAULT false,
+ "orgAutoAcceptEmail" TEXT NOT NULL,
+
+ CONSTRAINT "OrganizationSettings_pkey" PRIMARY KEY ("id")
+);
+
+-- CreateIndex
+CREATE UNIQUE INDEX "OrganizationSettings_organizationId_key" ON "OrganizationSettings"("organizationId");
+
+-- CreateIndex
+CREATE UNIQUE INDEX "Team_slug_isOrganization_key" ON "Team"("slug", "isOrganization");
+
+-- AddForeignKey
+ALTER TABLE "OrganizationSettings" ADD CONSTRAINT "OrganizationSettings_organizationId_fkey" FOREIGN KEY ("organizationId") REFERENCES "Team"("id") ON DELETE CASCADE ON UPDATE CASCADE;
+-- -- Set team field to notify if it is an organization -> Easier than metadata.parse(X).isOrganization
+UPDATE "Team"
+ SET "isOrganization" = (metadata ->> 'isOrganization')::BOOLEAN
+WHERE
+ metadata ->> 'isOrganization' = 'true';
+
+-- -- Insert data into org settings
+INSERT INTO "OrganizationSettings" ("organizationId", "isOrganizationConfigured", "orgAutoAcceptEmail", "isOrganizationVerified")
+SELECT
+ t.id,
+ (t.metadata ->> 'isOrganizationConfigured')::BOOLEAN AS "isOrganizationConfigured",
+ t.metadata ->> 'orgAutoAcceptEmail' AS "orgAutoAcceptEmail",
+ (t.metadata ->> 'isOrganizationVerified')::BOOLEAN AS "isOrganizationVerified"
+FROM (
+ SELECT
+ id,
+ metadata
+ FROM
+ "Team"
+ WHERE
+ metadata ->> 'isOrganization' = 'true'
+) AS t;
\ No newline at end of file
diff --git a/packages/prisma/schema.prisma b/packages/prisma/schema.prisma
index 37f0045905..111a3f4d89 100644
--- a/packages/prisma/schema.prisma
+++ b/packages/prisma/schema.prisma
@@ -315,9 +315,21 @@ model Team {
apiKeys ApiKey[]
credentials Credential[]
accessCodes AccessCode[]
+ isOrganization Boolean @default(false)
+ organizationSettings OrganizationSettings?
instantMeetingTokens InstantMeetingToken[]
@@unique([slug, parentId])
+ @@unique([slug, isOrganization])
+}
+
+model OrganizationSettings {
+ id Int @id @default(autoincrement())
+ organization Team @relation(fields: [organizationId], references: [id], onDelete: Cascade)
+ organizationId Int @unique
+ isOrganizationConfigured Boolean @default(false)
+ isOrganizationVerified Boolean @default(false)
+ orgAutoAcceptEmail String
}
enum MembershipRole {
diff --git a/packages/prisma/zod-utils.ts b/packages/prisma/zod-utils.ts
index 2cd4232581..be1d5b9e2e 100644
--- a/packages/prisma/zod-utils.ts
+++ b/packages/prisma/zod-utils.ts
@@ -332,6 +332,13 @@ export const userMetadata = z
})
.nullable();
+export const orgSettingsSchema = z
+ .object({
+ isOrganizationVerified: z.boolean().optional(),
+ isOrganizationConfigured: z.boolean().optional(),
+ orgAutoAcceptEmail: z.string().optional(),
+ })
+ .nullable();
export type userMetadataType = z.infer;
export const teamMetadataSchema = z
@@ -340,10 +347,6 @@ export const teamMetadataSchema = z
paymentId: z.string(),
subscriptionId: z.string().nullable(),
subscriptionItemId: z.string().nullable(),
- isOrganization: z.boolean().nullable(),
- isOrganizationVerified: z.boolean().nullable(),
- isOrganizationConfigured: z.boolean().nullable(),
- orgAutoAcceptEmail: z.string().nullable(),
migratedToOrgFrom: z
.object({
teamSlug: z.string().or(z.null()).optional(),
diff --git a/packages/trpc/server/routers/loggedInViewer/teamsAndUserProfilesQuery.handler.ts b/packages/trpc/server/routers/loggedInViewer/teamsAndUserProfilesQuery.handler.ts
index 4dd6a0ef2f..ed481f9f62 100644
--- a/packages/trpc/server/routers/loggedInViewer/teamsAndUserProfilesQuery.handler.ts
+++ b/packages/trpc/server/routers/loggedInViewer/teamsAndUserProfilesQuery.handler.ts
@@ -1,4 +1,4 @@
-import { isOrganization, withRoleCanCreateEntity } from "@calcom/lib/entityPermissionUtils";
+import { withRoleCanCreateEntity } from "@calcom/lib/entityPermissionUtils";
import { getTeamAvatarUrl, getUserAvatarUrl } from "@calcom/lib/getAvatarUrl";
import type { PrismaClient } from "@calcom/prisma";
import { teamMetadataSchema } from "@calcom/prisma/zod-utils";
@@ -34,6 +34,7 @@ export const teamsAndUserProfilesQuery = async ({ ctx }: TeamsAndUserProfileOpti
team: {
select: {
id: true,
+ isOrganization: true,
name: true,
slug: true,
metadata: true,
@@ -55,7 +56,7 @@ export const teamsAndUserProfilesQuery = async ({ ctx }: TeamsAndUserProfileOpti
}
const nonOrgTeams = user.teams
- .filter((membership) => !isOrganization({ team: membership.team }))
+ .filter((membership) => !membership.team.isOrganization)
.map((membership) => ({
...membership,
team: {
diff --git a/packages/trpc/server/routers/viewer/eventTypes/getByViewer.handler.ts b/packages/trpc/server/routers/viewer/eventTypes/getByViewer.handler.ts
index 9ce7b0ae0f..24ac64ebea 100644
--- a/packages/trpc/server/routers/viewer/eventTypes/getByViewer.handler.ts
+++ b/packages/trpc/server/routers/viewer/eventTypes/getByViewer.handler.ts
@@ -11,8 +11,7 @@ import { markdownToSafeHTML } from "@calcom/lib/markdownToSafeHTML";
import type { PrismaClient } from "@calcom/prisma";
import { baseEventTypeSelect } from "@calcom/prisma";
import { MembershipRole, SchedulingType } from "@calcom/prisma/enums";
-import { teamMetadataSchema } from "@calcom/prisma/zod-utils";
-import { EventTypeMetaDataSchema } from "@calcom/prisma/zod-utils";
+import { EventTypeMetaDataSchema, teamMetadataSchema } from "@calcom/prisma/zod-utils";
import { TRPCError } from "@trpc/server";
@@ -113,6 +112,7 @@ export const getByViewerHandler = async ({ ctx, input }: GetByViewerOptions) =>
team: {
select: {
id: true,
+ isOrganization: true,
name: true,
slug: true,
parentId: true,
@@ -240,8 +240,7 @@ export const getByViewerHandler = async ({ ctx, input }: GetByViewerOptions) =>
eventTypeGroups,
memberships
.filter((mmship) => {
- const metadata = mmship.team.metadata;
- if (metadata?.isOrganization) {
+ if (mmship.team.isOrganization) {
return false;
} else {
if (!input?.filters || !hasFilter(input?.filters)) {
diff --git a/packages/trpc/server/routers/viewer/organizations/adminGet.handler.ts b/packages/trpc/server/routers/viewer/organizations/adminGet.handler.ts
index 6574f8e6ee..a7466421a1 100644
--- a/packages/trpc/server/routers/viewer/organizations/adminGet.handler.ts
+++ b/packages/trpc/server/routers/viewer/organizations/adminGet.handler.ts
@@ -23,6 +23,7 @@ export const adminGetHandler = async ({ input }: AdminGetOptions) => {
name: true,
slug: true,
metadata: true,
+ isOrganization: true,
members: {
where: {
role: "OWNER",
@@ -37,6 +38,13 @@ export const adminGetHandler = async ({ input }: AdminGetOptions) => {
},
},
},
+ organizationSettings: {
+ select: {
+ isOrganizationConfigured: true,
+ isOrganizationVerified: true,
+ orgAutoAcceptEmail: true,
+ },
+ },
},
});
@@ -47,7 +55,7 @@ export const adminGetHandler = async ({ input }: AdminGetOptions) => {
});
}
const parsedMetadata = teamMetadataSchema.parse(org.metadata);
- if (!parsedMetadata?.isOrganization) {
+ if (!org?.isOrganization) {
throw new TRPCError({
code: "NOT_FOUND",
message: "Organization not found",
diff --git a/packages/trpc/server/routers/viewer/organizations/adminGetAll.handler.ts b/packages/trpc/server/routers/viewer/organizations/adminGetAll.handler.ts
index 7c2f01a1cb..16d9996b55 100644
--- a/packages/trpc/server/routers/viewer/organizations/adminGetAll.handler.ts
+++ b/packages/trpc/server/routers/viewer/organizations/adminGetAll.handler.ts
@@ -12,20 +12,14 @@ type AdminGetAllOptions = {
export const adminGetUnverifiedHandler = async ({}: AdminGetAllOptions) => {
const allOrgs = await prisma.team.findMany({
where: {
- AND: [
- {
- metadata: {
- path: ["isOrganization"],
- equals: true,
- },
- },
- ],
+ isOrganization: true,
},
select: {
id: true,
name: true,
slug: true,
metadata: true,
+ organizationSettings: true,
members: {
where: {
role: "OWNER",
diff --git a/packages/trpc/server/routers/viewer/organizations/adminUpdate.handler.ts b/packages/trpc/server/routers/viewer/organizations/adminUpdate.handler.ts
index 54e6b6c5bf..8dff9e7d74 100644
--- a/packages/trpc/server/routers/viewer/organizations/adminUpdate.handler.ts
+++ b/packages/trpc/server/routers/viewer/organizations/adminUpdate.handler.ts
@@ -17,11 +17,14 @@ type AdminUpdateOptions = {
};
export const adminUpdateHandler = async ({ input }: AdminUpdateOptions) => {
- const { id, ...restInput } = input;
+ const { id, organizationSettings, ...restInput } = input;
const existingOrg = await prisma.team.findUnique({
where: {
id: id,
},
+ include: {
+ organizationSettings: true,
+ },
});
if (!existingOrg) {
@@ -32,10 +35,8 @@ export const adminUpdateHandler = async ({ input }: AdminUpdateOptions) => {
}
const { mergeMetadata } = getMetadataHelpers(teamMetadataSchema.unwrap(), existingOrg.metadata);
- const data: Prisma.TeamUpdateArgs["data"] = {
- ...restInput,
- metadata: mergeMetadata({ ...restInput.metadata }),
- };
+
+ const data: Prisma.TeamUpdateArgs["data"] = restInput;
if (restInput.slug) {
await throwIfSlugConflicts({ id, slug: restInput.slug });
@@ -49,13 +50,33 @@ export const adminUpdateHandler = async ({ input }: AdminUpdateOptions) => {
data.metadata = mergeMetadata({
// If we save slug, we don't need the requestedSlug anymore
requestedSlug: undefined,
- ...input.metadata,
});
}
- const updatedOrganisation = await prisma.team.update({
- where: { id },
- data,
+ const updatedOrganisation = await prisma.$transaction(async (tx) => {
+ const updatedOrganisation = await tx.team.update({
+ where: { id },
+ data,
+ });
+
+ if (organizationSettings || existingOrg.organizationSettings) {
+ await tx.organizationSettings.update({
+ where: {
+ organizationId: updatedOrganisation.id,
+ },
+ data: {
+ isOrganizationConfigured:
+ organizationSettings?.isOrganizationConfigured ||
+ existingOrg.organizationSettings?.isOrganizationConfigured,
+ isOrganizationVerified:
+ organizationSettings?.isOrganizationVerified ||
+ existingOrg.organizationSettings?.isOrganizationVerified,
+ orgAutoAcceptEmail:
+ organizationSettings?.orgAutoAcceptEmail || existingOrg.organizationSettings?.orgAutoAcceptEmail,
+ },
+ });
+ }
+ return updatedOrganisation;
});
return updatedOrganisation;
diff --git a/packages/trpc/server/routers/viewer/organizations/adminUpdate.schema.ts b/packages/trpc/server/routers/viewer/organizations/adminUpdate.schema.ts
index 82ca9a3572..d8b2704f02 100644
--- a/packages/trpc/server/routers/viewer/organizations/adminUpdate.schema.ts
+++ b/packages/trpc/server/routers/viewer/organizations/adminUpdate.schema.ts
@@ -1,12 +1,12 @@
import { z } from "zod";
-import { teamMetadataSchema } from "@calcom/prisma/zod-utils";
+import { orgSettingsSchema } from "@calcom/prisma/zod-utils";
export const ZAdminUpdate = z.object({
id: z.number(),
name: z.string().optional(),
slug: z.string().nullish(),
- metadata: teamMetadataSchema.optional(),
+ organizationSettings: orgSettingsSchema.unwrap().optional(),
});
export type TAdminUpdate = z.infer;
diff --git a/packages/trpc/server/routers/viewer/organizations/adminVerify.handler.ts b/packages/trpc/server/routers/viewer/organizations/adminVerify.handler.ts
index 52d795edaf..1e0b5982c2 100644
--- a/packages/trpc/server/routers/viewer/organizations/adminVerify.handler.ts
+++ b/packages/trpc/server/routers/viewer/organizations/adminVerify.handler.ts
@@ -1,5 +1,4 @@
import { prisma } from "@calcom/prisma";
-import { teamMetadataSchema } from "@calcom/prisma/zod-utils";
import { TRPCError } from "@trpc/server";
@@ -17,10 +16,7 @@ export const adminVerifyHandler = async ({ input }: AdminVerifyOptions) => {
const foundOrg = await prisma.team.findFirst({
where: {
id: input.orgId,
- metadata: {
- path: ["isOrganization"],
- equals: true,
- },
+ isOrganization: true,
},
include: {
members: {
@@ -42,17 +38,12 @@ export const adminVerifyHandler = async ({ input }: AdminVerifyOptions) => {
const acceptedEmailDomain = foundOrg.members[0].user.email.split("@")[1];
- const existingMetadataParsed = teamMetadataSchema.parse(foundOrg.metadata);
-
- await prisma.team.update({
+ await prisma.organizationSettings.update({
where: {
- id: input.orgId,
+ organizationId: input.orgId,
},
data: {
- metadata: {
- ...existingMetadataParsed,
- isOrganizationVerified: true,
- },
+ isOrganizationVerified: true,
},
});
diff --git a/packages/trpc/server/routers/viewer/organizations/checkIfOrgNeedsUpgrade.handler.ts b/packages/trpc/server/routers/viewer/organizations/checkIfOrgNeedsUpgrade.handler.ts
index 29e46303fe..5c618ccfd5 100644
--- a/packages/trpc/server/routers/viewer/organizations/checkIfOrgNeedsUpgrade.handler.ts
+++ b/packages/trpc/server/routers/viewer/organizations/checkIfOrgNeedsUpgrade.handler.ts
@@ -33,7 +33,7 @@ export async function checkIfOrgNeedsUpgradeHandler({ ctx }: GetUpgradeableOptio
/** We only need to return teams that don't have a `subscriptionId` on their metadata */
teams = teams.filter((m) => {
const metadata = teamMetadataSchema.safeParse(m.team.metadata);
- if (metadata.success && !metadata.data?.isOrganization) return false;
+ if (m.team.isOrganization) return false;
if (metadata.success && metadata.data?.subscriptionId) return false;
return true;
});
diff --git a/packages/trpc/server/routers/viewer/organizations/create.handler.ts b/packages/trpc/server/routers/viewer/organizations/create.handler.ts
index 365644fb18..f26d5f3e4e 100644
--- a/packages/trpc/server/routers/viewer/organizations/create.handler.ts
+++ b/packages/trpc/server/routers/viewer/organizations/create.handler.ts
@@ -123,12 +123,16 @@ export const createHandler = async ({ input, ctx }: CreateOptions) => {
create: {
name,
...(!IS_TEAM_BILLING_ENABLED ? { slug } : {}),
+ isOrganization: true,
metadata: {
...(IS_TEAM_BILLING_ENABLED ? { requestedSlug: slug } : {}),
- isOrganization: true,
- isOrganizationVerified: true,
- isOrganizationConfigured,
- orgAutoAcceptEmail: emailDomain,
+ },
+ organizationSettings: {
+ create: {
+ isOrganizationVerified: true,
+ isOrganizationConfigured,
+ orgAutoAcceptEmail: emailDomain,
+ },
},
},
},
diff --git a/packages/trpc/server/routers/viewer/organizations/update.schema.ts b/packages/trpc/server/routers/viewer/organizations/update.schema.ts
index 40e577de6d..151f957844 100644
--- a/packages/trpc/server/routers/viewer/organizations/update.schema.ts
+++ b/packages/trpc/server/routers/viewer/organizations/update.schema.ts
@@ -30,7 +30,7 @@ export const ZUpdateInputSchema = z.object({
timeZone: z.string().optional(),
weekStart: z.string().optional(),
timeFormat: z.number().optional(),
- metadata: teamMetadataSchema.unwrap().pick({ isOrganizationConfigured: true }).optional(),
+ metadata: teamMetadataSchema.unwrap().optional(),
});
export type TUpdateInputSchema = z.infer;
diff --git a/packages/trpc/server/routers/viewer/teams/createInvite.handler.ts b/packages/trpc/server/routers/viewer/teams/createInvite.handler.ts
index d1e6315fe2..010782c6d1 100644
--- a/packages/trpc/server/routers/viewer/teams/createInvite.handler.ts
+++ b/packages/trpc/server/routers/viewer/teams/createInvite.handler.ts
@@ -3,7 +3,6 @@ import { randomBytes } from "crypto";
import { WEBAPP_URL } from "@calcom/lib/constants";
import { isTeamAdmin } from "@calcom/lib/server/queries/teams";
import { prisma } from "@calcom/prisma";
-import { teamMetadataSchema } from "@calcom/prisma/zod-utils";
import { TRPCError } from "@calcom/trpc/server";
import type { TrpcSessionUser } from "@calcom/trpc/server/trpc";
@@ -21,8 +20,7 @@ export const createInviteHandler = async ({ ctx, input }: CreateInviteOptions) =
const membership = await isTeamAdmin(ctx.user.id, teamId);
if (!membership || !membership?.team) throw new TRPCError({ code: "UNAUTHORIZED" });
- const teamMetadata = teamMetadataSchema.parse(membership.team.metadata);
- const isOrganizationOrATeamInOrganization = !!(membership.team?.parentId || teamMetadata?.isOrganization);
+ const isOrganizationOrATeamInOrganization = !!(membership.team?.parentId || membership.team.isOrganization);
if (input.token) {
const existingToken = await prisma.verificationToken.findFirst({
diff --git a/packages/trpc/server/routers/viewer/teams/delete.handler.ts b/packages/trpc/server/routers/viewer/teams/delete.handler.ts
index 0076e36a9d..1dcf685703 100644
--- a/packages/trpc/server/routers/viewer/teams/delete.handler.ts
+++ b/packages/trpc/server/routers/viewer/teams/delete.handler.ts
@@ -4,7 +4,6 @@ import { deleteDomain } from "@calcom/lib/domainManager/organization";
import { isTeamOwner } from "@calcom/lib/server/queries/teams";
import { closeComDeleteTeam } from "@calcom/lib/sync/SyncServiceManager";
import { prisma } from "@calcom/prisma";
-import { teamMetadataSchema } from "@calcom/prisma/zod-utils";
import { TRPCError } from "@trpc/server";
@@ -36,9 +35,7 @@ export const deleteHandler = async ({ ctx, input }: DeleteOptions) => {
},
});
- const deletedTeamMetadata = teamMetadataSchema.parse(deletedTeam.metadata);
-
- if (deletedTeamMetadata?.isOrganization && deletedTeam.slug) deleteDomain(deletedTeam.slug);
+ if (deletedTeam?.isOrganization && deletedTeam.slug) deleteDomain(deletedTeam.slug);
// Sync Services: Close.cm
closeComDeleteTeam(deletedTeam);
diff --git a/packages/trpc/server/routers/viewer/teams/getUpgradeable.handler.ts b/packages/trpc/server/routers/viewer/teams/getUpgradeable.handler.ts
index 941c77a11e..af29ce7cf6 100644
--- a/packages/trpc/server/routers/viewer/teams/getUpgradeable.handler.ts
+++ b/packages/trpc/server/routers/viewer/teams/getUpgradeable.handler.ts
@@ -37,7 +37,7 @@ export const getUpgradeableHandler = async ({ ctx }: GetUpgradeableOptions) => {
teams = teams.filter((m) => {
const metadata = teamMetadataSchema.safeParse(m.team.metadata);
if (metadata.success && metadata.data?.subscriptionId) return false;
- if (metadata.success && metadata.data?.isOrganization) return false; // We also dont return ORGs as it will be handled in OrgUpgradeBanner
+ if (m.team.isOrganization) return false; // We also dont return ORGs as it will be handled in OrgUpgradeBanner
if (m.team.children.length > 0) return false; // We also dont return ORGs as it will be handled in OrgUpgradeBanner
return true;
});
diff --git a/packages/trpc/server/routers/viewer/teams/inviteMember/inviteMember.handler.ts b/packages/trpc/server/routers/viewer/teams/inviteMember/inviteMember.handler.ts
index 114c9fca37..b1fec96320 100644
--- a/packages/trpc/server/routers/viewer/teams/inviteMember/inviteMember.handler.ts
+++ b/packages/trpc/server/routers/viewer/teams/inviteMember/inviteMember.handler.ts
@@ -37,23 +37,23 @@ export const inviteMemberHandler = async ({ ctx, input }: InviteMemberOptions) =
await checkRateLimitAndThrowError({
identifier: `invitedBy:${ctx.user.id}`,
});
+ const team = await getTeamOrThrow(input.teamId);
+
+ const isOrg = team.isOrganization;
+
+ if (isOrg && input.role === MembershipRole.OWNER && !(await isOrganisationOwner(ctx.user.id, input.teamId)))
+ throw new TRPCError({ code: "UNAUTHORIZED" });
+
await checkPermissions({
userId: ctx.user.id,
teamId:
ctx.user.organization.id && ctx.user.organization.isOrgAdmin ? ctx.user.organization.id : input.teamId,
- isOrg: input.isOrg,
+ isOrg,
});
+ const { autoAcceptEmailDomain, orgVerified } = getIsOrgVerified(isOrg, team);
// Only owners can award owner role in an organization.
- if (
- input.isOrg &&
- input.role === MembershipRole.OWNER &&
- !(await isOrganisationOwner(ctx.user.id, input.teamId))
- )
- throw new TRPCError({ code: "UNAUTHORIZED" });
- const team = await getTeamOrThrow(input.teamId, input.isOrg);
- const { autoAcceptEmailDomain, orgVerified } = getIsOrgVerified(input.isOrg, team);
const usernameOrEmailsToInvite = await getUsernameOrEmailsToInvite(input.usernameOrEmail);
const orgConnectInfoByUsernameOrEmail = usernameOrEmailsToInvite.reduce((acc, usernameOrEmail) => {
return {
@@ -63,13 +63,13 @@ export const inviteMemberHandler = async ({ ctx, input }: InviteMemberOptions) =
orgAutoAcceptDomain: autoAcceptEmailDomain,
usersEmail: usernameOrEmail,
team,
- isOrg: input.isOrg,
+ isOrg: isOrg,
}),
};
}, {} as Record>);
const existingUsersWithMembersips = await getUsersToInvite({
usernamesOrEmails: usernameOrEmailsToInvite,
- isInvitedToOrg: input.isOrg,
+ isInvitedToOrg: isOrg,
team,
});
const existingUsersEmailsAndUsernames = existingUsersWithMembersips.reduce(
@@ -153,7 +153,7 @@ export const inviteMemberHandler = async ({ ctx, input }: InviteMemberOptions) =
currentUserTeamName: team?.name,
existingUsersWithMembersips: regularUsers,
language: translation,
- isOrg: input.isOrg,
+ isOrg: isOrg,
teamId: team.id,
currentUserParentTeamName: team?.parent?.name,
});
diff --git a/packages/trpc/server/routers/viewer/teams/inviteMember/inviteMemberUtils.test.ts b/packages/trpc/server/routers/viewer/teams/inviteMember/inviteMemberUtils.test.ts
index d7207b8505..c3ac129944 100644
--- a/packages/trpc/server/routers/viewer/teams/inviteMember/inviteMemberUtils.test.ts
+++ b/packages/trpc/server/routers/viewer/teams/inviteMember/inviteMemberUtils.test.ts
@@ -61,6 +61,8 @@ const mockedTeam: TeamWithParent = {
parent: null,
isPrivate: false,
logoUrl: "",
+ isOrganization: false,
+ calVideoLogo: "",
};
const mockUser: Invitee = {
@@ -208,7 +210,10 @@ describe("Invite Member Utils", () => {
describe("getIsOrgVerified", () => {
it("should return the correct values when isOrg is true and teamMetadata.orgAutoAcceptEmail is true", () => {
const team = {
- metadata: {
+ organizationSettings: {
+ id: 1,
+ teamId: 1,
+ isOrganizationConfigured: false,
isOrganizationVerified: true,
orgAutoAcceptEmail: "example.com",
},
@@ -227,7 +232,10 @@ describe("Invite Member Utils", () => {
metadata: {},
parent: {
...mockedTeam,
- metadata: {
+ organizationSettings: {
+ id: 1,
+ teamId: 1,
+ isOrganizationConfigured: false,
isOrganizationVerified: false,
orgAutoAcceptEmail: "example.com",
},
diff --git a/packages/trpc/server/routers/viewer/teams/inviteMember/utils.ts b/packages/trpc/server/routers/viewer/teams/inviteMember/utils.ts
index 5768055791..6a89754dfe 100644
--- a/packages/trpc/server/routers/viewer/teams/inviteMember/utils.ts
+++ b/packages/trpc/server/routers/viewer/teams/inviteMember/utils.ts
@@ -8,10 +8,9 @@ import { isTeamAdmin } from "@calcom/lib/server/queries";
import { isOrganisationAdmin } from "@calcom/lib/server/queries/organisations";
import slugify from "@calcom/lib/slugify";
import { prisma } from "@calcom/prisma";
-import type { Membership, Team } from "@calcom/prisma/client";
+import type { Membership, OrganizationSettings, Team } from "@calcom/prisma/client";
import { Prisma, type User } from "@calcom/prisma/client";
import { MembershipRole } from "@calcom/prisma/enums";
-import { teamMetadataSchema } from "@calcom/prisma/zod-utils";
import { TRPCError } from "@trpc/server";
@@ -54,18 +53,26 @@ export function checkInputEmailIsValid(email: string) {
});
}
-export async function getTeamOrThrow(teamId: number, isOrg?: boolean) {
+export async function getTeamOrThrow(teamId: number) {
const team = await prisma.team.findFirst({
where: {
id: teamId,
},
include: {
- parent: true,
+ organizationSettings: true,
+ parent: {
+ include: {
+ organizationSettings: true,
+ },
+ },
},
});
if (!team)
- throw new TRPCError({ code: "NOT_FOUND", message: `${isOrg ? "Organization" : "Team"} not found` });
+ throw new TRPCError({
+ code: "NOT_FOUND",
+ message: `Team not found`,
+ });
return team;
}
@@ -385,27 +392,29 @@ export async function sendVerificationEmail({
}
}
+type TeamAndOrganizationSettings = Team & {
+ organizationSettings?: OrganizationSettings | null;
+};
+
export function getIsOrgVerified(
isOrg: boolean,
- team: Team & {
- parent: Team | null;
+ team: TeamAndOrganizationSettings & {
+ parent: TeamAndOrganizationSettings | null;
}
) {
- const teamMetadata = teamMetadataSchema.parse(team.metadata);
- const orgMetadataSafeParse = teamMetadataSchema.safeParse(team.parent?.metadata);
- const orgMetadataIfExists = orgMetadataSafeParse.success ? orgMetadataSafeParse.data : null;
+ const parentSettings = team.parent?.organizationSettings;
- if (isOrg && teamMetadata?.orgAutoAcceptEmail) {
+ if (isOrg && team.organizationSettings?.orgAutoAcceptEmail) {
return {
isInOrgScope: true,
- orgVerified: teamMetadata.isOrganizationVerified,
- autoAcceptEmailDomain: teamMetadata.orgAutoAcceptEmail,
+ orgVerified: team.organizationSettings.isOrganizationVerified,
+ autoAcceptEmailDomain: team.organizationSettings.orgAutoAcceptEmail,
};
- } else if (orgMetadataIfExists?.orgAutoAcceptEmail) {
+ } else if (parentSettings?.orgAutoAcceptEmail) {
return {
isInOrgScope: true,
- orgVerified: orgMetadataIfExists.isOrganizationVerified,
- autoAcceptEmailDomain: orgMetadataIfExists.orgAutoAcceptEmail,
+ orgVerified: parentSettings.isOrganizationVerified,
+ autoAcceptEmailDomain: parentSettings.orgAutoAcceptEmail,
};
}
diff --git a/packages/trpc/server/routers/viewer/teams/list.handler.ts b/packages/trpc/server/routers/viewer/teams/list.handler.ts
index 23a29411ff..946d3a1498 100644
--- a/packages/trpc/server/routers/viewer/teams/list.handler.ts
+++ b/packages/trpc/server/routers/viewer/teams/list.handler.ts
@@ -1,5 +1,4 @@
import { prisma } from "@calcom/prisma";
-import { teamMetadataSchema } from "@calcom/prisma/zod-utils";
import type { TrpcSessionUser } from "../../../trpc";
@@ -20,9 +19,15 @@ export const listHandler = async ({ ctx }: ListOptions) => {
},
include: {
team: {
- include: {
+ select: {
+ id: true,
+ name: true,
+ slug: true,
+ logo: true,
+ isOrganization: true,
inviteTokens: true,
parent: true,
+ parentId: true,
},
},
},
@@ -31,8 +36,7 @@ export const listHandler = async ({ ctx }: ListOptions) => {
return memberships
.filter((mmship) => {
- const metadata = teamMetadataSchema.parse(mmship.team.metadata);
- return !metadata?.isOrganization;
+ return !mmship.team.isOrganization;
})
.map(({ team: { inviteTokens, ..._team }, ...membership }) => ({
role: membership.role,
diff --git a/packages/trpc/server/routers/viewer/teams/listOwnedTeams.handler.ts b/packages/trpc/server/routers/viewer/teams/listOwnedTeams.handler.ts
index e28927a879..878228944e 100644
--- a/packages/trpc/server/routers/viewer/teams/listOwnedTeams.handler.ts
+++ b/packages/trpc/server/routers/viewer/teams/listOwnedTeams.handler.ts
@@ -1,6 +1,5 @@
import { prisma } from "@calcom/prisma";
import { MembershipRole } from "@calcom/prisma/enums";
-import { teamMetadataSchema } from "@calcom/prisma/zod-utils";
import type { TrpcSessionUser } from "../../../trpc";
@@ -33,8 +32,7 @@ export const listOwnedTeamsHandler = async ({ ctx }: ListOptions) => {
return user?.teams
?.filter((m) => {
- const metadata = teamMetadataSchema.parse(m.team.metadata);
- return !metadata?.isOrganization;
+ return !m.team.isOrganization;
})
?.map(({ team }) => team);
};
diff --git a/packages/trpc/server/routers/viewer/teams/removeMember.handler.ts b/packages/trpc/server/routers/viewer/teams/removeMember.handler.ts
index b3171438bd..0124a4fe18 100644
--- a/packages/trpc/server/routers/viewer/teams/removeMember.handler.ts
+++ b/packages/trpc/server/routers/viewer/teams/removeMember.handler.ts
@@ -4,7 +4,6 @@ import { IS_TEAM_BILLING_ENABLED } from "@calcom/lib/constants";
import { isTeamAdmin, isTeamOwner } from "@calcom/lib/server/queries/teams";
import { closeComDeleteTeamMembership } from "@calcom/lib/sync/SyncServiceManager";
import type { PrismaClient } from "@calcom/prisma";
-import { teamMetadataSchema } from "@calcom/prisma/zod-utils";
import type { TrpcSessionUser } from "@calcom/trpc/server/trpc";
import { TRPCError } from "@trpc/server";
@@ -75,20 +74,17 @@ export const removeMemberHandler = async ({ ctx, input }: RemoveMemberOptions) =
const orgInfo = await ctx.prisma.team.findUnique({
where: { id: input.teamId },
select: {
+ isOrganization: true,
+ organizationSettings: true,
metadata: true,
},
});
+ const orgMetadata = orgInfo?.organizationSettings;
if (!foundUser || !orgInfo) throw new TRPCError({ code: "NOT_FOUND" });
- const parsedMetadata = teamMetadataSchema.parse(orgInfo.metadata);
-
- if (
- parsedMetadata?.isOrganization &&
- parsedMetadata.isOrganizationVerified &&
- parsedMetadata.orgAutoAcceptEmail
- ) {
- if (foundUser.email.endsWith(parsedMetadata.orgAutoAcceptEmail)) {
+ if (orgInfo?.isOrganization && orgMetadata?.isOrganizationVerified && orgMetadata?.orgAutoAcceptEmail) {
+ if (foundUser.email.endsWith(orgMetadata.orgAutoAcceptEmail)) {
await ctx.prisma.user.delete({
where: { id: input.memberId },
});
diff --git a/packages/trpc/server/routers/viewer/teams/resendInvitation.handler.ts b/packages/trpc/server/routers/viewer/teams/resendInvitation.handler.ts
index a3390fb8eb..0985e1f7a7 100644
--- a/packages/trpc/server/routers/viewer/teams/resendInvitation.handler.ts
+++ b/packages/trpc/server/routers/viewer/teams/resendInvitation.handler.ts
@@ -15,7 +15,7 @@ type InviteMemberOptions = {
};
export const resendInvitationHandler = async ({ ctx, input }: InviteMemberOptions) => {
- const team = await getTeamOrThrow(input.teamId, input.isOrg);
+ const team = await getTeamOrThrow(input.teamId);
await checkPermissions({
userId: ctx.user.id,
diff --git a/packages/trpc/server/routers/viewer/webhook/getByViewer.handler.ts b/packages/trpc/server/routers/viewer/webhook/getByViewer.handler.ts
index e0a60bd8b9..afc90cd5f1 100644
--- a/packages/trpc/server/routers/viewer/webhook/getByViewer.handler.ts
+++ b/packages/trpc/server/routers/viewer/webhook/getByViewer.handler.ts
@@ -2,7 +2,6 @@ import { getBookerBaseUrl } from "@calcom/lib/getBookerUrl/server";
import { prisma } from "@calcom/prisma";
import type { Webhook } from "@calcom/prisma/client";
import { MembershipRole } from "@calcom/prisma/enums";
-import { teamMetadataSchema } from "@calcom/prisma/zod-utils";
import type { TrpcSessionUser } from "@calcom/trpc/server/trpc";
import { TRPCError } from "@trpc/server";
@@ -68,6 +67,7 @@ export const getByViewerHandler = async ({ ctx }: GetByViewerOptions) => {
team: {
select: {
id: true,
+ isOrganization: true,
name: true,
slug: true,
parentId: true,
@@ -115,8 +115,7 @@ export const getByViewerHandler = async ({ ctx }: GetByViewerOptions) => {
const teamWebhookGroups: WebhookGroup[] = user.teams
.filter((mmship) => {
- const metadata = teamMetadataSchema.parse(mmship.team.metadata);
- return !metadata?.isOrganization;
+ return !mmship.team.isOrganization;
})
.map((membership) => {
const orgMembership = teamMemberships.find(