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.

Sign up for GitHub

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

Jump to bottom

fix: create platform handler isPlatform flag #14288

Merged
merged 9 commits into from
Apr 5, 2024
Merged

fix: create platform handler isPlatform flag #14288

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
65ad3d9
fix: create platform handler
supalarry Apr 2, 2024
e292467
Merge branch 'main' into fix-create-platform-handler
supalarry Apr 2, 2024
bca80d3
Merge branch 'main' into fix-create-platform-handler
keithwillcode Apr 2, 2024
97f4cf0
Merge branch 'main' into fix-create-platform-handler
supalarry Apr 2, 2024
386968f
Merge branch 'main' into fix-create-platform-handler
zomars Apr 4, 2024
8c65947
Update organization.ts
zomars Apr 4, 2024
76fc4e9
Merge branch 'main' into fix-create-platform-handler
supalarry Apr 4, 2024
e62a8fa
Merge branch 'main' into fix-create-platform-handler
supalarry Apr 5, 2024
b1f11bc
Merge branch 'main' into fix-create-platform-handler
supalarry Apr 5, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
2 changes: 2 additions & 0 deletions packages/lib/server/repository/organization.ts
View file
Open in desktop
Expand Up @@ -25,6 +25,7 @@ export class OrganizationRepository {
autoAcceptEmail: string;
seats: number | null;
pricePerSeat: number | null;
isPlatform: boolean;
};
owner: {
id: number;
Expand All @@ -50,6 +51,7 @@ export class OrganizationRepository {
...(IS_TEAM_BILLING_ENABLED ? { requestedSlug: orgData.slug } : {}),
orgSeats: orgData.seats,
orgPricePerSeat: orgData.pricePerSeat,
isPlatform: orgData.isPlatform,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@zomars Why was this moved to metadata here? There is a standalone column isPlatform on the team table that the platform team has intended to use.

},
},
});
Expand Down
1 change: 1 addition & 0 deletions packages/trpc/server/routers/viewer/organizations/create.handler.ts
View file
Open in desktop
Expand Up @@ -110,6 +110,7 @@ export const createHandler = async ({ input, ctx }: CreateOptions) => {
autoAcceptEmail,
seats: seats ?? null,
pricePerSeat: pricePerSeat ?? null,
isPlatform,
},
owner: {
id: orgOwner.id,
Expand Down