Feat/project detail page#37
Conversation
|
Hi @abhishek-nexgen-dev 👋 Refactored the Project module to align with the existing React feature-based architecture used across the repository. ✅ Changes made
The feature is now structured as a production-ready React module. Please let me know if any further refinements are needed 🚀 |
There was a problem hiding this comment.
Code Review
This pull request implements a comprehensive project management feature, including a detailed project page with components for overview, team, timeline, and submission tracking, alongside specialized panels for judging and moderation. It also includes a mock API service and a robust permission utility to handle various user roles. The review feedback suggests improving UI consistency by using library components instead of raw HTML, enhancing security for external links with 'noopener', preventing potential 'NaN' issues in numeric inputs, and optimizing React component lifecycle management by avoiding unnecessary remounts via the 'key' prop.
|
Hi @abhishek-nexgen-dev 👋 Got it bro, checking the routing/data mapping issue. I’ll debug and push the fix shortly 🚀 |
|
@Pranavkale11 Okay |
|
For debugging run pnpm run tauri dev |
|
Hi @abhishek-nexgen-dev 👋 I’ve fixed the “Project not found” issue and refactored the route/data resolution flow properly. Main fixes:
Verified routes:
Also verified invalid and unauthorized flows separately. The remaining TypeScript warnings are unrelated existing auth-page import issues and not caused by the Projects feature. |
Hi @abhishek-nexgen-dev 👋 I verified the issue by running the app locally using The Verified flows:
Also verified the refactored production-ready structure inside Please review once when you get time 🙌 |
|
@Pranavkale11 The ui didn't looks good make sure the ui looks best and attractive Render Component permission wise create a permission array which contain object like this {
name: "Create Event",
action: "create",
resource: Event_Permissions.CREATE_EVENT,
description: "Permission to create events",
userId: null,
},permision constant
export const Event_Permissions = {
CREATE_EVENT: "event:create",
UPDATE_EVENT: "event:update",
DELETE_EVENT: "event:delete",
VIEW_EVENT: "event:view",
PUBLISH_EVENT: "event:publish",
JOIN_EVENT: "event:join",
LEAVE_EVENT: "event:leave",
};
Make front-end like this Backend Schemaimport mongoose from "mongoose";
export type PermissionSchemaType = {
name: string;
action: "create" | "read" | "update" | "delete"; // create, read, update, delete
resource: string; // create event , read event, update event, delete event
description?: string;
userId: string | null; // Reference to the user who created the permission
};
const permissionSchema = new mongoose.Schema<PermissionSchemaType>(
{
name: {
type: String,
required: true,
unique: true,
},
action: {
type: String, // create, read, update, delete
required: true,
},
resource: {
type: String, // create event , read event, update event, delete event
required: true,
},
description: String,
userId: {
type: mongoose.Schema.Types.ObjectId,
ref: "Auth",
required: true,
},
},
{ timestamps: true },
);
export const Permission = mongoose.model("Permission", permissionSchema);
|
|
Hi @abhishek-nexgen-dev 👋 Implemented the requested UI and permission-system improvements. ✅ Changes completed
Also re-verified the project lifecycle flow after the refactor. Please review the latest updates 🙌 |
|
@Pranavkale11 bro are you human or ai |
|
Haha 😄 Still reviewing, understanding, and integrating everything manually before pushing 🙌 |
|
@Pranavkale11 okay but most of the ai loot are expensive all the ai have limited free Context window |
|
True 😅 I mainly use AI tools for debugging, testing, architecture ideas, and speeding up repetitive tasks — but I still review, understand, and integrate everything manually while learning from the process 🙌 |
|
Hi @abhishek-nexgen-dev 👋 I’ve fixed the routing issue and refactored the Project Detail route to properly work inside the protected Changes made
Verified✅ The Project Detail Page is now fully integrated with the org dashboard architecture. Please review the latest updates 🙌 |





No description provided.