Merge pull request #46 from Saasfy/fix-workflow-filtering

fix(web): fix workspace access problems

Author: IKatsuba

.github/FUNDING.yml

@@ -1,6 +1,6 @@
 # These are supported funding model platforms
 
-github: [IKatsuba,Saasfy]
+github: [IKatsuba, Saasfy]
 patreon: # Replace with a single Patreon username
 open_collective: # Replace with a single Open Collective username
 ko_fi: # Replace with a single Ko-fi username

apps/web/app/app/(dashboard)/[workspaceSlug]/page.tsx

@@ -20,7 +20,7 @@ export default async function Component({ params }: { params: { workspaceSlug: s
 
   const { data: workspace } = await supabase
     .from('workspaces')
-    .select('*, workspace_users(*)')
+    .select('*, workspace_users!inner(*)')
     .eq('slug', params.workspaceSlug)
     .eq('workspace_users.user_id', user.id)
     .single();

apps/web/app/app/(dashboard)/[workspaceSlug]/settings/domains/page.tsx

@@ -1,29 +1,35 @@
+import { redirect } from 'next/navigation';
+
 import { ExternalLinkIcon } from 'lucide-react';
 
-import { createAdminClient } from '@saasfy/supabase/server';
+import { createAdminClient, getUser } from '@saasfy/supabase/server';
 import { Badge } from '@saasfy/ui/badge';
 
 import { AddDomainForm } from './add-form';
 import { DomainConfiguration } from './domain-configuration';
 import { RemoveDomainButton } from './remove-domain-button';
 
 export default async function Component({ params }: { params: { workspaceSlug: string } }) {
+  const user = await getUser();
+
+  if (!user) {
+    return redirect('/login');
+  }
+
   const supabase = createAdminClient();
 
   const { data: workspace } = await supabase
     .from('workspaces')
-    .select('*')
+    .select('*, workspace_users!inner(*), domains(*)')
     .eq('slug', params.workspaceSlug)
+    .eq('workspace_users.user_id', user.id)
     .single();
 
   if (!workspace) {
     return null;
   }
 
-  const { data: domains, error } = await supabase
-    .from('domains')
-    .select('*')
-    .eq('workspace_id', workspace.id);
+  const domains = workspace.domains;
 
   return (
     <div className="p-8">

apps/web/app/app/(dashboard)/[workspaceSlug]/settings/layout.tsx

@@ -1,7 +1,7 @@
 import { ReactNode } from 'react';
 import { redirect } from 'next/navigation';
 
-import { createAdminClient } from '@saasfy/supabase/server';
+import { createAdminClient, getUser } from '@saasfy/supabase/server';
 
 import { Nav } from './nav';
 
@@ -12,11 +12,18 @@ export default async function SettingsLayout({
   children: ReactNode;
   params: { workspaceSlug: string };
 }) {
+  const user = await getUser();
+
+  if (!user) {
+    return redirect(`/signin/signin`);
+  }
+
   const supabase = createAdminClient();
 
   const { data: workspace } = await supabase
     .from('workspaces')
-    .select('*')
+    .select('*, workspace_users!inner(*)')
+    .eq('workspace_users.user_id', user.id)
     .eq('slug', params.workspaceSlug)
     .single();
 

apps/web/app/app/(dashboard)/[workspaceSlug]/settings/members/page.tsx

@@ -4,7 +4,7 @@ import { redirect } from 'next/navigation';
 import { FilterIcon, PlusIcon } from 'lucide-react';
 import postgres from 'postgres';
 
-import { createAdminClient, Tables } from '@saasfy/supabase/server';
+import { createAdminClient, getUser, Tables } from '@saasfy/supabase/server';
 import { Button } from '@saasfy/ui/button';
 import {
   DropdownMenu,
@@ -22,12 +22,19 @@ import { InviteTable } from './invite-table';
 import { MemberTable } from './member-table';
 
 export default async function Component({ params }: { params: { workspaceSlug: string } }) {
+  const user = await getUser();
+
+  if (!user) {
+    return redirect('/login');
+  }
+
   const supabase = createAdminClient();
 
   const { data: workspace } = await supabase
     .from('workspaces')
-    .select('*')
+    .select('*, workspace_users!inner(*)')
     .eq('slug', params.workspaceSlug)
+    .eq('workspace_users.user_id', user.id)
     .single();
 
   if (!workspace) {

apps/web/app/app/(dashboard)/[workspaceSlug]/settings/page.tsx

@@ -23,7 +23,7 @@ export default async function Settings({ params }: { params: { workspaceSlug: st
 
   const { data: workspace } = await supabase
     .from('workspaces')
-    .select('*, workspace_users(*)')
+    .select('*, workspace_users!inner(*)')
     .eq('slug', params.workspaceSlug)
     .eq('workspace_users.user_id', user.id)
     .single();

apps/web/app/app/(dashboard)/layout.tsx

@@ -20,7 +20,7 @@ export default async function DashboardLayout({ children }: { children: ReactNod
 
   const { data: workspaces } = await supabase
     .from('workspaces')
-    .select('*, workspace_users(*)')
+    .select('*, workspace_users!inner(*)')
     .eq('workspace_users.user_id', user.id);
 
   return (

apps/web/app/app/(dashboard)/page.tsx

@@ -20,7 +20,7 @@ export default async function Component() {
   const supabase = createAdminClient();
   const { data: workspaces } = await supabase
     .from('workspaces')
-    .select('*, projects(id), domains(id), workspace_users(id), plans(name)')
+    .select('*, projects(id), domains(id), workspace_users!inner(id), plans(name)')
     .eq('workspace_users.user_id', user.id)
     .limit(8);
 

packages/api/src/lib/with-workspace-user.ts

@@ -24,7 +24,7 @@ export function withWorkspaceUser<T>(
 
     const { data: workspace } = await supabase
       .from('workspaces')
-      .select('*, workspace_users(*)')
+      .select('*, workspace_users!inner(*)')
       .eq('slug', params.workspaceSlug)
       .eq('workspace_users.user_id', user.id)
       .in('workspace_users.role', roles)

packages/components/src/lib/create-project/actions.ts

@@ -46,7 +46,7 @@ export async function createProject(formData: FormData) {
 
     const { data: workspace } = await supabase
       .from('workspaces')
-      .select('*, workspace_users(*)')
+      .select('*, workspace_users!inner(*)')
       .eq('slug', workspaceSlug)
       .eq('workspace_users.user_id', user.id)
       .single();