feat: revamp and finilise api #2
Conversation
There was a problem hiding this comment.
Pull request overview
This PR represents a major architectural refactoring that transitions the codebase from Prisma to Drizzle ORM, removes the existing frontend implementation, and restructures the API using a modular architecture.
Key Changes:
- Complete database layer migration from Prisma (SQLite) to Drizzle ORM (PostgreSQL)
- Removal of the
typespackage and replacement with a simplifiedsharedpackage - API restructuring from route-based to module-based architecture (store/service/route pattern)
- Removal of comprehensive web frontend, replaced with minimal placeholder
- Addition of registration module with capacity management and status tracking
Reviewed changes
Copilot reviewed 141 out of 159 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| apps/api/src/db/schema.ts | New Drizzle schema definition with critical ID type mismatches |
| apps/api/src/db/db.ts | Database connection setup with incorrect type definition |
| apps/api/src/server.ts | Refactored server setup using modular architecture |
| apps/api/src/modules/*/route.ts | New modular route handlers for users, events, registrations |
| apps/api/src/modules/*/service.ts | Business logic layer with authorization |
| apps/api/src/modules/*/store.ts | Data access layer using Drizzle |
| apps/api/drizzle/* | Migration files showing integer IDs vs schema text IDs |
| apps/shared/src/index.ts | New simplified shared types package |
| apps/web/src/index.ts | Minimal web placeholder replacing full React app |
| package.json | Updated dependencies and workspace structure |
Files not reviewed (1)
- apps/api/pnpm-lock.yaml: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 138 out of 159 changed files in this pull request and generated no new comments.
Files not reviewed (1)
- apps/api/pnpm-lock.yaml: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 148 out of 166 changed files in this pull request and generated no new comments.
Files not reviewed (1)
- apps/api/pnpm-lock.yaml: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 149 out of 167 changed files in this pull request and generated no new comments.
Files not reviewed (1)
- apps/api/pnpm-lock.yaml: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
No description provided.