fix: error handling in the thingy mabob for websites

Author: izadoesdev

apps/dashboard/components/website-dialog.tsx

@@ -103,11 +103,33 @@ export function WebsiteDialog({
 				toast.success('Website created successfully!');
 			}
 			onOpenChange(false);
-		} catch (error) {
-			const message =
-				error.data?.code === 'CONFLICT'
-					? 'A website with this domain already exists.'
-					: `Failed to ${isEditing ? 'update' : 'create'} website.`;
+		} catch (error: any) {
+			let message = `Failed to ${isEditing ? 'update' : 'create'} website.`;
+
+			if (error?.data?.code) {
+				switch (error.data.code) {
+					case 'CONFLICT':
+						message = 'A website with this domain already exists.';
+						break;
+					case 'FORBIDDEN':
+						message =
+							error.message ||
+							'You do not have permission to perform this action.';
+						break;
+					case 'UNAUTHORIZED':
+						message = 'You must be logged in to perform this action.';
+						break;
+					case 'BAD_REQUEST':
+						message =
+							error.message || 'Invalid request. Please check your input.';
+						break;
+					default:
+						message = error.message || message;
+				}
+			} else if (error?.message) {
+				message = error.message;
+			}
+
 			toast.error(message);
 		}
 	});

packages/rpc/src/routers/websites.ts

@@ -12,11 +12,7 @@ import { nanoid } from 'nanoid';
 import { z } from 'zod';
 import { createTRPCRouter, protectedProcedure, publicProcedure } from '../trpc';
 import { authorizeWebsiteAccess } from '../utils/auth';
-import {
-	checkAndTrackWebsiteCreation,
-	getBillingCustomerId,
-	trackWebsiteUsage,
-} from '../utils/billing';
+
 import { invalidateWebsiteCaches } from '../utils/cache-invalidation';
 
 const websiteCache = createDrizzleCache({ redis, namespace: 'websites' });
@@ -254,20 +250,6 @@ export const websitesRouter = createTRPCRouter({
 				}
 			}
 
-			const billingCustomerId = await getBillingCustomerId(
-				ctx.user.id,
-				input.organizationId
-			);
-
-			const creationLimitCheck =
-				await checkAndTrackWebsiteCreation(billingCustomerId);
-			if (!creationLimitCheck.allowed) {
-				throw new TRPCError({
-					code: 'BAD_REQUEST',
-					message: creationLimitCheck.error,
-				});
-			}
-
 			const domainToCreate = buildFullDomain(input.domain, input.subdomain);
 			const websiteFilter = and(
 				eq(websites.domain, domainToCreate),
@@ -391,16 +373,8 @@ export const websitesRouter = createTRPCRouter({
 				input.id,
 				'delete'
 			);
-			const billingCustomerId = await getBillingCustomerId(
-				ctx.user.id,
-				websiteToDelete.organizationId
-			);
-
 			await ctx.db.transaction(async (tx) => {
 				await tx.delete(websites).where(eq(websites.id, input.id));
-
-				// Track billing usage (decrement)
-				await trackWebsiteUsage(billingCustomerId, -1);
 			});
 
 			logger.warning(