Merging new codebase with original #35

PMS61 · 2026-01-26T15:29:40Z

Added Events, Projects, Dashboards.

…f overview page and implemented sidebar hiding at smaller viewports

…more resources, added cybersec domain

- Implemented ParticipantManagement component for viewing and managing event participants. - Added filtering and searching capabilities for participants. - Included modals for viewing and editing participant details. - Created CSV export functionality for participant data. - Developed UI components for dialogs and text areas. - Defined types for events and participants to ensure type safety. - Updated next-auth types to include user admin status.

…t-related components

…ers and improving data parsing

CP Club Page

…age-lock.json

… for ESLint settings

…ntation

…esources table

…ntsCarousel and DevCore components style: Update SectionHero for improved responsiveness and spacing fix: Modify AnimatedList to cycle through items and show a fixed window size style: Adjust Lamp component for better responsiveness and visual appeal chore: Add new images for Senate members Mohammed and Priyank

Removed unused import of Link from next/link.

Updated team member roles and adjusted layout for improved presentation.

…event filtering and display

- Implemented a new Select component with various subcomponents (SelectTrigger, SelectContent, SelectItem, etc.) for better UI control. - Integrated icons for dropdown functionality using lucide-react. - Added styling utilities for consistent design. - Updated package.json and package-lock.json to include @radix-ui/react-select dependency. feat: define project types for better type safety - Created types/projects.ts to define ProjectCategory, ProjectStatus, TeamMember, Project, ProjectWithLikeStatus, ProjectSubmission, and ProjectReviewData interfaces. - Enhanced type safety for project management features.

Fixed some small bugs

… status badges

…s open

Updated dependencies to fix Next.js and React CVE vulnerabilities. The fix-react2shell-next tool automatically updated the following packages to their secure versions: - next - react-server-dom-webpack - react-server-dom-parcel - react-server-dom-turbopack All package.json files have been scanned and vulnerable versions have been patched to the correct fixed versions based on the official React advisory. Co-authored-by: Vercel <vercel[bot]@users.noreply.github.com>

…u-72uq6c Fix React Server Components CVE vulnerabilities

…nt variable syntax

…ent settings

Copilot

Pull request overview

This pull request merges significant new functionality into the original codebase, adding Events, Projects, and Dashboard features to create a comprehensive community platform for VJTI students.

Changes:

Adds complete event management system with registration, team support, and admin dashboard
Implements project showcase functionality with approval workflow
Introduces authentication enhancements with admin role support and Supabase integration
Adds new UI components and improves mobile responsiveness

Reviewed changes

Copilot reviewed 82 out of 121 changed files in this pull request and generated 34 comments.

Show a summary per file

File	Description
wrangler.toml	Updates Node.js version to 22, adds Cloudflare Workers configuration with secrets
types/*.ts	Adds TypeScript type definitions for events, projects, and NextAuth extensions
schema.sql	Defines database schema for events, participants, and users tables
middleware.ts	Enhances authentication middleware with admin checks and email validation
lib/auth.ts	Implements user synchronization with Supabase and admin role management
app/actions/*.ts	Adds server actions for events, projects, teams, and user management
components/*	Introduces numerous UI components for events, projects, dashboards, and club pages
app/**/page.tsx	Updates pages with new hero sections, team displays, and club-specific content
package.json	Adds dependencies for Supabase, Radix UI components, and markdown rendering
next.config.mjs	Configures webpack fallbacks, image patterns, and build optimizations

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-01-26T15:39:02Z

components/admin/AdminStats.tsx

+                  <div className={`p-3 rounded-lg bg-${stat.color}-500/10`}>
+                    <stat.icon className={`w-6 h-6 text-${stat.color}-400`} />
+                  </div>


Hardcoded dynamic Tailwind classes in the template string will not work correctly. The classes 'bg-emerald-500/10', 'bg-blue-500/10', etc. need to be included in the safelist in tailwind.config or use a different approach. Tailwind's JIT compiler cannot detect dynamically constructed class names, so these colors won't be generated in the final CSS.

Copilot · 2026-01-26T15:39:03Z

middleware.ts

+    if (token.email && !token.email.endsWith('.vjti.ac.in')) {
+      return NextResponse.redirect(new URL("/auth/error", request.url));


The email domain check is too restrictive and only allows '.vjti.ac.in' emails. This excludes valid VJTI email addresses that might use different subdomains (e.g., faculty emails or other department-specific domains). Consider using a more flexible pattern like 'vjti.ac.in' to allow for subdomains.

Copilot · 2026-01-26T15:39:03Z

middleware.ts

+        const supabase = createClient(
+          process.env.NEXT_PUBLIC_SUPABASE_URL!,
+          process.env.SUPABASE_SERVICE_ROLE_KEY!,
+          {
+            auth: {
+              autoRefreshToken: false,
+              persistSession: false
+            }
+          }
+        );


Creating a new Supabase client instance on every middleware request is inefficient and can lead to connection pool exhaustion under high load. Consider creating a singleton instance or caching the client to reuse across requests.

Copilot · 2026-01-26T15:39:03Z

middleware.ts

+        const { data: userData, error } = await supabase
+          .from('users')
+          .select('is_admin')
+          .eq('email', token.email)
+          .single();
+
+        const isAdmin = userData?.is_admin === 1;

-  if (token && !token.email?.endsWith('.vjti.ac.in')) {
-    return NextResponse.redirect(new URL("/auth/error", request.url));
+        // Admin-only routes protection
+        if (isAdminRoute) {
+          if (!isAdmin) {
+            // Non-admin users trying to access admin dashboard - redirect to regular dashboard
+            return NextResponse.redirect(new URL("/dashboard", request.url));
+          }
+        }


The admin status is being fetched on every request to protected routes, which adds latency and database load. Consider caching the admin status in the JWT token during sign-in, or implementing a caching layer to reduce database queries.

Copilot · 2026-01-26T15:39:04Z

types/events.ts

+  | 'other';
+
+export type EventStatus = 'upcoming' | 'ongoing' | 'completed' | 'cancelled';
+export type registrationstatus = 'open' | 'closed' | 'upcoming';


The type name 'registrationstatus' should use PascalCase to match the naming convention used for other types in this file (e.g., 'EventStatus', 'EventCategory'). Consider renaming to 'RegistrationStatus'.

Copilot · 2026-01-26T15:39:14Z

components/sections/EventsSection.tsx

@@ -1,11 +1,13 @@
 "use client";
-import React from "react";
+import React, { useState } from "react";


Unused import useState.

Copilot · 2026-01-26T15:39:15Z

components/sections/EventsSection.tsx

+import React, { useState } from "react";
 import { cn } from "@/lib/utils";
-import { motion } from "framer-motion";
+import { motion, AnimatePresence } from "framer-motion";


Unused import AnimatePresence.

Copilot · 2026-01-26T15:39:15Z

config/navigation.ts

+import {
+  Book, Brain, Code2, GraduationCap
+  , Blocks
+  , Smartphone
+  , Laptop,
+  Shield,
+  Briefcase
+} from "lucide-react";


Unused import GraduationCap.

Copilot · 2026-01-26T15:39:15Z

next.config.mjs

@@ -1,8 +1,51 @@
 /** @type {import('next').NextConfig} */
+import { createRequire } from 'module';
+const require = createRequire(import.meta.url);


Unused variable require.

Copilot · 2026-01-26T15:39:16Z

middleware.ts

+          }
+        );
+
+        const { data: userData, error } = await supabase


Unused variable error.

ahaandesai27 and others added 30 commits September 6, 2025 14:33

added resources for webdev and appdev

987e17d

Added more resources, created more domains, improved responsiveness o…

0f93c0d

…f overview page and implemented sidebar hiding at smaller viewports

refactor(Navbar): improve dropdown handling with timeout and cleanup

58257da

Modified UI of dashboard and individual domain resource pages, added …

01a870c

…more resources, added cybersec domain

added more resources for blockchain and cybersec

9c6a75f

Merge remote-tracking branch 'origin/ahaan-resources'

936b619

Done with Google OAuth Fix and Supabase connectivity

b2e9ffb

some local changes

24bfffb

Merge remote-tracking branch 'origin/ahaan-resources'

41292ef

Integrated event management and registration

38c3e00

Migrating from api to actions

e383b72

chore: add crypto-browserify and stream-browserify dependencies

910f626

refactor: standardize registration status property naming across even…

bdac014

…t-related components

feat(cp-club page) : added the cp club page

459aad3

fix(cp-club page) : minor fix

46857dd

feat: enhance team member management by including leader in team_memb…

04e25bd

…ers and improving data parsing

Merge branch 'main' into ktk

5686646

Merge pull request #1 from PMS61/ktk

f451494

CP Club Page

fix: restore embla-carousel-react dependency in package.json and pack…

a1d3a5f

…age-lock.json

refactor: streamline authentication handler and update Next.js config…

5b526fa

… for ESLint settings

feat: update team member profiles and add images for enhanced represe…

4b94caa

…ntation

Fixed dashboard resources view on chrome, added description icon on r…

f6fbb66

…esources table

Remove unused Link import in ExploreSection

455bb92

Removed unused import of Link from next/link.

Refactor DevCore component and team member roles

0040337

Updated team member roles and adjusted layout for improved presentation.

Refactor TeamsPage component and member data

02fe1e2

fixes

728ba7d

feat: Add registration deadline and event status management, enhance …

c2c6912

…event filtering and display

PMS61 and others added 10 commits January 6, 2026 21:00

fix: correct regex pattern for extracting year from email

23c62e3

fix:small bugs found during testing fixed

1f8f723

added the coc favicon

1a6bd21

Merge pull request #5 from PMS61/ktk

ce66270

Fixed some small bugs

feat: update team members and event management UI with new images and…

33615a9

… status badges

fix: improve event registration status check to ensure registration i…

b034943

…s open

Merge pull request #6 from PMS61/vercel/react-server-components-cve-v…

e12207b

…u-72uq6c Fix React Server Components CVE vulnerabilities

fix: update Dockerfile to use Node.js 22-alpine and improve environme…

5b205b2

…nt variable syntax

fix: update configuration files to improve compatibility and environm…

fe70967

…ent settings

PMS61 requested a review from Copilot January 26, 2026 15:29

Copilot started reviewing on behalf of PMS61 January 26, 2026 15:30 View session

Copilot AI reviewed Jan 26, 2026

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Merging new codebase with original #35

Merging new codebase with original #35

Uh oh!

PMS61 commented Jan 26, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Jan 26, 2026

Uh oh!

Copilot AI Jan 26, 2026

Uh oh!

Copilot AI Jan 26, 2026

Uh oh!

Copilot AI Jan 26, 2026

Uh oh!

Copilot AI Jan 26, 2026

Uh oh!

Copilot AI Jan 26, 2026

Uh oh!

Copilot AI Jan 26, 2026

Uh oh!

Copilot AI Jan 26, 2026

Uh oh!

Copilot AI Jan 26, 2026

Uh oh!

Copilot AI Jan 26, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

		if (token.email && !token.email.endsWith('.vjti.ac.in')) {
		return NextResponse.redirect(new URL("/auth/error", request.url));

Merging new codebase with original #35

Are you sure you want to change the base?

Merging new codebase with original #35

Uh oh!

Conversation

PMS61 commented Jan 26, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Jan 26, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 26, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 26, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 26, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 26, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 26, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 26, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 26, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 26, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 26, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants