Skip to content

refactor: Clean up dashboard page and remove unused styles from global #30

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

Merged
merged 2 commits into from
Oct 2, 2025
+12 −77
Merged

refactor: Clean up dashboard page and remove unused styles from global #30

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
2c29d7a
refactor: Clean up dashboard page and remove unused styles from globa…
ginzahatemi Oct 2, 2025
08629bd
Merge branch 'main' into cleanuo
tasin2610 Oct 2, 2025
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
3 changes: 2 additions & 1 deletion package-lock.json
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
"private": true,
"type": "module",
"scripts": {
"dev": "next dev --turbopack",
"dev": "next dev",
"build": "prisma generate && next build --turbopack",
Comment on lines +7 to 8
Copy link

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟠 Major

Inconsistent turbopack usage between dev and build.

The dev script no longer uses --turbopack, but the build script still does. This creates an inconsistency where development and production builds use different compilation pipelines, potentially causing divergence in behavior, bundle output, or build errors that only surface in one environment.

Consider one of these approaches:

  1. Remove turbopack from both (if deprecating turbopack):
-    "dev": "next dev",
-    "build": "prisma generate && next build --turbopack",
+    "dev": "next dev",
+    "build": "prisma generate && next build",
  1. Keep turbopack in both (if using turbopack):
-    "dev": "next dev",
+    "dev": "next dev --turbopack",
     "build": "prisma generate && next build --turbopack",
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
"dev": "next dev",
"build": "prisma generate && next build --turbopack",
"dev": "next dev",
"build": "prisma generate && next build",
Suggested change
"dev": "next dev",
"build": "prisma generate && next build --turbopack",
"dev": "next dev --turbopack",
"build": "prisma generate && next build --turbopack",
🤖 Prompt for AI Agents 
In package.json around lines 7-8, the dev and build scripts are inconsistent:
dev uses "next dev" while build uses "next build --turbopack"; make them
consistent by choosing one approach and updating scripts accordingly — either
remove "--turbopack" from the build script so both use classic Next.js ("dev":
"next dev", "build": "prisma generate && next build"), or add "--turbopack" to
the dev script so both use Turbopack ("dev": "next dev --turbopack", "build":
"prisma generate && next build --turbopack"); update package.json scripts to
reflect the chosen approach and run a quick local dev/build to verify no flags
are missing.

"postinstall": "prisma generate",
"start": "next start",
Expand Down
14 changes: 9 additions & 5 deletions src/app/dashboard/page.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,13 +1,17 @@
"use client";

// import { useSession } from "next-auth/react";
import { useRouter } from "next/navigation";
import { useEffect } from "react";

export default function DashboardPage() {
// const { data: session, status } = useSession();

return (
<div className="full-vh flex-center">
<div className="login-card">
<h1 className="login-title">Welcome!</h1>
<p className="login-desc">
Dashboard is temporarily available without authentication.
</p>
<div>
<h1>Welcome!</h1>
<p>You are now signed in to SentioPulse.</p>
</div>
</div>
);
Expand Down
70 changes: 0 additions & 70 deletions src/app/globals.css
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,77 +1,7 @@
/* Login page background */
.login-bg {
background: linear-gradient(90deg, #e0e7ff 0%, #f9fafb 100%);
}

/* Flex utilities */
.flex-center {
display: flex;
align-items: center;
justify-content: center;
}
.flex-col-center {
display: flex;
flex-direction: column;
align-items: center;
}

/* Full viewport height utility */
.full-vh {
min-height: 100vh;
}

/* Login card styles */
.login-card {
background: #fff;
padding: 48px 36px;
border-radius: 16px;
box-shadow: 0 8px 32px rgba(0,36,100,0.08);
}

.login-title {
font-size: 2rem;
font-weight: 700;
margin-bottom: 12px;
color: #334155;
}

.login-desc {
color: #64748b;
margin-bottom: 28px;
}

/* Google button styles */
.google-btn {
display: flex;
align-items: center;
justify-content: center;
gap: 10px;
background: #fff;
color: #222;
border: 1px solid #dadada;
border-radius: 8px;
padding: 12px 28px;
font-weight: 500;
font-size: 18px;
box-shadow: 0 2px 8px rgba(64,86,120,0.07);
cursor: pointer;
transition: box-shadow 0.2s, border 0.2s;
outline: none;
}
.google-btn:hover {
box-shadow: 0 4px 14px rgba(0,0,0,.10);
}

.google-logo {
background: #fff;
border-radius: 50%;
}

@import "tailwindcss";
@import "tw-animate-css";

@custom-variant dark (&:is(.dark *));

@theme inline {
--color-background: var(--background);
--color-foreground: var(--foreground);
Expand Down