Skip to content

fix: make admin panel responsive #238

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

Open
AmzBG wants to merge 1 commit into
base: mhmdfarhat-mhmdali-amir
Choose a base branch
from
Open

fix: make admin panel responsive #238

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
257 changes: 182 additions & 75 deletions apps/web/app/(authenticated)/admin/accounts/page.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -102,16 +102,23 @@ export default function AdminAccountsPage() {
description="Track developer and hiring accounts, connections, activity, and access status."
/>
<Card>
<CardContent className="space-y-5 pt-6">
<CardContent className="space-y-5 px-4 pt-6 sm:px-6">
<div className="flex flex-col gap-3 lg:flex-row">
<form className="flex flex-1 gap-2" onSubmit={submitSearch}>
<form
className="flex flex-1 flex-col gap-2 sm:flex-row"
onSubmit={submitSearch}
>
<Input
value={searchInput}
onChange={(event) => setSearchInput(event.target.value)}
placeholder="Search email, name, organization, or GitHub username"
aria-label="Search accounts"
/>
<Button type="submit" variant="outline">
<Button
type="submit"
variant="outline"
className="w-full sm:w-auto"
>
Search
</Button>
</form>
Comment on lines +107 to 124

@coderabbitai coderabbitai Bot Aug 11, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

📐 Maintainability & Code Quality | 🟠 Major | 🏗️ Heavy lift

Use the required form pattern.

Replace these controlled native forms with react-hook-form and zodResolver. Pass each schema from @repo/contracts. Keep the page-reset behavior in the valid submit handler.

  • apps/web/app/(authenticated)/admin/accounts/page.tsx#L107-L124: Use the account-search schema and handleSubmit.
  • apps/web/app/(authenticated)/admin/projects/page.tsx#L105-L122: Use the project-search schema and handleSubmit.
  • apps/web/app/(authenticated)/admin/logs/page.tsx#L78-L95: Use the audit-log-search schema and handleSubmit.

As per coding guidelines: apps/web/app/**/*.tsx: Use react-hook-form with zodResolver for forms, passing the schema from @repo/contracts.

📍 Affects 3 files
  • apps/web/app/(authenticated)/admin/accounts/page.tsx#L107-L124 (this comment)
  • apps/web/app/(authenticated)/admin/projects/page.tsx#L105-L122
  • apps/web/app/(authenticated)/admin/logs/page.tsx#L78-L95
🤖 Prompt for AI Agents 
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@apps/web/app/`(authenticated)/admin/accounts/page.tsx around lines 107 - 124,
Replace the controlled native search forms with react-hook-form configured
through zodResolver and the corresponding schema from `@repo/contracts`: use the
account-search schema in apps/web/app/(authenticated)/admin/accounts/page.tsx
(107-124), the project-search schema in
apps/web/app/(authenticated)/admin/projects/page.tsx (105-122), and the
audit-log-search schema in apps/web/app/(authenticated)/admin/logs/page.tsx
(78-95). Submit each form through handleSubmit and preserve the existing
page-reset behavior in the valid submit handlers.

Source: Coding guidelines

Expand Down Expand Up @@ -159,89 +166,189 @@ export default function AdminAccountsPage() {
No accounts match these filters.
</p>
) : (
<div className="overflow-x-auto">
<Table>
<TableHeader>
<TableRow>
<TableHead>Account</TableHead>
<TableHead>Type</TableHead>
<TableHead>Status</TableHead>
<TableHead>Activity</TableHead>
<TableHead>Joined</TableHead>
<TableHead className="text-right">Action</TableHead>
</TableRow>
</TableHeader>
<TableBody>
{response.data.map((account) => (
<TableRow key={account.id}>
<TableCell>
<div className="font-medium">{account.displayName}</div>
<div className="text-muted-foreground text-xs">
<>
<div className="grid gap-3 md:hidden">
{response.data.map((account) => (
<article key={account.id} className="rounded-lg border p-4">
<div className="flex min-w-0 items-start justify-between gap-3">
<div className="min-w-0">
<div className="break-words font-medium">
{account.displayName}
</div>
<div className="break-all text-xs text-muted-foreground">
{account.email}
</div>
{account.githubUsername && (
<div className="text-primary-base text-xs">
<div className="mt-1 break-all text-xs text-primary-base">
@{account.githubUsername}
</div>
)}
</TableCell>
<TableCell>
<StatusBadge tone="info">
</div>
<StatusBadge
tone={
account.status === 'ACTIVE' ? 'success' : 'danger'
}
>
{account.status}
</StatusBadge>
</div>

<div className="mt-4 grid grid-cols-2 gap-3 text-xs">
<div>
<div className="text-muted-foreground">Type</div>
<div className="mt-1 font-medium">
{account.accountType.replace('_', ' ')}
</StatusBadge>
</TableCell>
<TableCell>
<div className="flex flex-col items-start gap-1">
<StatusBadge
tone={
account.status === 'ACTIVE' ? 'success' : 'danger'
}
>
{account.status}
</StatusBadge>
{!account.isConfirmed && (
<span className="text-warning-dark text-xs">
Unconfirmed
</span>
)}
</div>
</TableCell>
<TableCell className="text-muted-foreground text-xs">
<div>{account.counts.ownedProjects} owned projects</div>
<div>
{account.counts.projectMemberships} collaborations
</div>
<div>
<div className="text-muted-foreground">Joined</div>
<div className="mt-1 font-medium">
{new Date(account.createdAt).toLocaleDateString()}
</div>
<div>
{account.counts.activeSessions} active sessions
</div>
</div>

<div className="mt-4 grid grid-cols-3 gap-2 rounded-md bg-muted/40 p-3 text-center text-xs">
<div>
<div className="font-semibold">
{account.counts.ownedProjects}
</div>
</TableCell>
<TableCell className="text-muted-foreground text-xs">
{new Date(account.createdAt).toLocaleDateString()}
</TableCell>
<TableCell className="text-right">
{account.accountType === 'SUPER_ADMIN' ? (
<StatusBadge>Protected</StatusBadge>
) : (
<Button
size="sm"
variant={
account.status === 'ACTIVE'
? 'destructive'
: 'outline'
}
onClick={() => setSelected(account)}
>
{account.status === 'ACTIVE'
? 'Suspend'
: 'Reactivate'}
</Button>
)}
</TableCell>
<div className="text-muted-foreground">Owned</div>
</div>
<div>
<div className="font-semibold">
{account.counts.projectMemberships}
</div>
<div className="text-muted-foreground">Teams</div>
</div>
<div>
<div className="font-semibold">
{account.counts.activeSessions}
</div>
<div className="text-muted-foreground">Sessions</div>
</div>
</div>

{!account.isConfirmed && (
<p className="mt-3 text-xs text-warning-dark">
Email unconfirmed
</p>
)}

<div className="mt-4">
{account.accountType === 'SUPER_ADMIN' ? (
<StatusBadge>Protected</StatusBadge>
) : (
<Button
className="w-full"
size="sm"
variant={
account.status === 'ACTIVE'
? 'destructive'
: 'outline'
}
onClick={() => setSelected(account)}
>
{account.status === 'ACTIVE'
? 'Suspend'
: 'Reactivate'}
</Button>
)}
</div>
</article>
))}
</div>

<div className="hidden overflow-x-auto md:block">
<Table>
<TableHeader>
<TableRow>
<TableHead>Account</TableHead>
<TableHead>Type</TableHead>
<TableHead>Status</TableHead>
<TableHead>Activity</TableHead>
<TableHead>Joined</TableHead>
<TableHead className="text-right">Action</TableHead>
</TableRow>
))}
</TableBody>
</Table>
</div>
</TableHeader>
<TableBody>
{response.data.map((account) => (
<TableRow key={account.id}>
<TableCell>
<div className="font-medium">
{account.displayName}
</div>
<div className="text-muted-foreground text-xs">
{account.email}
</div>
{account.githubUsername && (
<div className="text-primary-base text-xs">
@{account.githubUsername}
</div>
)}
</TableCell>
<TableCell>
<StatusBadge tone="info">
{account.accountType.replace('_', ' ')}
</StatusBadge>
</TableCell>
<TableCell>
<div className="flex flex-col items-start gap-1">
<StatusBadge
tone={
account.status === 'ACTIVE'
? 'success'
: 'danger'
}
>
{account.status}
</StatusBadge>
{!account.isConfirmed && (
<span className="text-warning-dark text-xs">
Unconfirmed
</span>
)}
</div>
</TableCell>
<TableCell className="text-muted-foreground text-xs">
<div>
{account.counts.ownedProjects} owned projects
</div>
<div>
{account.counts.projectMemberships} collaborations
</div>
<div>
{account.counts.activeSessions} active sessions
</div>
</TableCell>
<TableCell className="text-muted-foreground text-xs">
{new Date(account.createdAt).toLocaleDateString()}
</TableCell>
<TableCell className="text-right">
{account.accountType === 'SUPER_ADMIN' ? (
<StatusBadge>Protected</StatusBadge>
) : (
<Button
size="sm"
variant={
account.status === 'ACTIVE'
? 'destructive'
: 'outline'
}
onClick={() => setSelected(account)}
>
{account.status === 'ACTIVE'
? 'Suspend'
: 'Reactivate'}
</Button>
)}
</TableCell>
</TableRow>
))}
</TableBody>
</Table>
</div>
</>
)}
{response && (
<Pagination
Expand Down
2 changes: 1 addition & 1 deletion apps/web/app/(authenticated)/admin/layout.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,5 +23,5 @@ export default function AdminLayout({
return <Skeleton className="h-[520px] w-full rounded-xl" />;
}

return children;
return <div className="min-w-0">{children}</div>;
}
Loading
Loading