Compare commits
44 Commits
6e0567d021
...
bugfix-tim
| Author | SHA1 | Date | |
|---|---|---|---|
| 1964f76f74 | |||
| 1f4e12298e | |||
|
|
1049410fee | ||
| c9bd0abf18 | |||
| 7ec76e3e8e | |||
| 784e71e187 | |||
| 7677fdd73d | |||
| 924b83eb4d | |||
| 91d13b19db | |||
| 2a5e6d4a22 | |||
| b7bd875462 | |||
| a58dfcfa4a | |||
| 7101f38bc8 | |||
| 3850e2db06 | |||
| 5b7b8e47cb | |||
| 7dd3873148 | |||
| 850f12e09d | |||
| 74999ce265 | |||
| 0c0fbf42ef | |||
| 0d116c8c26 | |||
| 25b7371d08 | |||
|
|
ddb0926dba | ||
|
|
1b0f5866a1 | ||
|
|
159022ef38 | ||
|
|
1a7d13d5b9 | ||
|
|
685a311001 | ||
| d09247d2a5 | |||
|
|
078dc8c304 | ||
|
|
59eda58ee6 | ||
| d56eed8dde | |||
|
|
3fa13e1428 | ||
|
|
2e629d8017 | ||
|
|
3ab39643dd | ||
|
|
e01e5e59df | ||
|
|
06596dcee9 | ||
|
|
7358fa6256 | ||
|
|
c99bdf56e6 | ||
|
|
15abfe0511 | ||
|
|
544b86c948 | ||
|
|
b971569983 | ||
| b613fe4edd | |||
| 30d5139ad8 | |||
| ef38578596 | |||
| ba4765b8a2 |
250
AGENTS.md
250
AGENTS.md
@@ -2,203 +2,125 @@
|
||||
|
||||
This document describes the structure, conventions, and commands for the `vibe_coding_timetracker` monorepo. Read it in full before making changes.
|
||||
|
||||
---
|
||||
|
||||
## Repository Structure
|
||||
|
||||
This is a monorepo with three sub-projects:
|
||||
|
||||
```
|
||||
```text
|
||||
/
|
||||
├── frontend/ # React SPA (Vite + TypeScript + Tailwind)
|
||||
│ └── src/
|
||||
│ ├── api/ # Axios API client modules
|
||||
│ ├── components/# Shared UI components (PascalCase .tsx)
|
||||
│ ├── contexts/ # React Context providers
|
||||
│ ├── hooks/ # TanStack React Query hooks (useXxx.ts)
|
||||
│ ├── pages/ # Route-level page components
|
||||
│ ├── types/ # TypeScript interfaces (index.ts)
|
||||
│ └── utils/ # Pure utility functions
|
||||
├── backend/ # Express REST API (TypeScript + Prisma + PostgreSQL)
|
||||
│ └── src/
|
||||
│ ├── auth/ # OIDC + JWT logic
|
||||
│ ├── errors/ # AppError subclasses
|
||||
│ ├── middleware/# Express middlewares
|
||||
│ ├── prisma/ # Prisma client singleton
|
||||
│ ├── routes/ # Express routers (xxx.routes.ts)
|
||||
│ ├── schemas/ # Zod validation schemas
|
||||
│ └── services/ # Business logic classes (xxx.service.ts)
|
||||
├── ios/ # Native iOS app (Swift/Xcode)
|
||||
├── timetracker-chart/ # Helm chart for Kubernetes deployment
|
||||
├── docker-compose.yml
|
||||
└── project.md # Product requirements document
|
||||
└── docker-compose.yml
|
||||
```
|
||||
|
||||
### Frontend layout (`frontend/src/`)
|
||||
```
|
||||
api/ # Axios API client modules (one file per resource)
|
||||
components/ # Shared UI components (PascalCase .tsx)
|
||||
contexts/ # React Context providers: AuthContext, TimerContext
|
||||
hooks/ # TanStack React Query custom hooks (useXxx.ts)
|
||||
pages/ # Route-level page components (XxxPage.tsx)
|
||||
types/ # All TypeScript interfaces (index.ts)
|
||||
utils/ # Pure utility functions (dateUtils.ts)
|
||||
```
|
||||
|
||||
### Backend layout (`backend/src/`)
|
||||
```
|
||||
auth/ # OIDC + JWT authentication logic
|
||||
config/ # Environment variable configuration
|
||||
errors/ # Custom AppError subclasses
|
||||
middleware/ # auth, errorHandler, validation middleware
|
||||
prisma/ # Prisma client singleton
|
||||
routes/ # Express routers (xxx.routes.ts)
|
||||
schemas/ # Zod validation schemas (index.ts)
|
||||
services/ # Business logic classes (xxx.service.ts)
|
||||
types/ # TypeScript interfaces + Express augmentation
|
||||
utils/ # timeUtils.ts
|
||||
```
|
||||
|
||||
---
|
||||
|
||||
## Build, Lint, and Dev Commands
|
||||
|
||||
### Frontend (`frontend/`)
|
||||
```bash
|
||||
npm run dev # Start Vite dev server (port 5173)
|
||||
npm run build # Type-check (tsc) then bundle (vite build)
|
||||
npm run preview # Preview production build locally
|
||||
npm run lint # ESLint over .ts/.tsx, zero warnings allowed
|
||||
```
|
||||
- **Dev Server:** `npm run dev` (port 5173)
|
||||
- **Build:** `npm run build` (tsc & vite build)
|
||||
- **Lint:** `npm run lint` (ESLint, zero warnings allowed)
|
||||
- **Preview:** `npm run preview`
|
||||
|
||||
### Backend (`backend/`)
|
||||
```bash
|
||||
npm run dev # Hot-reload dev server via tsx watch
|
||||
npm run build # Compile TypeScript to dist/
|
||||
npm run start # Run compiled output (node dist/index.js)
|
||||
npm run db:migrate # Run Prisma migrations
|
||||
npm run db:generate # Regenerate Prisma client
|
||||
npm run db:seed # Seed the database
|
||||
```
|
||||
- **Dev Server:** `npm run dev` (tsx watch)
|
||||
- **Build:** `npm run build` (tsc to dist/)
|
||||
- **Start:** `npm run start` (node dist/index.js)
|
||||
- **Database:**
|
||||
- `npm run db:migrate` (Run migrations)
|
||||
- `npm run db:generate` (Regenerate client)
|
||||
- `npm run db:seed` (Seed database)
|
||||
|
||||
### Full stack (repo root)
|
||||
```bash
|
||||
docker-compose up # Start all services (frontend, backend, postgres)
|
||||
```
|
||||
### Full Stack (Root)
|
||||
- **Run all:** `docker-compose up`
|
||||
|
||||
### Testing
|
||||
**There is no test framework configured.** No test runner (`jest`, `vitest`, etc.) is installed and no `.spec.ts` / `.test.ts` files exist. When adding tests, set up Vitest (already aligned with Vite) and add a `test` script to `package.json`. To run a single test file with Vitest once installed:
|
||||
```bash
|
||||
npx vitest run src/path/to/file.test.ts
|
||||
```
|
||||
|
||||
---
|
||||
|
||||
## TypeScript Configuration
|
||||
|
||||
### Frontend (`frontend/tsconfig.json`)
|
||||
- `strict: true`, `noUnusedLocals: true`, `noUnusedParameters: true`
|
||||
- `noEmit: true` — Vite handles all output
|
||||
- Path alias `@/*` → `src/*` (use `@/` for all internal imports)
|
||||
- `target: ES2020`, `module: ESNext`, `moduleResolution: bundler`
|
||||
- `isolatedModules: true`, `resolveJsonModule: true`
|
||||
|
||||
### Backend (`backend/tsconfig.json`)
|
||||
- `strict: true`, `esModuleInterop: true`
|
||||
- `target: ES2022`, `module: Node16`, `moduleResolution: Node16`
|
||||
- `outDir: ./dist`, `rootDir: ./src`
|
||||
- `declaration: true` (emits `.d.ts` files)
|
||||
|
||||
---
|
||||
**No test framework is currently configured.** No test runner (`jest`, `vitest`) is installed and no `.spec.ts` or `.test.ts` files exist.
|
||||
- When adding tests, set up **Vitest** (aligned with Vite).
|
||||
- Add a `test` script to `package.json`.
|
||||
- **To run a single test file with Vitest once installed:**
|
||||
```bash
|
||||
npx vitest run src/path/to/file.test.ts
|
||||
```
|
||||
|
||||
## Code Style Guidelines
|
||||
|
||||
### Imports
|
||||
- Use the `@/` alias for all internal frontend imports: `import { useAuth } from "@/contexts/AuthContext"`
|
||||
- Use `import type { ... }` for type-only imports: `import type { User } from "@/types"`
|
||||
- Order: external libraries first, then internal `@/` imports
|
||||
- Named exports are the standard; avoid default exports (only `App.tsx` uses one)
|
||||
### Imports & Exports
|
||||
- Use `@/` for all internal frontend imports: `import { useAuth } from "@/contexts/AuthContext"`
|
||||
- Use `import type { ... }` for type-only imports. Order external libraries first.
|
||||
- Named exports are standard. Avoid default exports (except in `App.tsx`).
|
||||
|
||||
### Formatting
|
||||
- 2-space indentation throughout
|
||||
- No Prettier config exists — maintain consistency with surrounding code
|
||||
- Trailing commas in multi-line objects and arrays
|
||||
- Quote style is mixed across the codebase (no enforcer); prefer double quotes to match the majority of files
|
||||
- 2-space indentation. No Prettier config exists; maintain consistency with surrounding code.
|
||||
- Prefer double quotes. Trailing commas in multi-line objects/arrays.
|
||||
|
||||
### Types and Interfaces
|
||||
- Define all shared types as `interface` (not `type` aliases) in the relevant `types/index.ts`
|
||||
- Suffix input/mutation types: `CreateClientInput`, `UpdateProjectInput`
|
||||
- Use `?` for optional fields, not `field: T | undefined`
|
||||
- Use `string | null` for nullable fields (not `undefined`)
|
||||
- Backend Zod schemas live in `backend/src/schemas/index.ts`, named `<Entity>Schema` (e.g., `CreateClientSchema`)
|
||||
- Backend custom errors extend `AppError`: `NotFoundError`, `BadRequestError`, `ConflictError`, `UnauthorizedError`
|
||||
|
||||
### Naming Conventions
|
||||
|
||||
| Category | Convention | Example |
|
||||
|---|---|---|
|
||||
| React components | `PascalCase.tsx` | `TimerWidget.tsx`, `Modal.tsx` |
|
||||
| Page components | `PascalCasePage.tsx` | `DashboardPage.tsx`, `LoginPage.tsx` |
|
||||
| Context files | `PascalCaseContext.tsx` | `AuthContext.tsx`, `TimerContext.tsx` |
|
||||
| Custom hooks | `useXxx.ts` | `useTimeEntries.ts`, `useClients.ts` |
|
||||
| API modules | `camelCase.ts` | `timeEntries.ts`, `clients.ts` |
|
||||
| Utility files | `camelCaseUtils.ts` | `dateUtils.ts`, `timeUtils.ts` |
|
||||
| Backend routes | `camelCase.routes.ts` | `timeEntry.routes.ts` |
|
||||
| Backend services | `camelCase.service.ts` | `timeEntry.service.ts` |
|
||||
| Types / schemas | `index.ts` (aggregated) | `src/types/index.ts` |
|
||||
| Directories | `camelCase` | `api/`, `hooks/`, `routes/`, `services/` |
|
||||
### Types & Naming Conventions
|
||||
- Define shared types as `interface` in `types/index.ts`.
|
||||
- Suffix input types: `CreateClientInput`.
|
||||
- Use `?` for optional fields, `string | null` for nullable fields (not `undefined`).
|
||||
- **Components:** `PascalCase.tsx` (`DashboardPage.tsx`)
|
||||
- **Hooks/Utils/API:** `camelCase.ts` (`useTimeEntries.ts`, `dateUtils.ts`)
|
||||
- **Backend Routes/Services:** `camelCase.routes.ts`, `camelCase.service.ts`
|
||||
- **Backend Schemas:** Zod schemas in `backend/src/schemas/index.ts` (e.g., `CreateClientSchema`).
|
||||
|
||||
### React Components
|
||||
- Use named function declarations, not arrow functions assigned to `const`:
|
||||
```ts
|
||||
// correct
|
||||
export function DashboardPage() { ... }
|
||||
|
||||
// avoid
|
||||
export const DashboardPage = () => { ... }
|
||||
```
|
||||
- Context hooks (`useAuth`, `useTimer`) throw an error if called outside their provider — maintain this pattern for all new contexts
|
||||
- Use named function declarations: `export function DashboardPage() { ... }`
|
||||
- Context hooks throw an error if called outside their provider.
|
||||
|
||||
### State Management
|
||||
- **Server state**: TanStack React Query (all remote data). Never use `useState` for server data.
|
||||
- Custom hooks encapsulate `useQuery` + `useMutation` + cache invalidation
|
||||
- Query keys are arrays: `["timeEntries", filters]`, `["projects", clientId]`
|
||||
- Use `mutateAsync` (not `mutate`) so callers can `await` and handle errors
|
||||
- Invalidate related queries after mutations via `queryClient.invalidateQueries`
|
||||
- **Shared client state**: React Context (`AuthContext`, `TimerContext`)
|
||||
- **Local UI state**: `useState` per component (modals, form data, error messages)
|
||||
- No Redux or Zustand — do not introduce them
|
||||
- **Server state:** TanStack React Query. Never use `useState` for server data.
|
||||
- Use `mutateAsync` so callers can await and handle errors.
|
||||
- Invalidate related queries after mutations: `queryClient.invalidateQueries`.
|
||||
- **Shared client state:** React Context.
|
||||
- **Local UI state:** `useState`.
|
||||
- **NO Redux or Zustand.**
|
||||
|
||||
### Error Handling
|
||||
|
||||
**Frontend:**
|
||||
```ts
|
||||
try {
|
||||
await someAsyncOperation()
|
||||
} catch (err) {
|
||||
setError(err instanceof Error ? err.message : "An error occurred")
|
||||
}
|
||||
```
|
||||
- Store errors in local `useState<string | null>` and render inline as red text
|
||||
- No global error boundary exists; handle errors close to where they occur
|
||||
|
||||
**Backend:**
|
||||
```ts
|
||||
router.get("/resource/:id", async (req, res, next) => {
|
||||
- **Frontend:**
|
||||
```typescript
|
||||
try {
|
||||
const result = await service.getById(req.params.id)
|
||||
res.json(result)
|
||||
} catch (error) {
|
||||
next(error) // always forward to errorHandler middleware
|
||||
await someAsyncOperation()
|
||||
} catch (err) {
|
||||
setError(err instanceof Error ? err.message : "An error occurred")
|
||||
}
|
||||
})
|
||||
```
|
||||
- Throw `AppError` subclasses from services; never send raw error responses from route handlers
|
||||
- The global `errorHandler` middleware handles Prisma error codes (P2002, P2025, P2003) and `AppError` subclasses
|
||||
```
|
||||
Store errors in local state and render inline as red text. No global error boundary exists.
|
||||
- **Backend:** Throw `AppError` subclasses from services.
|
||||
```typescript
|
||||
router.get("/:id", async (req, res, next) => {
|
||||
try {
|
||||
res.json(await service.getById(req.params.id))
|
||||
} catch (error) {
|
||||
next(error) // Always forward to errorHandler middleware
|
||||
}
|
||||
})
|
||||
```
|
||||
|
||||
### Styling
|
||||
- **Tailwind CSS v3** for all styling — no CSS modules, no styled-components
|
||||
- Use `clsx` + `tailwind-merge` for conditional class merging when needed
|
||||
- Icons from `lucide-react` only
|
||||
- **Tailwind CSS v3** only. No CSS modules or styled-components.
|
||||
- Use `clsx` + `tailwind-merge` for class merging. Icons from `lucide-react` only.
|
||||
|
||||
### Backend Validation
|
||||
- All incoming request data validated with Zod schemas before reaching service layer
|
||||
- Schemas defined in `backend/src/schemas/index.ts`
|
||||
- Validation middleware applied per-route; never trust `req.body` without parsing through a schema
|
||||
|
||||
### Database
|
||||
- Prisma v6 with PostgreSQL
|
||||
- Database column names are `snake_case`, mapped to `camelCase` TypeScript via `@map` in the Prisma schema
|
||||
- Always use the Prisma client singleton from `backend/src/prisma/`
|
||||
|
||||
---
|
||||
### Backend Validation & Database
|
||||
- Validate all incoming request data with Zod schemas in middleware.
|
||||
- Prisma v6 with PostgreSQL. Use the Prisma client singleton from `backend/src/prisma/`.
|
||||
- DB columns are `snake_case`, mapped to `camelCase` TypeScript via `@map`.
|
||||
|
||||
## Key Architectural Decisions
|
||||
- The frontend communicates with the backend exclusively through the typed Axios modules in `frontend/src/api/`
|
||||
- Authentication supports two flows: OIDC (web, via `express-session`) and JWT (iOS client, via `jsonwebtoken`)
|
||||
- The iOS app lives in `ios/` and shares no code with the web frontend — do not couple them
|
||||
- All business logic belongs in service classes; routes only handle HTTP concerns (parsing, validation, response formatting)
|
||||
- Frontend communicates with Backend exclusively via typed Axios modules in `frontend/src/api/`.
|
||||
- iOS app shares no code with the web frontend.
|
||||
- Backend routes only handle HTTP concerns (parsing, validation, formatting); business logic belongs purely in services.
|
||||
|
||||
76
backend/package-lock.json
generated
76
backend/package-lock.json
generated
@@ -9,6 +9,7 @@
|
||||
"version": "1.0.0",
|
||||
"dependencies": {
|
||||
"@prisma/client": "^6.19.2",
|
||||
"@quixo3/prisma-session-store": "^3.1.19",
|
||||
"cors": "^2.8.5",
|
||||
"dotenv": "^17.3.1",
|
||||
"express": "^4.18.2",
|
||||
@@ -470,6 +471,27 @@
|
||||
"node": ">=18"
|
||||
}
|
||||
},
|
||||
"node_modules/@noble/hashes": {
|
||||
"version": "1.8.0",
|
||||
"resolved": "https://registry.npmjs.org/@noble/hashes/-/hashes-1.8.0.tgz",
|
||||
"integrity": "sha512-jCs9ldd7NwzpgXDIf6P3+NrHh9/sD6CQdxHyjQI+h/6rDNo88ypBxxz45UDuZHz9r3tNz7N/VInSVoVdtXEI4A==",
|
||||
"license": "MIT",
|
||||
"engines": {
|
||||
"node": "^14.21.3 || >=16"
|
||||
},
|
||||
"funding": {
|
||||
"url": "https://paulmillr.com/funding/"
|
||||
}
|
||||
},
|
||||
"node_modules/@paralleldrive/cuid2": {
|
||||
"version": "2.3.1",
|
||||
"resolved": "https://registry.npmjs.org/@paralleldrive/cuid2/-/cuid2-2.3.1.tgz",
|
||||
"integrity": "sha512-XO7cAxhnTZl0Yggq6jOgjiOHhbgcO4NqFqwSmQpjK3b6TEE6Uj/jfSk6wzYyemh3+I0sHirKSetjQwn5cZktFw==",
|
||||
"license": "MIT",
|
||||
"dependencies": {
|
||||
"@noble/hashes": "^1.1.5"
|
||||
}
|
||||
},
|
||||
"node_modules/@prisma/client": {
|
||||
"version": "6.19.2",
|
||||
"resolved": "https://registry.npmjs.org/@prisma/client/-/client-6.19.2.tgz",
|
||||
@@ -555,6 +577,24 @@
|
||||
"@prisma/debug": "6.19.2"
|
||||
}
|
||||
},
|
||||
"node_modules/@quixo3/prisma-session-store": {
|
||||
"version": "3.1.19",
|
||||
"resolved": "https://registry.npmjs.org/@quixo3/prisma-session-store/-/prisma-session-store-3.1.19.tgz",
|
||||
"integrity": "sha512-fCG7dzmd8dyqoj4XSi5IHETqrbzN+roz4+4pPS1uMo0kVQu8CT9HRbULuIaOxWCAODT7yGyNGNvVywEeGI80lw==",
|
||||
"license": "MIT",
|
||||
"dependencies": {
|
||||
"@paralleldrive/cuid2": "^2.2.0",
|
||||
"ts-dedent": "^2.2.0",
|
||||
"type-fest": "^5.3.1"
|
||||
},
|
||||
"engines": {
|
||||
"node": ">=12.0"
|
||||
},
|
||||
"peerDependencies": {
|
||||
"@prisma/client": ">=2.16.1",
|
||||
"express-session": ">=1.17.1"
|
||||
}
|
||||
},
|
||||
"node_modules/@standard-schema/spec": {
|
||||
"version": "1.1.0",
|
||||
"resolved": "https://registry.npmjs.org/@standard-schema/spec/-/spec-1.1.0.tgz",
|
||||
@@ -2133,6 +2173,18 @@
|
||||
"node": ">= 0.8"
|
||||
}
|
||||
},
|
||||
"node_modules/tagged-tag": {
|
||||
"version": "1.0.0",
|
||||
"resolved": "https://registry.npmjs.org/tagged-tag/-/tagged-tag-1.0.0.tgz",
|
||||
"integrity": "sha512-yEFYrVhod+hdNyx7g5Bnkkb0G6si8HJurOoOEgC8B/O0uXLHlaey/65KRv6cuWBNhBgHKAROVpc7QyYqE5gFng==",
|
||||
"license": "MIT",
|
||||
"engines": {
|
||||
"node": ">=20"
|
||||
},
|
||||
"funding": {
|
||||
"url": "https://github.com/sponsors/sindresorhus"
|
||||
}
|
||||
},
|
||||
"node_modules/tinyexec": {
|
||||
"version": "1.0.2",
|
||||
"resolved": "https://registry.npmjs.org/tinyexec/-/tinyexec-1.0.2.tgz",
|
||||
@@ -2152,6 +2204,15 @@
|
||||
"node": ">=0.6"
|
||||
}
|
||||
},
|
||||
"node_modules/ts-dedent": {
|
||||
"version": "2.2.0",
|
||||
"resolved": "https://registry.npmjs.org/ts-dedent/-/ts-dedent-2.2.0.tgz",
|
||||
"integrity": "sha512-q5W7tVM71e2xjHZTlgfTDoPF/SmqKG5hddq9SzR49CH2hayqRKJtQ4mtRlSxKaJlR/+9rEM+mnBHf7I2/BQcpQ==",
|
||||
"license": "MIT",
|
||||
"engines": {
|
||||
"node": ">=6.10"
|
||||
}
|
||||
},
|
||||
"node_modules/tsx": {
|
||||
"version": "4.21.0",
|
||||
"resolved": "https://registry.npmjs.org/tsx/-/tsx-4.21.0.tgz",
|
||||
@@ -2172,6 +2233,21 @@
|
||||
"fsevents": "~2.3.3"
|
||||
}
|
||||
},
|
||||
"node_modules/type-fest": {
|
||||
"version": "5.4.4",
|
||||
"resolved": "https://registry.npmjs.org/type-fest/-/type-fest-5.4.4.tgz",
|
||||
"integrity": "sha512-JnTrzGu+zPV3aXIUhnyWJj4z/wigMsdYajGLIYakqyOW1nPllzXEJee0QQbHj+CTIQtXGlAjuK0UY+2xTyjVAw==",
|
||||
"license": "(MIT OR CC0-1.0)",
|
||||
"dependencies": {
|
||||
"tagged-tag": "^1.0.0"
|
||||
},
|
||||
"engines": {
|
||||
"node": ">=20"
|
||||
},
|
||||
"funding": {
|
||||
"url": "https://github.com/sponsors/sindresorhus"
|
||||
}
|
||||
},
|
||||
"node_modules/type-is": {
|
||||
"version": "1.6.18",
|
||||
"resolved": "https://registry.npmjs.org/type-is/-/type-is-1.6.18.tgz",
|
||||
|
||||
@@ -11,6 +11,7 @@
|
||||
},
|
||||
"dependencies": {
|
||||
"@prisma/client": "^6.19.2",
|
||||
"@quixo3/prisma-session-store": "^3.1.19",
|
||||
"cors": "^2.8.5",
|
||||
"dotenv": "^17.3.1",
|
||||
"express": "^4.18.2",
|
||||
|
||||
@@ -0,0 +1,12 @@
|
||||
-- CreateTable
|
||||
CREATE TABLE "sessions" (
|
||||
"id" TEXT NOT NULL,
|
||||
"sid" TEXT NOT NULL,
|
||||
"data" TEXT NOT NULL,
|
||||
"expires_at" TIMESTAMP(3) NOT NULL,
|
||||
|
||||
CONSTRAINT "sessions_pkey" PRIMARY KEY ("id")
|
||||
);
|
||||
|
||||
-- CreateIndex
|
||||
CREATE UNIQUE INDEX "sessions_sid_key" ON "sessions"("sid");
|
||||
@@ -0,0 +1,2 @@
|
||||
-- AlterTable
|
||||
ALTER TABLE "time_entries" ADD COLUMN "break_minutes" INTEGER NOT NULL DEFAULT 0;
|
||||
@@ -0,0 +1,8 @@
|
||||
-- AlterTable: add deleted_at column to clients
|
||||
ALTER TABLE "clients" ADD COLUMN "deleted_at" TIMESTAMP(3);
|
||||
|
||||
-- AlterTable: add deleted_at column to projects
|
||||
ALTER TABLE "projects" ADD COLUMN "deleted_at" TIMESTAMP(3);
|
||||
|
||||
-- AlterTable: add deleted_at column to time_entries
|
||||
ALTER TABLE "time_entries" ADD COLUMN "deleted_at" TIMESTAMP(3);
|
||||
@@ -0,0 +1,5 @@
|
||||
-- AlterTable: add deleted_at column to client_targets
|
||||
ALTER TABLE "client_targets" ADD COLUMN "deleted_at" TIMESTAMP(3);
|
||||
|
||||
-- AlterTable: add deleted_at column to balance_corrections
|
||||
ALTER TABLE "balance_corrections" ADD COLUMN "deleted_at" TIMESTAMP(3);
|
||||
@@ -0,0 +1,10 @@
|
||||
-- CreateEnum
|
||||
CREATE TYPE "PeriodType" AS ENUM ('WEEKLY', 'MONTHLY');
|
||||
|
||||
-- AlterTable: rename weekly_hours -> target_hours, add period_type, add working_days
|
||||
ALTER TABLE "client_targets"
|
||||
RENAME COLUMN "weekly_hours" TO "target_hours";
|
||||
|
||||
ALTER TABLE "client_targets"
|
||||
ADD COLUMN "period_type" "PeriodType" NOT NULL DEFAULT 'WEEKLY',
|
||||
ADD COLUMN "working_days" TEXT[] NOT NULL DEFAULT ARRAY['MON','TUE','WED','THU','FRI']::TEXT[];
|
||||
@@ -10,7 +10,7 @@ datasource db {
|
||||
model User {
|
||||
id String @id @db.VarChar(255)
|
||||
username String @db.VarChar(255)
|
||||
fullName String? @db.VarChar(255) @map("full_name")
|
||||
fullName String? @map("full_name") @db.VarChar(255)
|
||||
email String @db.VarChar(255)
|
||||
createdAt DateTime @default(now()) @map("created_at")
|
||||
updatedAt DateTime @updatedAt @map("updated_at")
|
||||
@@ -25,15 +25,16 @@ model User {
|
||||
}
|
||||
|
||||
model Client {
|
||||
id String @id @default(uuid())
|
||||
name String @db.VarChar(255)
|
||||
description String? @db.Text
|
||||
createdAt DateTime @default(now()) @map("created_at")
|
||||
updatedAt DateTime @updatedAt @map("updated_at")
|
||||
id String @id @default(uuid())
|
||||
name String @db.VarChar(255)
|
||||
description String? @db.Text
|
||||
createdAt DateTime @default(now()) @map("created_at")
|
||||
updatedAt DateTime @updatedAt @map("updated_at")
|
||||
deletedAt DateTime? @map("deleted_at")
|
||||
|
||||
userId String @map("user_id") @db.VarChar(255)
|
||||
user User @relation(fields: [userId], references: [id], onDelete: Cascade)
|
||||
projects Project[]
|
||||
userId String @map("user_id") @db.VarChar(255)
|
||||
user User @relation(fields: [userId], references: [id], onDelete: Cascade)
|
||||
projects Project[]
|
||||
clientTargets ClientTarget[]
|
||||
|
||||
@@index([userId])
|
||||
@@ -41,19 +42,20 @@ model Client {
|
||||
}
|
||||
|
||||
model Project {
|
||||
id String @id @default(uuid())
|
||||
name String @db.VarChar(255)
|
||||
description String? @db.Text
|
||||
color String? @db.VarChar(7) // Hex color code
|
||||
createdAt DateTime @default(now()) @map("created_at")
|
||||
updatedAt DateTime @updatedAt @map("updated_at")
|
||||
id String @id @default(uuid())
|
||||
name String @db.VarChar(255)
|
||||
description String? @db.Text
|
||||
color String? @db.VarChar(7) // Hex color code
|
||||
createdAt DateTime @default(now()) @map("created_at")
|
||||
updatedAt DateTime @updatedAt @map("updated_at")
|
||||
deletedAt DateTime? @map("deleted_at")
|
||||
|
||||
userId String @map("user_id") @db.VarChar(255)
|
||||
user User @relation(fields: [userId], references: [id], onDelete: Cascade)
|
||||
clientId String @map("client_id")
|
||||
client Client @relation(fields: [clientId], references: [id], onDelete: Cascade)
|
||||
|
||||
timeEntries TimeEntry[]
|
||||
timeEntries TimeEntry[]
|
||||
ongoingTimers OngoingTimer[]
|
||||
|
||||
@@index([userId])
|
||||
@@ -62,16 +64,18 @@ model Project {
|
||||
}
|
||||
|
||||
model TimeEntry {
|
||||
id String @id @default(uuid())
|
||||
startTime DateTime @map("start_time") @db.Timestamptz()
|
||||
endTime DateTime @map("end_time") @db.Timestamptz()
|
||||
description String? @db.Text
|
||||
createdAt DateTime @default(now()) @map("created_at")
|
||||
updatedAt DateTime @updatedAt @map("updated_at")
|
||||
id String @id @default(uuid())
|
||||
startTime DateTime @map("start_time") @db.Timestamptz()
|
||||
endTime DateTime @map("end_time") @db.Timestamptz()
|
||||
breakMinutes Int @default(0) @map("break_minutes")
|
||||
description String? @db.Text
|
||||
createdAt DateTime @default(now()) @map("created_at")
|
||||
updatedAt DateTime @updatedAt @map("updated_at")
|
||||
deletedAt DateTime? @map("deleted_at")
|
||||
|
||||
userId String @map("user_id") @db.VarChar(255)
|
||||
user User @relation(fields: [userId], references: [id], onDelete: Cascade)
|
||||
projectId String @map("project_id")
|
||||
userId String @map("user_id") @db.VarChar(255)
|
||||
user User @relation(fields: [userId], references: [id], onDelete: Cascade)
|
||||
projectId String @map("project_id")
|
||||
project Project @relation(fields: [projectId], references: [id], onDelete: Cascade)
|
||||
|
||||
@@index([userId])
|
||||
@@ -86,21 +90,29 @@ model OngoingTimer {
|
||||
createdAt DateTime @default(now()) @map("created_at")
|
||||
updatedAt DateTime @updatedAt @map("updated_at")
|
||||
|
||||
userId String @map("user_id") @db.VarChar(255) @unique
|
||||
user User @relation(fields: [userId], references: [id], onDelete: Cascade)
|
||||
projectId String? @map("project_id")
|
||||
userId String @unique @map("user_id") @db.VarChar(255)
|
||||
user User @relation(fields: [userId], references: [id], onDelete: Cascade)
|
||||
projectId String? @map("project_id")
|
||||
project Project? @relation(fields: [projectId], references: [id], onDelete: SetNull)
|
||||
|
||||
@@index([userId])
|
||||
@@map("ongoing_timers")
|
||||
}
|
||||
|
||||
enum PeriodType {
|
||||
WEEKLY
|
||||
MONTHLY
|
||||
}
|
||||
|
||||
model ClientTarget {
|
||||
id String @id @default(uuid())
|
||||
weeklyHours Float @map("weekly_hours")
|
||||
startDate DateTime @map("start_date") @db.Date // Always a Monday
|
||||
createdAt DateTime @default(now()) @map("created_at")
|
||||
updatedAt DateTime @updatedAt @map("updated_at")
|
||||
id String @id @default(uuid())
|
||||
targetHours Float @map("target_hours")
|
||||
periodType PeriodType @default(WEEKLY) @map("period_type")
|
||||
workingDays String[] @map("working_days") // e.g. ["MON","WED","FRI"]
|
||||
startDate DateTime @map("start_date") @db.Date
|
||||
createdAt DateTime @default(now()) @map("created_at")
|
||||
updatedAt DateTime @updatedAt @map("updated_at")
|
||||
deletedAt DateTime? @map("deleted_at")
|
||||
|
||||
userId String @map("user_id") @db.VarChar(255)
|
||||
user User @relation(fields: [userId], references: [id], onDelete: Cascade)
|
||||
@@ -116,16 +128,26 @@ model ClientTarget {
|
||||
}
|
||||
|
||||
model BalanceCorrection {
|
||||
id String @id @default(uuid())
|
||||
date DateTime @map("date") @db.Date
|
||||
id String @id @default(uuid())
|
||||
date DateTime @map("date") @db.Date
|
||||
hours Float
|
||||
description String? @db.VarChar(255)
|
||||
createdAt DateTime @default(now()) @map("created_at")
|
||||
updatedAt DateTime @updatedAt @map("updated_at")
|
||||
description String? @db.VarChar(255)
|
||||
createdAt DateTime @default(now()) @map("created_at")
|
||||
updatedAt DateTime @updatedAt @map("updated_at")
|
||||
deletedAt DateTime? @map("deleted_at")
|
||||
|
||||
clientTargetId String @map("client_target_id")
|
||||
clientTarget ClientTarget @relation(fields: [clientTargetId], references: [id], onDelete: Cascade)
|
||||
|
||||
@@index([clientTargetId])
|
||||
@@map("balance_corrections")
|
||||
}
|
||||
}
|
||||
|
||||
model Session {
|
||||
id String @id
|
||||
sid String @unique
|
||||
data String @db.Text
|
||||
expiresAt DateTime @map("expires_at")
|
||||
|
||||
@@map("sessions")
|
||||
}
|
||||
|
||||
@@ -1,8 +1,9 @@
|
||||
import express from "express";
|
||||
import cors from "cors";
|
||||
import session from "express-session";
|
||||
import { PrismaSessionStore } from "@quixo3/prisma-session-store";
|
||||
import { config, validateConfig } from "./config";
|
||||
import { connectDatabase } from "./prisma/client";
|
||||
import { connectDatabase, prisma } from "./prisma/client";
|
||||
import { errorHandler, notFoundHandler } from "./middleware/errorHandler";
|
||||
|
||||
// Import routes
|
||||
@@ -43,6 +44,11 @@ async function main() {
|
||||
resave: false,
|
||||
saveUninitialized: false,
|
||||
name: "sessionId",
|
||||
store: new PrismaSessionStore(prisma, {
|
||||
checkPeriod: 2 * 60 * 1000, // ms
|
||||
dbRecordIdIsSessionId: true,
|
||||
dbRecordIdFunction: undefined,
|
||||
}),
|
||||
cookie: {
|
||||
secure: config.nodeEnv === "production",
|
||||
httpOnly: true,
|
||||
|
||||
@@ -48,6 +48,16 @@ router.put(
|
||||
}
|
||||
);
|
||||
|
||||
// DELETE /api/timer - Cancel (discard) the ongoing timer without creating a time entry
|
||||
router.delete('/', requireAuth, async (req: AuthenticatedRequest, res, next) => {
|
||||
try {
|
||||
await timerService.cancel(req.user!.id);
|
||||
res.status(204).send();
|
||||
} catch (error) {
|
||||
next(error);
|
||||
}
|
||||
});
|
||||
|
||||
// POST /api/timer/stop - Stop timer
|
||||
router.post(
|
||||
'/stop',
|
||||
|
||||
@@ -31,6 +31,7 @@ export const UpdateProjectSchema = z.object({
|
||||
export const CreateTimeEntrySchema = z.object({
|
||||
startTime: z.string().datetime(),
|
||||
endTime: z.string().datetime(),
|
||||
breakMinutes: z.number().int().min(0).optional(),
|
||||
description: z.string().max(1000).optional(),
|
||||
projectId: z.string().uuid(),
|
||||
});
|
||||
@@ -38,6 +39,7 @@ export const CreateTimeEntrySchema = z.object({
|
||||
export const UpdateTimeEntrySchema = z.object({
|
||||
startTime: z.string().datetime().optional(),
|
||||
endTime: z.string().datetime().optional(),
|
||||
breakMinutes: z.number().int().min(0).optional(),
|
||||
description: z.string().max(1000).optional(),
|
||||
projectId: z.string().uuid().optional(),
|
||||
});
|
||||
@@ -64,20 +66,27 @@ export const StartTimerSchema = z.object({
|
||||
|
||||
export const UpdateTimerSchema = z.object({
|
||||
projectId: z.string().uuid().optional().nullable(),
|
||||
startTime: z.string().datetime().optional(),
|
||||
});
|
||||
|
||||
export const StopTimerSchema = z.object({
|
||||
projectId: z.string().uuid().optional(),
|
||||
});
|
||||
|
||||
const WorkingDayEnum = z.enum(['MON', 'TUE', 'WED', 'THU', 'FRI', 'SAT', 'SUN']);
|
||||
|
||||
export const CreateClientTargetSchema = z.object({
|
||||
clientId: z.string().uuid(),
|
||||
weeklyHours: z.number().positive().max(168),
|
||||
targetHours: z.number().positive().max(168),
|
||||
periodType: z.enum(['weekly', 'monthly']),
|
||||
workingDays: z.array(WorkingDayEnum).min(1, 'At least one working day is required'),
|
||||
startDate: z.string().regex(/^\d{4}-\d{2}-\d{2}$/, 'startDate must be a date in YYYY-MM-DD format'),
|
||||
});
|
||||
|
||||
export const UpdateClientTargetSchema = z.object({
|
||||
weeklyHours: z.number().positive().max(168).optional(),
|
||||
targetHours: z.number().positive().max(168).optional(),
|
||||
periodType: z.enum(['weekly', 'monthly']).optional(),
|
||||
workingDays: z.array(WorkingDayEnum).min(1, 'At least one working day is required').optional(),
|
||||
startDate: z.string().regex(/^\d{4}-\d{2}-\d{2}$/, 'startDate must be a date in YYYY-MM-DD format').optional(),
|
||||
});
|
||||
|
||||
|
||||
@@ -5,14 +5,14 @@ import type { CreateClientInput, UpdateClientInput } from "../types";
|
||||
export class ClientService {
|
||||
async findAll(userId: string) {
|
||||
return prisma.client.findMany({
|
||||
where: { userId },
|
||||
where: { userId, deletedAt: null },
|
||||
orderBy: { name: "asc" },
|
||||
});
|
||||
}
|
||||
|
||||
async findById(id: string, userId: string) {
|
||||
return prisma.client.findFirst({
|
||||
where: { id, userId },
|
||||
where: { id, userId, deletedAt: null },
|
||||
});
|
||||
}
|
||||
|
||||
@@ -43,8 +43,9 @@ export class ClientService {
|
||||
throw new NotFoundError("Client not found");
|
||||
}
|
||||
|
||||
await prisma.client.delete({
|
||||
await prisma.client.update({
|
||||
where: { id },
|
||||
data: { deletedAt: new Date() },
|
||||
});
|
||||
}
|
||||
}
|
||||
|
||||
@@ -3,44 +3,191 @@ import { NotFoundError, BadRequestError } from '../errors/AppError';
|
||||
import type { CreateClientTargetInput, UpdateClientTargetInput, CreateCorrectionInput } from '../types';
|
||||
import { Prisma } from '@prisma/client';
|
||||
|
||||
// Returns the Monday of the week containing the given date
|
||||
function getMondayOfWeek(date: Date): Date {
|
||||
const d = new Date(date);
|
||||
const day = d.getUTCDay(); // 0 = Sunday, 1 = Monday, ...
|
||||
// ---------------------------------------------------------------------------
|
||||
// Day-of-week helpers
|
||||
// ---------------------------------------------------------------------------
|
||||
|
||||
const DAY_NAMES = ['SUN', 'MON', 'TUE', 'WED', 'THU', 'FRI', 'SAT'] as const;
|
||||
|
||||
/** Returns the UTC day index (0=Sun … 6=Sat) for a YYYY-MM-DD string. */
|
||||
function dayIndex(dateStr: string): number {
|
||||
return new Date(dateStr + 'T00:00:00Z').getUTCDay();
|
||||
}
|
||||
|
||||
/** Checks whether a day-name string (e.g. "MON") is in the working-days array. */
|
||||
function isWorkingDay(dateStr: string, workingDays: string[]): boolean {
|
||||
return workingDays.includes(DAY_NAMES[dayIndex(dateStr)]);
|
||||
}
|
||||
|
||||
/** Adds `n` calendar days to a YYYY-MM-DD string and returns a new YYYY-MM-DD. */
|
||||
function addDays(dateStr: string, n: number): string {
|
||||
const d = new Date(dateStr + 'T00:00:00Z');
|
||||
d.setUTCDate(d.getUTCDate() + n);
|
||||
return d.toISOString().split('T')[0];
|
||||
}
|
||||
|
||||
/** Returns the Monday of the ISO week that contains the given date string. */
|
||||
function getMondayOfWeek(dateStr: string): string {
|
||||
const d = new Date(dateStr + 'T00:00:00Z');
|
||||
const day = d.getUTCDay(); // 0=Sun
|
||||
const diff = day === 0 ? -6 : 1 - day;
|
||||
d.setUTCDate(d.getUTCDate() + diff);
|
||||
d.setUTCHours(0, 0, 0, 0);
|
||||
return d;
|
||||
return d.toISOString().split('T')[0];
|
||||
}
|
||||
|
||||
// Returns the Sunday (end of week) for a given Monday
|
||||
function getSundayOfWeek(monday: Date): Date {
|
||||
const d = new Date(monday);
|
||||
d.setUTCDate(d.getUTCDate() + 6);
|
||||
d.setUTCHours(23, 59, 59, 999);
|
||||
return d;
|
||||
/** Returns the Sunday of the ISO week given its Monday date string. */
|
||||
function getSundayOfWeek(monday: string): string {
|
||||
return addDays(monday, 6);
|
||||
}
|
||||
|
||||
// Returns all Mondays from startDate up to and including the current week's Monday
|
||||
function getWeekMondays(startDate: Date): Date[] {
|
||||
const mondays: Date[] = [];
|
||||
const currentMonday = getMondayOfWeek(new Date());
|
||||
let cursor = new Date(startDate);
|
||||
cursor.setUTCHours(0, 0, 0, 0);
|
||||
while (cursor <= currentMonday) {
|
||||
mondays.push(new Date(cursor));
|
||||
cursor.setUTCDate(cursor.getUTCDate() + 7);
|
||||
/** Returns the first day of the month for a given date string. */
|
||||
function getMonthStart(dateStr: string): string {
|
||||
return dateStr.slice(0, 7) + '-01';
|
||||
}
|
||||
|
||||
/** Returns the last day of the month for a given date string. */
|
||||
function getMonthEnd(dateStr: string): string {
|
||||
const d = new Date(dateStr + 'T00:00:00Z');
|
||||
// Set to first day of next month then subtract 1 day
|
||||
const last = new Date(Date.UTC(d.getUTCFullYear(), d.getUTCMonth() + 1, 0));
|
||||
return last.toISOString().split('T')[0];
|
||||
}
|
||||
|
||||
/** Total calendar days in the month containing dateStr. */
|
||||
function daysInMonth(dateStr: string): number {
|
||||
const d = new Date(dateStr + 'T00:00:00Z');
|
||||
return new Date(Date.UTC(d.getUTCFullYear(), d.getUTCMonth() + 1, 0)).getUTCDate();
|
||||
}
|
||||
|
||||
/** Compare two YYYY-MM-DD strings. Returns negative, 0, or positive. */
|
||||
function cmpDate(a: string, b: string): number {
|
||||
return a < b ? -1 : a > b ? 1 : 0;
|
||||
}
|
||||
|
||||
// ---------------------------------------------------------------------------
|
||||
// Period enumeration
|
||||
// ---------------------------------------------------------------------------
|
||||
|
||||
interface Period {
|
||||
start: string; // YYYY-MM-DD
|
||||
end: string; // YYYY-MM-DD
|
||||
}
|
||||
|
||||
/**
|
||||
* Returns the period (start + end) that contains the given date.
|
||||
* For weekly: Mon–Sun.
|
||||
* For monthly: 1st–last day of month.
|
||||
*/
|
||||
function getPeriodForDate(dateStr: string, periodType: 'weekly' | 'monthly'): Period {
|
||||
if (periodType === 'weekly') {
|
||||
const monday = getMondayOfWeek(dateStr);
|
||||
return { start: monday, end: getSundayOfWeek(monday) };
|
||||
} else {
|
||||
return { start: getMonthStart(dateStr), end: getMonthEnd(dateStr) };
|
||||
}
|
||||
return mondays;
|
||||
}
|
||||
|
||||
interface WeekBalance {
|
||||
weekStart: string; // ISO date string (Monday)
|
||||
weekEnd: string; // ISO date string (Sunday)
|
||||
/**
|
||||
* Returns the start of the NEXT period after `currentPeriodEnd`.
|
||||
*/
|
||||
function nextPeriodStart(currentPeriodEnd: string, periodType: 'weekly' | 'monthly'): string {
|
||||
if (periodType === 'weekly') {
|
||||
return addDays(currentPeriodEnd, 1); // Monday of next week
|
||||
} else {
|
||||
// First day of next month
|
||||
const d = new Date(currentPeriodEnd + 'T00:00:00Z');
|
||||
const next = new Date(Date.UTC(d.getUTCFullYear(), d.getUTCMonth() + 1, 1));
|
||||
return next.toISOString().split('T')[0];
|
||||
}
|
||||
}
|
||||
|
||||
/**
|
||||
* Enumerates all periods from startDate's period through today's period (inclusive).
|
||||
*/
|
||||
function enumeratePeriods(startDate: string, periodType: 'weekly' | 'monthly'): Period[] {
|
||||
const today = new Date().toISOString().split('T')[0];
|
||||
const periods: Period[] = [];
|
||||
|
||||
const firstPeriod = getPeriodForDate(startDate, periodType);
|
||||
let cursor = firstPeriod;
|
||||
|
||||
while (cmpDate(cursor.start, today) <= 0) {
|
||||
periods.push(cursor);
|
||||
const ns = nextPeriodStart(cursor.end, periodType);
|
||||
cursor = getPeriodForDate(ns, periodType);
|
||||
}
|
||||
|
||||
return periods;
|
||||
}
|
||||
|
||||
// ---------------------------------------------------------------------------
|
||||
// Working-day counting
|
||||
// ---------------------------------------------------------------------------
|
||||
|
||||
/**
|
||||
* Counts working days in [from, to] (both inclusive) matching the given pattern.
|
||||
*/
|
||||
function countWorkingDays(from: string, to: string, workingDays: string[]): number {
|
||||
if (cmpDate(from, to) > 0) return 0;
|
||||
let count = 0;
|
||||
let cur = from;
|
||||
while (cmpDate(cur, to) <= 0) {
|
||||
if (isWorkingDay(cur, workingDays)) count++;
|
||||
cur = addDays(cur, 1);
|
||||
}
|
||||
return count;
|
||||
}
|
||||
|
||||
// ---------------------------------------------------------------------------
|
||||
// Pro-ration helpers
|
||||
// ---------------------------------------------------------------------------
|
||||
|
||||
/**
|
||||
* Returns the pro-rated target hours for the first period, applying §5 of the spec.
|
||||
* If startDate falls on the natural first day of the period, no pro-ration occurs.
|
||||
*/
|
||||
function computePeriodTargetHours(
|
||||
period: Period,
|
||||
startDate: string,
|
||||
targetHours: number,
|
||||
periodType: 'weekly' | 'monthly',
|
||||
): number {
|
||||
const naturalStart = period.start;
|
||||
if (cmpDate(startDate, naturalStart) <= 0) {
|
||||
// startDate is at or before the natural period start — no pro-ration needed
|
||||
return targetHours;
|
||||
}
|
||||
|
||||
// startDate is inside the period → pro-rate by calendar days
|
||||
const fullDays = periodType === 'weekly' ? 7 : daysInMonth(period.start);
|
||||
const remainingDays = daysBetween(startDate, period.end); // inclusive both ends
|
||||
return (remainingDays / fullDays) * targetHours;
|
||||
}
|
||||
|
||||
/** Calendar days between two dates (both inclusive). */
|
||||
function daysBetween(from: string, to: string): number {
|
||||
const a = new Date(from + 'T00:00:00Z').getTime();
|
||||
const b = new Date(to + 'T00:00:00Z').getTime();
|
||||
return Math.round((b - a) / 86400000) + 1;
|
||||
}
|
||||
|
||||
// ---------------------------------------------------------------------------
|
||||
// Response types
|
||||
// ---------------------------------------------------------------------------
|
||||
|
||||
export interface PeriodBalance {
|
||||
periodStart: string;
|
||||
periodEnd: string;
|
||||
targetHours: number;
|
||||
trackedSeconds: number;
|
||||
targetSeconds: number;
|
||||
correctionHours: number;
|
||||
balanceSeconds: number; // positive = overtime, negative = undertime
|
||||
balanceSeconds: number;
|
||||
isOngoing: boolean;
|
||||
// only when isOngoing = true
|
||||
dailyRateHours?: number;
|
||||
workingDaysInPeriod?: number;
|
||||
elapsedWorkingDays?: number;
|
||||
expectedHours?: number;
|
||||
}
|
||||
|
||||
export interface ClientTargetWithBalance {
|
||||
@@ -48,7 +195,9 @@ export interface ClientTargetWithBalance {
|
||||
clientId: string;
|
||||
clientName: string;
|
||||
userId: string;
|
||||
weeklyHours: number;
|
||||
periodType: 'weekly' | 'monthly';
|
||||
targetHours: number;
|
||||
workingDays: string[];
|
||||
startDate: string;
|
||||
createdAt: string;
|
||||
updatedAt: string;
|
||||
@@ -59,53 +208,91 @@ export interface ClientTargetWithBalance {
|
||||
description: string | null;
|
||||
createdAt: string;
|
||||
}>;
|
||||
totalBalanceSeconds: number; // running total across all weeks
|
||||
currentWeekTrackedSeconds: number;
|
||||
currentWeekTargetSeconds: number;
|
||||
weeks: WeekBalance[];
|
||||
totalBalanceSeconds: number;
|
||||
currentPeriodTrackedSeconds: number;
|
||||
currentPeriodTargetSeconds: number;
|
||||
periods: PeriodBalance[];
|
||||
/** True when an active timer is running for a project belonging to this client. */
|
||||
hasOngoingTimer: boolean;
|
||||
}
|
||||
|
||||
// ---------------------------------------------------------------------------
|
||||
// Prisma record shape accepted by computeBalance
|
||||
// ---------------------------------------------------------------------------
|
||||
|
||||
type TargetRecord = {
|
||||
id: string;
|
||||
clientId: string;
|
||||
userId: string;
|
||||
targetHours: number;
|
||||
periodType: 'WEEKLY' | 'MONTHLY';
|
||||
workingDays: string[];
|
||||
startDate: Date;
|
||||
createdAt: Date;
|
||||
updatedAt: Date;
|
||||
client: { id: string; name: string };
|
||||
corrections: Array<{ id: string; date: Date; hours: number; description: string | null; createdAt: Date }>;
|
||||
};
|
||||
|
||||
// ---------------------------------------------------------------------------
|
||||
// Service
|
||||
// ---------------------------------------------------------------------------
|
||||
|
||||
export class ClientTargetService {
|
||||
async findAll(userId: string): Promise<ClientTargetWithBalance[]> {
|
||||
const targets = await prisma.clientTarget.findMany({
|
||||
where: { userId },
|
||||
where: { userId, deletedAt: null, client: { deletedAt: null } },
|
||||
include: {
|
||||
client: { select: { id: true, name: true } },
|
||||
corrections: { orderBy: { date: 'asc' } },
|
||||
corrections: { where: { deletedAt: null }, orderBy: { date: 'asc' } },
|
||||
},
|
||||
orderBy: { client: { name: 'asc' } },
|
||||
});
|
||||
|
||||
return Promise.all(targets.map(t => this.computeBalance(t)));
|
||||
return Promise.all(targets.map(t => this.computeBalance(t as unknown as TargetRecord)));
|
||||
}
|
||||
|
||||
async findById(id: string, userId: string) {
|
||||
return prisma.clientTarget.findFirst({
|
||||
where: { id, userId },
|
||||
where: { id, userId, deletedAt: null, client: { deletedAt: null } },
|
||||
include: {
|
||||
client: { select: { id: true, name: true } },
|
||||
corrections: { orderBy: { date: 'asc' } },
|
||||
corrections: { where: { deletedAt: null }, orderBy: { date: 'asc' } },
|
||||
},
|
||||
});
|
||||
}
|
||||
|
||||
async create(userId: string, data: CreateClientTargetInput): Promise<ClientTargetWithBalance> {
|
||||
// Validate startDate is a Monday
|
||||
const startDate = new Date(data.startDate + 'T00:00:00Z');
|
||||
const dayOfWeek = startDate.getUTCDay();
|
||||
if (dayOfWeek !== 1) {
|
||||
throw new BadRequestError('startDate must be a Monday');
|
||||
}
|
||||
|
||||
// Ensure the client belongs to this user
|
||||
const client = await prisma.client.findFirst({ where: { id: data.clientId, userId } });
|
||||
// Ensure the client belongs to this user and is not soft-deleted
|
||||
const client = await prisma.client.findFirst({ where: { id: data.clientId, userId, deletedAt: null } });
|
||||
if (!client) {
|
||||
throw new NotFoundError('Client not found');
|
||||
}
|
||||
|
||||
const startDate = new Date(data.startDate + 'T00:00:00Z');
|
||||
const periodType = data.periodType.toUpperCase() as 'WEEKLY' | 'MONTHLY';
|
||||
|
||||
// Check for existing target (unique per user+client)
|
||||
const existing = await prisma.clientTarget.findFirst({ where: { userId, clientId: data.clientId } });
|
||||
if (existing) {
|
||||
if (existing.deletedAt !== null) {
|
||||
// Reactivate the soft-deleted target with the new settings
|
||||
const reactivated = await prisma.clientTarget.update({
|
||||
where: { id: existing.id },
|
||||
data: {
|
||||
deletedAt: null,
|
||||
targetHours: data.targetHours,
|
||||
periodType,
|
||||
workingDays: data.workingDays,
|
||||
startDate,
|
||||
},
|
||||
include: {
|
||||
client: { select: { id: true, name: true } },
|
||||
corrections: { where: { deletedAt: null }, orderBy: { date: 'asc' } },
|
||||
},
|
||||
});
|
||||
return this.computeBalance(reactivated as unknown as TargetRecord);
|
||||
}
|
||||
throw new BadRequestError('A target already exists for this client. Delete the existing one first or update it.');
|
||||
}
|
||||
|
||||
@@ -113,52 +300,55 @@ export class ClientTargetService {
|
||||
data: {
|
||||
userId,
|
||||
clientId: data.clientId,
|
||||
weeklyHours: data.weeklyHours,
|
||||
targetHours: data.targetHours,
|
||||
periodType,
|
||||
workingDays: data.workingDays,
|
||||
startDate,
|
||||
},
|
||||
include: {
|
||||
client: { select: { id: true, name: true } },
|
||||
corrections: { orderBy: { date: 'asc' } },
|
||||
corrections: { where: { deletedAt: null }, orderBy: { date: 'asc' } },
|
||||
},
|
||||
});
|
||||
|
||||
return this.computeBalance(target);
|
||||
return this.computeBalance(target as unknown as TargetRecord);
|
||||
}
|
||||
|
||||
async update(id: string, userId: string, data: UpdateClientTargetInput): Promise<ClientTargetWithBalance> {
|
||||
const existing = await this.findById(id, userId);
|
||||
if (!existing) throw new NotFoundError('Client target not found');
|
||||
|
||||
const updateData: { weeklyHours?: number; startDate?: Date } = {};
|
||||
const updateData: {
|
||||
targetHours?: number;
|
||||
periodType?: 'WEEKLY' | 'MONTHLY';
|
||||
workingDays?: string[];
|
||||
startDate?: Date;
|
||||
} = {};
|
||||
|
||||
if (data.weeklyHours !== undefined) {
|
||||
updateData.weeklyHours = data.weeklyHours;
|
||||
}
|
||||
|
||||
if (data.startDate !== undefined) {
|
||||
const startDate = new Date(data.startDate + 'T00:00:00Z');
|
||||
if (startDate.getUTCDay() !== 1) {
|
||||
throw new BadRequestError('startDate must be a Monday');
|
||||
}
|
||||
updateData.startDate = startDate;
|
||||
}
|
||||
if (data.targetHours !== undefined) updateData.targetHours = data.targetHours;
|
||||
if (data.periodType !== undefined) updateData.periodType = data.periodType.toUpperCase() as 'WEEKLY' | 'MONTHLY';
|
||||
if (data.workingDays !== undefined) updateData.workingDays = data.workingDays;
|
||||
if (data.startDate !== undefined) updateData.startDate = new Date(data.startDate + 'T00:00:00Z');
|
||||
|
||||
const updated = await prisma.clientTarget.update({
|
||||
where: { id },
|
||||
data: updateData,
|
||||
include: {
|
||||
client: { select: { id: true, name: true } },
|
||||
corrections: { orderBy: { date: 'asc' } },
|
||||
corrections: { where: { deletedAt: null }, orderBy: { date: 'asc' } },
|
||||
},
|
||||
});
|
||||
|
||||
return this.computeBalance(updated);
|
||||
return this.computeBalance(updated as unknown as TargetRecord);
|
||||
}
|
||||
|
||||
async delete(id: string, userId: string): Promise<void> {
|
||||
const existing = await this.findById(id, userId);
|
||||
if (!existing) throw new NotFoundError('Client target not found');
|
||||
await prisma.clientTarget.delete({ where: { id } });
|
||||
await prisma.clientTarget.update({
|
||||
where: { id },
|
||||
data: { deletedAt: new Date() },
|
||||
});
|
||||
}
|
||||
|
||||
async addCorrection(targetId: string, userId: string, data: CreateCorrectionInput) {
|
||||
@@ -188,99 +378,221 @@ export class ClientTargetService {
|
||||
if (!target) throw new NotFoundError('Client target not found');
|
||||
|
||||
const correction = await prisma.balanceCorrection.findFirst({
|
||||
where: { id: correctionId, clientTargetId: targetId },
|
||||
where: { id: correctionId, clientTargetId: targetId, deletedAt: null },
|
||||
});
|
||||
if (!correction) throw new NotFoundError('Correction not found');
|
||||
|
||||
await prisma.balanceCorrection.delete({ where: { id: correctionId } });
|
||||
await prisma.balanceCorrection.update({
|
||||
where: { id: correctionId },
|
||||
data: { deletedAt: new Date() },
|
||||
});
|
||||
}
|
||||
|
||||
private async computeBalance(target: {
|
||||
id: string;
|
||||
clientId: string;
|
||||
userId: string;
|
||||
weeklyHours: number;
|
||||
startDate: Date;
|
||||
createdAt: Date;
|
||||
updatedAt: Date;
|
||||
client: { id: string; name: string };
|
||||
corrections: Array<{ id: string; date: Date; hours: number; description: string | null; createdAt: Date }>;
|
||||
}): Promise<ClientTargetWithBalance> {
|
||||
const mondays = getWeekMondays(target.startDate);
|
||||
// ---------------------------------------------------------------------------
|
||||
// Balance computation
|
||||
// ---------------------------------------------------------------------------
|
||||
|
||||
if (mondays.length === 0) {
|
||||
return this.emptyBalance(target);
|
||||
private async computeBalance(target: TargetRecord): Promise<ClientTargetWithBalance> {
|
||||
const startDateStr = target.startDate.toISOString().split('T')[0];
|
||||
const periodType = target.periodType.toLowerCase() as 'weekly' | 'monthly';
|
||||
const workingDays = target.workingDays;
|
||||
|
||||
const periods = enumeratePeriods(startDateStr, periodType);
|
||||
|
||||
if (periods.length === 0) {
|
||||
return this.emptyBalance(target, periodType);
|
||||
}
|
||||
|
||||
// Fetch all tracked time for this user on this client's projects in one query
|
||||
// covering startDate to end of current week
|
||||
const periodStart = mondays[0];
|
||||
const periodEnd = getSundayOfWeek(mondays[mondays.length - 1]);
|
||||
const overallStart = periods[0].start;
|
||||
const overallEnd = periods[periods.length - 1].end;
|
||||
const today = new Date().toISOString().split('T')[0];
|
||||
|
||||
type TrackedRow = { week_start: Date; tracked_seconds: bigint };
|
||||
// Fetch active timer for this user (if any) and check if it belongs to this client
|
||||
const ongoingTimer = await prisma.ongoingTimer.findUnique({
|
||||
where: { userId: target.userId },
|
||||
include: { project: { select: { clientId: true } } },
|
||||
});
|
||||
|
||||
const rows = await prisma.$queryRaw<TrackedRow[]>(Prisma.sql`
|
||||
SELECT
|
||||
DATE_TRUNC('week', te.start_time AT TIME ZONE 'UTC') AS week_start,
|
||||
COALESCE(SUM(EXTRACT(EPOCH FROM (te.end_time - te.start_time))), 0)::bigint AS tracked_seconds
|
||||
FROM time_entries te
|
||||
JOIN projects p ON p.id = te.project_id
|
||||
WHERE te.user_id = ${target.userId}
|
||||
AND p.client_id = ${target.clientId}
|
||||
AND te.start_time >= ${periodStart}
|
||||
AND te.start_time <= ${periodEnd}
|
||||
GROUP BY DATE_TRUNC('week', te.start_time AT TIME ZONE 'UTC')
|
||||
`);
|
||||
// Elapsed seconds from the active timer attributed to this client target.
|
||||
// We only count it if the timer has a project assigned and that project
|
||||
// belongs to the same client as this target.
|
||||
let ongoingTimerSeconds = 0;
|
||||
let ongoingTimerPeriodStart: string | null = null;
|
||||
|
||||
// Index tracked seconds by week start (ISO Monday string)
|
||||
const trackedByWeek = new Map<string, number>();
|
||||
for (const row of rows) {
|
||||
// DATE_TRUNC with 'week' gives Monday in Postgres (ISO week)
|
||||
const monday = getMondayOfWeek(new Date(row.week_start));
|
||||
const key = monday.toISOString().split('T')[0];
|
||||
trackedByWeek.set(key, Number(row.tracked_seconds));
|
||||
if (
|
||||
ongoingTimer &&
|
||||
ongoingTimer.projectId !== null &&
|
||||
ongoingTimer.project?.clientId === target.clientId
|
||||
) {
|
||||
ongoingTimerSeconds = Math.floor(
|
||||
(Date.now() - ongoingTimer.startTime.getTime()) / 1000,
|
||||
);
|
||||
// Determine which period the timer's start time falls into
|
||||
const timerDateStr = ongoingTimer.startTime.toISOString().split('T')[0];
|
||||
const timerPeriod = getPeriodForDate(timerDateStr, periodType);
|
||||
ongoingTimerPeriodStart = timerPeriod.start;
|
||||
}
|
||||
|
||||
// Index corrections by week
|
||||
const correctionsByWeek = new Map<string, number>();
|
||||
// Fetch all time tracked for this client across the full range in one query
|
||||
type TrackedRow = { period_start: string; tracked_seconds: bigint };
|
||||
|
||||
let trackedRows: TrackedRow[];
|
||||
if (periodType === 'weekly') {
|
||||
trackedRows = await prisma.$queryRaw<TrackedRow[]>(Prisma.sql`
|
||||
SELECT
|
||||
TO_CHAR(
|
||||
DATE_TRUNC('week', te.start_time AT TIME ZONE 'UTC'),
|
||||
'YYYY-MM-DD'
|
||||
) AS period_start,
|
||||
COALESCE(
|
||||
SUM(EXTRACT(EPOCH FROM (te.end_time - te.start_time)) - (te.break_minutes * 60)),
|
||||
0
|
||||
)::bigint AS tracked_seconds
|
||||
FROM time_entries te
|
||||
JOIN projects p ON p.id = te.project_id
|
||||
WHERE te.user_id = ${target.userId}
|
||||
AND p.client_id = ${target.clientId}
|
||||
AND te.start_time >= ${new Date(overallStart + 'T00:00:00Z')}
|
||||
AND te.start_time <= ${new Date(overallEnd + 'T23:59:59Z')}
|
||||
AND te.deleted_at IS NULL
|
||||
AND p.deleted_at IS NULL
|
||||
GROUP BY DATE_TRUNC('week', te.start_time AT TIME ZONE 'UTC')
|
||||
`);
|
||||
} else {
|
||||
trackedRows = await prisma.$queryRaw<TrackedRow[]>(Prisma.sql`
|
||||
SELECT
|
||||
TO_CHAR(
|
||||
DATE_TRUNC('month', te.start_time AT TIME ZONE 'UTC'),
|
||||
'YYYY-MM-DD'
|
||||
) AS period_start,
|
||||
COALESCE(
|
||||
SUM(EXTRACT(EPOCH FROM (te.end_time - te.start_time)) - (te.break_minutes * 60)),
|
||||
0
|
||||
)::bigint AS tracked_seconds
|
||||
FROM time_entries te
|
||||
JOIN projects p ON p.id = te.project_id
|
||||
WHERE te.user_id = ${target.userId}
|
||||
AND p.client_id = ${target.clientId}
|
||||
AND te.start_time >= ${new Date(overallStart + 'T00:00:00Z')}
|
||||
AND te.start_time <= ${new Date(overallEnd + 'T23:59:59Z')}
|
||||
AND te.deleted_at IS NULL
|
||||
AND p.deleted_at IS NULL
|
||||
GROUP BY DATE_TRUNC('month', te.start_time AT TIME ZONE 'UTC')
|
||||
`);
|
||||
}
|
||||
|
||||
// Map tracked seconds by period start date string
|
||||
const trackedByPeriod = new Map<string, number>();
|
||||
for (const row of trackedRows) {
|
||||
// Normalise: for weekly, Postgres DATE_TRUNC('week') already gives Monday
|
||||
const key = typeof row.period_start === 'string'
|
||||
? row.period_start
|
||||
: (row.period_start as Date).toISOString().split('T')[0];
|
||||
trackedByPeriod.set(key, Number(row.tracked_seconds));
|
||||
}
|
||||
|
||||
// Index corrections by period start date
|
||||
const correctionsByPeriod = new Map<string, number>();
|
||||
for (const c of target.corrections) {
|
||||
const monday = getMondayOfWeek(new Date(c.date));
|
||||
const key = monday.toISOString().split('T')[0];
|
||||
correctionsByWeek.set(key, (correctionsByWeek.get(key) ?? 0) + c.hours);
|
||||
const corrDateStr = c.date.toISOString().split('T')[0];
|
||||
const period = getPeriodForDate(corrDateStr, periodType);
|
||||
const key = period.start;
|
||||
correctionsByPeriod.set(key, (correctionsByPeriod.get(key) ?? 0) + c.hours);
|
||||
}
|
||||
|
||||
const targetSecondsPerWeek = target.weeklyHours * 3600;
|
||||
const weeks: WeekBalance[] = [];
|
||||
const periodBalances: PeriodBalance[] = [];
|
||||
let totalBalanceSeconds = 0;
|
||||
const isFirstPeriod = (i: number) => i === 0;
|
||||
|
||||
for (let i = 0; i < periods.length; i++) {
|
||||
const period = periods[i];
|
||||
|
||||
// Effective start for this period (clamped to startDate for first period)
|
||||
const effectiveStart = isFirstPeriod(i) && cmpDate(startDateStr, period.start) > 0
|
||||
? startDateStr
|
||||
: period.start;
|
||||
|
||||
// Period target hours (with possible pro-ration on the first period)
|
||||
const periodTargetHours = isFirstPeriod(i)
|
||||
? computePeriodTargetHours(period, startDateStr, target.targetHours, periodType)
|
||||
: target.targetHours;
|
||||
|
||||
// Add ongoing timer seconds to the period it started in (if it belongs to this client)
|
||||
const timerContribution =
|
||||
ongoingTimerPeriodStart !== null && period.start === ongoingTimerPeriodStart
|
||||
? ongoingTimerSeconds
|
||||
: 0;
|
||||
|
||||
const trackedSeconds = (trackedByPeriod.get(period.start) ?? 0) + timerContribution;
|
||||
const correctionHours = correctionsByPeriod.get(period.start) ?? 0;
|
||||
|
||||
const isOngoing = cmpDate(period.start, today) <= 0 && cmpDate(today, period.end) <= 0;
|
||||
|
||||
let balanceSeconds: number;
|
||||
let extra: Partial<PeriodBalance> = {};
|
||||
|
||||
if (isOngoing) {
|
||||
// §6: ongoing period — expected hours based on elapsed working days
|
||||
const workingDaysInPeriod = countWorkingDays(effectiveStart, period.end, workingDays);
|
||||
const dailyRateHours = workingDaysInPeriod > 0 ? periodTargetHours / workingDaysInPeriod : 0;
|
||||
|
||||
const elapsedEnd = today < period.end ? today : period.end;
|
||||
const elapsedWorkingDays = countWorkingDays(effectiveStart, elapsedEnd, workingDays);
|
||||
const expectedHours = elapsedWorkingDays * dailyRateHours;
|
||||
|
||||
// Only count corrections up to and including today — future corrections
|
||||
// within the ongoing period must not be counted until those days have elapsed,
|
||||
// otherwise a +8h correction for tomorrow inflates the balance immediately.
|
||||
const correctionHoursToDate = target.corrections.reduce((sum, c) => {
|
||||
const d = c.date.toISOString().split('T')[0];
|
||||
if (cmpDate(d, effectiveStart) >= 0 && cmpDate(d, today) <= 0) {
|
||||
return sum + c.hours;
|
||||
}
|
||||
return sum;
|
||||
}, 0);
|
||||
|
||||
balanceSeconds = Math.round(
|
||||
(trackedSeconds + correctionHoursToDate * 3600) - expectedHours * 3600,
|
||||
);
|
||||
|
||||
extra = {
|
||||
dailyRateHours,
|
||||
workingDaysInPeriod,
|
||||
elapsedWorkingDays,
|
||||
expectedHours,
|
||||
};
|
||||
} else {
|
||||
// §4: completed period — simple formula
|
||||
balanceSeconds = Math.round(
|
||||
(trackedSeconds + correctionHours * 3600) - periodTargetHours * 3600,
|
||||
);
|
||||
}
|
||||
|
||||
for (const monday of mondays) {
|
||||
const key = monday.toISOString().split('T')[0];
|
||||
const sunday = getSundayOfWeek(monday);
|
||||
const trackedSeconds = trackedByWeek.get(key) ?? 0;
|
||||
const correctionHours = correctionsByWeek.get(key) ?? 0;
|
||||
const effectiveTargetSeconds = targetSecondsPerWeek - correctionHours * 3600;
|
||||
const balanceSeconds = trackedSeconds - effectiveTargetSeconds;
|
||||
totalBalanceSeconds += balanceSeconds;
|
||||
|
||||
weeks.push({
|
||||
weekStart: key,
|
||||
weekEnd: sunday.toISOString().split('T')[0],
|
||||
periodBalances.push({
|
||||
periodStart: period.start,
|
||||
periodEnd: period.end,
|
||||
targetHours: periodTargetHours,
|
||||
trackedSeconds,
|
||||
targetSeconds: effectiveTargetSeconds,
|
||||
correctionHours,
|
||||
balanceSeconds,
|
||||
isOngoing,
|
||||
...extra,
|
||||
});
|
||||
}
|
||||
|
||||
const currentWeek = weeks[weeks.length - 1];
|
||||
const currentPeriod = periodBalances.find(p => p.isOngoing) ?? periodBalances[periodBalances.length - 1];
|
||||
|
||||
return {
|
||||
id: target.id,
|
||||
clientId: target.clientId,
|
||||
clientName: target.client.name,
|
||||
userId: target.userId,
|
||||
weeklyHours: target.weeklyHours,
|
||||
startDate: target.startDate.toISOString().split('T')[0],
|
||||
periodType,
|
||||
targetHours: target.targetHours,
|
||||
workingDays,
|
||||
startDate: startDateStr,
|
||||
createdAt: target.createdAt.toISOString(),
|
||||
updatedAt: target.updatedAt.toISOString(),
|
||||
corrections: target.corrections.map(c => ({
|
||||
@@ -291,37 +603,33 @@ export class ClientTargetService {
|
||||
createdAt: c.createdAt.toISOString(),
|
||||
})),
|
||||
totalBalanceSeconds,
|
||||
currentWeekTrackedSeconds: currentWeek?.trackedSeconds ?? 0,
|
||||
currentWeekTargetSeconds: currentWeek?.targetSeconds ?? targetSecondsPerWeek,
|
||||
weeks,
|
||||
currentPeriodTrackedSeconds: currentPeriod?.trackedSeconds ?? 0,
|
||||
currentPeriodTargetSeconds: currentPeriod
|
||||
? Math.round(currentPeriod.targetHours * 3600)
|
||||
: Math.round(target.targetHours * 3600),
|
||||
periods: periodBalances,
|
||||
hasOngoingTimer: ongoingTimerSeconds > 0,
|
||||
};
|
||||
}
|
||||
|
||||
private emptyBalance(target: {
|
||||
id: string;
|
||||
clientId: string;
|
||||
userId: string;
|
||||
weeklyHours: number;
|
||||
startDate: Date;
|
||||
createdAt: Date;
|
||||
updatedAt: Date;
|
||||
client: { id: string; name: string };
|
||||
corrections: Array<{ id: string; date: Date; hours: number; description: string | null; createdAt: Date }>;
|
||||
}): ClientTargetWithBalance {
|
||||
private emptyBalance(target: TargetRecord, periodType: 'weekly' | 'monthly'): ClientTargetWithBalance {
|
||||
return {
|
||||
id: target.id,
|
||||
clientId: target.clientId,
|
||||
clientName: target.client.name,
|
||||
userId: target.userId,
|
||||
weeklyHours: target.weeklyHours,
|
||||
periodType,
|
||||
targetHours: target.targetHours,
|
||||
workingDays: target.workingDays,
|
||||
startDate: target.startDate.toISOString().split('T')[0],
|
||||
createdAt: target.createdAt.toISOString(),
|
||||
updatedAt: target.updatedAt.toISOString(),
|
||||
corrections: [],
|
||||
totalBalanceSeconds: 0,
|
||||
currentWeekTrackedSeconds: 0,
|
||||
currentWeekTargetSeconds: target.weeklyHours * 3600,
|
||||
weeks: [],
|
||||
currentPeriodTrackedSeconds: 0,
|
||||
currentPeriodTargetSeconds: Math.round(target.targetHours * 3600),
|
||||
periods: [],
|
||||
hasOngoingTimer: false,
|
||||
};
|
||||
}
|
||||
}
|
||||
|
||||
@@ -7,6 +7,8 @@ export class ProjectService {
|
||||
return prisma.project.findMany({
|
||||
where: {
|
||||
userId,
|
||||
deletedAt: null,
|
||||
client: { deletedAt: null },
|
||||
...(clientId && { clientId }),
|
||||
},
|
||||
orderBy: { name: "asc" },
|
||||
@@ -23,7 +25,12 @@ export class ProjectService {
|
||||
|
||||
async findById(id: string, userId: string) {
|
||||
return prisma.project.findFirst({
|
||||
where: { id, userId },
|
||||
where: {
|
||||
id,
|
||||
userId,
|
||||
deletedAt: null,
|
||||
client: { deletedAt: null },
|
||||
},
|
||||
include: {
|
||||
client: {
|
||||
select: {
|
||||
@@ -36,9 +43,9 @@ export class ProjectService {
|
||||
}
|
||||
|
||||
async create(userId: string, data: CreateProjectInput) {
|
||||
// Verify the client belongs to the user
|
||||
// Verify the client belongs to the user and is not soft-deleted
|
||||
const client = await prisma.client.findFirst({
|
||||
where: { id: data.clientId, userId },
|
||||
where: { id: data.clientId, userId, deletedAt: null },
|
||||
});
|
||||
|
||||
if (!client) {
|
||||
@@ -70,10 +77,10 @@ export class ProjectService {
|
||||
throw new NotFoundError("Project not found");
|
||||
}
|
||||
|
||||
// If clientId is being updated, verify it belongs to the user
|
||||
// If clientId is being updated, verify it belongs to the user and is not soft-deleted
|
||||
if (data.clientId) {
|
||||
const client = await prisma.client.findFirst({
|
||||
where: { id: data.clientId, userId },
|
||||
where: { id: data.clientId, userId, deletedAt: null },
|
||||
});
|
||||
|
||||
if (!client) {
|
||||
@@ -108,8 +115,9 @@ export class ProjectService {
|
||||
throw new NotFoundError("Project not found");
|
||||
}
|
||||
|
||||
await prisma.project.delete({
|
||||
await prisma.project.update({
|
||||
where: { id },
|
||||
data: { deletedAt: new Date() },
|
||||
});
|
||||
}
|
||||
}
|
||||
|
||||
@@ -42,11 +42,15 @@ export class TimeEntryService {
|
||||
p.id AS project_id,
|
||||
p.name AS project_name,
|
||||
p.color AS project_color,
|
||||
COALESCE(SUM(EXTRACT(EPOCH FROM (te.end_time - te.start_time))), 0)::bigint AS total_seconds,
|
||||
COALESCE(SUM(EXTRACT(EPOCH FROM (te.end_time - te.start_time)) - (te.break_minutes * 60)), 0)::bigint AS total_seconds,
|
||||
COUNT(te.id)::bigint AS entry_count
|
||||
FROM time_entries te
|
||||
JOIN projects p ON p.id = te.project_id
|
||||
JOIN clients c ON c.id = p.client_id
|
||||
WHERE te.user_id = ${userId}
|
||||
AND te.deleted_at IS NULL
|
||||
AND p.deleted_at IS NULL
|
||||
AND c.deleted_at IS NULL
|
||||
${filterClause}
|
||||
GROUP BY p.id, p.name, p.color
|
||||
ORDER BY total_seconds DESC
|
||||
@@ -63,12 +67,15 @@ export class TimeEntryService {
|
||||
SELECT
|
||||
c.id AS client_id,
|
||||
c.name AS client_name,
|
||||
COALESCE(SUM(EXTRACT(EPOCH FROM (te.end_time - te.start_time))), 0)::bigint AS total_seconds,
|
||||
COALESCE(SUM(EXTRACT(EPOCH FROM (te.end_time - te.start_time)) - (te.break_minutes * 60)), 0)::bigint AS total_seconds,
|
||||
COUNT(te.id)::bigint AS entry_count
|
||||
FROM time_entries te
|
||||
JOIN projects p ON p.id = te.project_id
|
||||
JOIN clients c ON c.id = p.client_id
|
||||
WHERE te.user_id = ${userId}
|
||||
AND te.deleted_at IS NULL
|
||||
AND p.deleted_at IS NULL
|
||||
AND c.deleted_at IS NULL
|
||||
${filterClause}
|
||||
GROUP BY c.id, c.name
|
||||
ORDER BY total_seconds DESC
|
||||
@@ -77,11 +84,15 @@ export class TimeEntryService {
|
||||
prisma.$queryRaw<{ total_seconds: bigint; entry_count: bigint }[]>(
|
||||
Prisma.sql`
|
||||
SELECT
|
||||
COALESCE(SUM(EXTRACT(EPOCH FROM (te.end_time - te.start_time))), 0)::bigint AS total_seconds,
|
||||
COALESCE(SUM(EXTRACT(EPOCH FROM (te.end_time - te.start_time)) - (te.break_minutes * 60)), 0)::bigint AS total_seconds,
|
||||
COUNT(te.id)::bigint AS entry_count
|
||||
FROM time_entries te
|
||||
JOIN projects p ON p.id = te.project_id
|
||||
JOIN clients c ON c.id = p.client_id
|
||||
WHERE te.user_id = ${userId}
|
||||
AND te.deleted_at IS NULL
|
||||
AND p.deleted_at IS NULL
|
||||
AND c.deleted_at IS NULL
|
||||
${filterClause}
|
||||
`,
|
||||
),
|
||||
@@ -125,10 +136,11 @@ export class TimeEntryService {
|
||||
|
||||
const where: {
|
||||
userId: string;
|
||||
deletedAt: null;
|
||||
startTime?: { gte?: Date; lte?: Date };
|
||||
projectId?: string;
|
||||
project?: { clientId?: string };
|
||||
} = { userId };
|
||||
project?: { deletedAt: null; clientId?: string; client: { deletedAt: null } };
|
||||
} = { userId, deletedAt: null };
|
||||
|
||||
if (startDate || endDate) {
|
||||
where.startTime = {};
|
||||
@@ -140,9 +152,13 @@ export class TimeEntryService {
|
||||
where.projectId = projectId;
|
||||
}
|
||||
|
||||
if (clientId) {
|
||||
where.project = { clientId };
|
||||
}
|
||||
// Always filter out entries whose project or client is soft-deleted,
|
||||
// merging the optional clientId filter into the project relation filter.
|
||||
where.project = {
|
||||
deletedAt: null,
|
||||
client: { deletedAt: null },
|
||||
...(clientId && { clientId }),
|
||||
};
|
||||
|
||||
const [entries, total] = await Promise.all([
|
||||
prisma.timeEntry.findMany({
|
||||
@@ -182,7 +198,12 @@ export class TimeEntryService {
|
||||
|
||||
async findById(id: string, userId: string) {
|
||||
return prisma.timeEntry.findFirst({
|
||||
where: { id, userId },
|
||||
where: {
|
||||
id,
|
||||
userId,
|
||||
deletedAt: null,
|
||||
project: { deletedAt: null, client: { deletedAt: null } },
|
||||
},
|
||||
include: {
|
||||
project: {
|
||||
select: {
|
||||
@@ -204,15 +225,22 @@ export class TimeEntryService {
|
||||
async create(userId: string, data: CreateTimeEntryInput) {
|
||||
const startTime = new Date(data.startTime);
|
||||
const endTime = new Date(data.endTime);
|
||||
const breakMinutes = data.breakMinutes ?? 0;
|
||||
|
||||
// Validate end time is after start time
|
||||
if (endTime <= startTime) {
|
||||
throw new BadRequestError("End time must be after start time");
|
||||
}
|
||||
|
||||
// Verify the project belongs to the user
|
||||
// Validate break time doesn't exceed duration
|
||||
const durationMinutes = (endTime.getTime() - startTime.getTime()) / 60000;
|
||||
if (breakMinutes > durationMinutes) {
|
||||
throw new BadRequestError("Break time cannot exceed total duration");
|
||||
}
|
||||
|
||||
// Verify the project belongs to the user and is not soft-deleted (nor its client)
|
||||
const project = await prisma.project.findFirst({
|
||||
where: { id: data.projectId, userId },
|
||||
where: { id: data.projectId, userId, deletedAt: null, client: { deletedAt: null } },
|
||||
});
|
||||
|
||||
if (!project) {
|
||||
@@ -235,6 +263,7 @@ export class TimeEntryService {
|
||||
data: {
|
||||
startTime,
|
||||
endTime,
|
||||
breakMinutes,
|
||||
description: data.description,
|
||||
userId,
|
||||
projectId: data.projectId,
|
||||
@@ -267,16 +296,23 @@ export class TimeEntryService {
|
||||
? new Date(data.startTime)
|
||||
: entry.startTime;
|
||||
const endTime = data.endTime ? new Date(data.endTime) : entry.endTime;
|
||||
const breakMinutes = data.breakMinutes ?? entry.breakMinutes;
|
||||
|
||||
// Validate end time is after start time
|
||||
if (endTime <= startTime) {
|
||||
throw new BadRequestError("End time must be after start time");
|
||||
}
|
||||
|
||||
// If project changed, verify it belongs to the user
|
||||
// Validate break time doesn't exceed duration
|
||||
const durationMinutes = (endTime.getTime() - startTime.getTime()) / 60000;
|
||||
if (breakMinutes > durationMinutes) {
|
||||
throw new BadRequestError("Break time cannot exceed total duration");
|
||||
}
|
||||
|
||||
// If project changed, verify it belongs to the user and is not soft-deleted
|
||||
if (data.projectId && data.projectId !== entry.projectId) {
|
||||
const project = await prisma.project.findFirst({
|
||||
where: { id: data.projectId, userId },
|
||||
where: { id: data.projectId, userId, deletedAt: null, client: { deletedAt: null } },
|
||||
});
|
||||
|
||||
if (!project) {
|
||||
@@ -302,6 +338,7 @@ export class TimeEntryService {
|
||||
data: {
|
||||
startTime,
|
||||
endTime,
|
||||
breakMinutes,
|
||||
description: data.description,
|
||||
projectId: data.projectId,
|
||||
},
|
||||
@@ -329,8 +366,9 @@ export class TimeEntryService {
|
||||
throw new NotFoundError("Time entry not found");
|
||||
}
|
||||
|
||||
await prisma.timeEntry.delete({
|
||||
await prisma.timeEntry.update({
|
||||
where: { id },
|
||||
data: { deletedAt: new Date() },
|
||||
});
|
||||
}
|
||||
}
|
||||
|
||||
@@ -102,9 +102,24 @@ export class TimerService {
|
||||
projectId = data.projectId;
|
||||
}
|
||||
|
||||
// Validate startTime if provided
|
||||
let startTime: Date | undefined = undefined;
|
||||
if (data.startTime) {
|
||||
const parsed = new Date(data.startTime);
|
||||
const now = new Date();
|
||||
if (parsed >= now) {
|
||||
throw new BadRequestError("Start time must be in the past");
|
||||
}
|
||||
startTime = parsed;
|
||||
}
|
||||
|
||||
const updateData: Record<string, unknown> = {};
|
||||
if (projectId !== undefined) updateData.projectId = projectId;
|
||||
if (startTime !== undefined) updateData.startTime = startTime;
|
||||
|
||||
return prisma.ongoingTimer.update({
|
||||
where: { userId },
|
||||
data: projectId !== undefined ? { projectId } : {},
|
||||
data: updateData,
|
||||
include: {
|
||||
project: {
|
||||
select: {
|
||||
@@ -123,6 +138,15 @@ export class TimerService {
|
||||
});
|
||||
}
|
||||
|
||||
async cancel(userId: string) {
|
||||
const timer = await this.getOngoingTimer(userId);
|
||||
if (!timer) {
|
||||
throw new NotFoundError("No timer is running");
|
||||
}
|
||||
|
||||
await prisma.ongoingTimer.delete({ where: { userId } });
|
||||
}
|
||||
|
||||
async stop(userId: string, data?: StopTimerInput) {
|
||||
const timer = await this.getOngoingTimer(userId);
|
||||
if (!timer) {
|
||||
|
||||
@@ -38,6 +38,7 @@ export interface UpdateProjectInput {
|
||||
export interface CreateTimeEntryInput {
|
||||
startTime: string;
|
||||
endTime: string;
|
||||
breakMinutes?: number;
|
||||
description?: string;
|
||||
projectId: string;
|
||||
}
|
||||
@@ -45,6 +46,7 @@ export interface CreateTimeEntryInput {
|
||||
export interface UpdateTimeEntryInput {
|
||||
startTime?: string;
|
||||
endTime?: string;
|
||||
breakMinutes?: number;
|
||||
description?: string;
|
||||
projectId?: string;
|
||||
}
|
||||
@@ -71,6 +73,7 @@ export interface StartTimerInput {
|
||||
|
||||
export interface UpdateTimerInput {
|
||||
projectId?: string | null;
|
||||
startTime?: string;
|
||||
}
|
||||
|
||||
export interface StopTimerInput {
|
||||
@@ -79,13 +82,17 @@ export interface StopTimerInput {
|
||||
|
||||
export interface CreateClientTargetInput {
|
||||
clientId: string;
|
||||
weeklyHours: number;
|
||||
startDate: string; // YYYY-MM-DD, always a Monday
|
||||
targetHours: number;
|
||||
periodType: 'weekly' | 'monthly';
|
||||
workingDays: string[]; // e.g. ["MON","WED","FRI"]
|
||||
startDate: string; // YYYY-MM-DD
|
||||
}
|
||||
|
||||
export interface UpdateClientTargetInput {
|
||||
weeklyHours?: number;
|
||||
startDate?: string; // YYYY-MM-DD, always a Monday
|
||||
targetHours?: number;
|
||||
periodType?: 'weekly' | 'monthly';
|
||||
workingDays?: string[];
|
||||
startDate?: string; // YYYY-MM-DD
|
||||
}
|
||||
|
||||
export interface CreateCorrectionInput {
|
||||
|
||||
@@ -19,6 +19,7 @@ export async function hasOverlappingEntries(
|
||||
const count = await prisma.timeEntry.count({
|
||||
where: {
|
||||
userId,
|
||||
deletedAt: null,
|
||||
...(excludeId ? { id: { not: excludeId } } : {}),
|
||||
// An entry overlaps when it starts before our end AND ends after our start.
|
||||
startTime: { lt: endTime },
|
||||
|
||||
285
docs/client-targets-v2-requirements.md
Normal file
285
docs/client-targets-v2-requirements.md
Normal file
@@ -0,0 +1,285 @@
|
||||
# Client Targets v2 — Feature Requirements
|
||||
|
||||
## Overview
|
||||
|
||||
This document defines the requirements for the second iteration of the Client Targets feature. The main additions are:
|
||||
|
||||
- Targets can be set on a **weekly or monthly** period.
|
||||
- Each target defines a **fixed weekly working-day pattern** (e.g. Mon + Wed).
|
||||
- The balance for the **current period** is calculated proportionally based on elapsed working days, so the user can see at any point in time whether they are ahead or behind.
|
||||
- The **start date** can be any calendar day (no longer restricted to Mondays).
|
||||
- Manual **balance corrections** are preserved and continue to work as before.
|
||||
|
||||
---
|
||||
|
||||
## 1. Target Configuration
|
||||
|
||||
| Field | Type | Constraints |
|
||||
|---|---|---|
|
||||
| `periodType` | `WEEKLY \| MONTHLY` | Required |
|
||||
| `weeklyOrMonthlyHours` | positive float, ≤ 168 | Required; represents hours per week or per month |
|
||||
| `workingDays` | array of day names | At least one of `MON TUE WED THU FRI SAT SUN`; fixed repeating pattern |
|
||||
| `startDate` | `YYYY-MM-DD` | Any calendar day; no longer restricted to Mondays |
|
||||
| `clientId` | UUID | Must belong to the authenticated user |
|
||||
|
||||
**One active target per client** — the unique `(userId, clientId)` constraint is preserved. To change period type, hours, or working days the user creates a new target with a new `startDate`; the old target is soft-deleted. History from the old target is retained as-is and is no longer recalculated.
|
||||
|
||||
---
|
||||
|
||||
## 2. Period Definitions
|
||||
|
||||
| `periodType` | Period start | Period end |
|
||||
|---|---|---|
|
||||
| `WEEKLY` | Monday 00:00 of the calendar week | Sunday 23:59 of that same calendar week |
|
||||
| `MONTHLY` | 1st of the calendar month 00:00 | Last day of the calendar month 23:59 |
|
||||
|
||||
---
|
||||
|
||||
## 3. Balance Calculation — Overview
|
||||
|
||||
The total balance is the **sum of individual period balances** from the period containing `startDate` up to and including the **current period** (the period that contains today).
|
||||
|
||||
Each period is classified as either **completed** or **ongoing**.
|
||||
|
||||
```
|
||||
total_balance_seconds = SUM( balance_seconds ) over all periods
|
||||
```
|
||||
|
||||
Positive = overtime. Negative = undertime.
|
||||
|
||||
---
|
||||
|
||||
## 4. Completed Period Balance
|
||||
|
||||
A period is **completed** when its end date is strictly before today.
|
||||
|
||||
```
|
||||
balance = tracked_hours + correction_hours - period_target_hours
|
||||
```
|
||||
|
||||
- `period_target_hours` — see §5 (pro-ration) for the first period; full `weeklyOrMonthlyHours` for all subsequent periods.
|
||||
- `tracked_hours` — sum of all time entries for this client whose date falls within `[period_start, period_end]`.
|
||||
- `correction_hours` — sum of manual corrections whose `date` falls within `[period_start, period_end]`.
|
||||
|
||||
No working-day logic is applied to completed periods. The target is simply the (optionally pro-rated) hours for that period.
|
||||
|
||||
---
|
||||
|
||||
## 5. First Period Pro-ration
|
||||
|
||||
If `startDate` does not fall on the natural first day of a period (Monday for weekly, 1st for monthly), the target hours for that first period are pro-rated by calendar days.
|
||||
|
||||
### Monthly
|
||||
|
||||
```
|
||||
full_period_days = total calendar days in that month
|
||||
remaining_days = (last day of month) − startDate + 1 // inclusive
|
||||
period_target_hours = (remaining_days / full_period_days) × weeklyOrMonthlyHours
|
||||
```
|
||||
|
||||
**Example:** startDate = Jan 25, target = 40 h/month, January has 31 days.
|
||||
`remaining_days = 7`, `period_target_hours = (7 / 31) × 40 = 9.032 h`
|
||||
|
||||
### Weekly
|
||||
|
||||
```
|
||||
full_period_days = 7
|
||||
remaining_days = Sunday of that calendar week − startDate + 1 // inclusive
|
||||
period_target_hours = (remaining_days / 7) × weeklyOrMonthlyHours
|
||||
```
|
||||
|
||||
**Example:** startDate = Wednesday, target = 40 h/week.
|
||||
`remaining_days = 5 (Wed–Sun)`, `period_target_hours = (5 / 7) × 40 = 28.571 h`
|
||||
|
||||
All periods after the first use the full `weeklyOrMonthlyHours`.
|
||||
|
||||
---
|
||||
|
||||
## 6. Ongoing Period Balance (Current Period)
|
||||
|
||||
The current period is **ongoing** when today falls within it. The balance reflects how the user is doing *so far* — future working days within the current period are not considered.
|
||||
|
||||
### Step 1 — Period target hours
|
||||
|
||||
Apply §5 if this is the first period; otherwise use full `weeklyOrMonthlyHours`.
|
||||
|
||||
### Step 2 — Daily rate
|
||||
|
||||
```
|
||||
working_days_in_period = COUNT of days in [period_start, period_end]
|
||||
that match the working day pattern
|
||||
daily_rate_hours = period_target_hours / working_days_in_period
|
||||
```
|
||||
|
||||
The rate is fixed at the start of the period and does not change as time passes.
|
||||
|
||||
### Step 3 — Elapsed working days
|
||||
|
||||
```
|
||||
elapsed_working_days = COUNT of days in [period_start, TODAY] (both inclusive)
|
||||
that match the working day pattern
|
||||
```
|
||||
|
||||
- If today matches the working day pattern, it is counted as a **full** elapsed working day.
|
||||
- If today does not match the working day pattern, it is not counted.
|
||||
|
||||
### Step 4 — Expected hours so far
|
||||
|
||||
```
|
||||
expected_hours = elapsed_working_days × daily_rate_hours
|
||||
```
|
||||
|
||||
### Step 5 — Balance
|
||||
|
||||
```
|
||||
tracked_hours = SUM of time entries for this client in [period_start, today]
|
||||
correction_hours = SUM of manual corrections whose date ∈ [period_start, today]
|
||||
balance = tracked_hours + correction_hours − expected_hours
|
||||
```
|
||||
|
||||
### Worked example
|
||||
|
||||
> Target: 40 h/month. Working days: Mon + Wed.
|
||||
> Current month has 4 Mondays and 4 Wednesdays → `working_days_in_period = 8`.
|
||||
> `daily_rate_hours = 40 / 8 = 5 h`.
|
||||
> 3 working days have elapsed → `expected_hours = 15 h`.
|
||||
> Tracked so far: 13 h, no corrections.
|
||||
> `balance = 13 − 15 = −2 h` (2 hours behind).
|
||||
|
||||
---
|
||||
|
||||
## 7. Manual Balance Corrections
|
||||
|
||||
| Field | Type | Constraints |
|
||||
|---|---|---|
|
||||
| `date` | `YYYY-MM-DD` | Must be ≥ `startDate`; not more than one period in the future |
|
||||
| `hours` | signed float | Positive = extra credit (reduces deficit). Negative = reduces tracked credit |
|
||||
| `description` | string | Optional, max 255 chars |
|
||||
|
||||
- The system automatically assigns a correction to the period that contains its `date`.
|
||||
- Corrections in **completed periods** are included in the completed period formula (§4).
|
||||
- Corrections in the **ongoing period** are included in the ongoing balance formula (§6).
|
||||
- Corrections in a **future period** (not yet started) are stored and will be applied when that period becomes active.
|
||||
- A correction whose `date` is before `startDate` is rejected with a validation error.
|
||||
|
||||
---
|
||||
|
||||
## 8. Edge Cases
|
||||
|
||||
| Scenario | Behaviour |
|
||||
|---|---|
|
||||
| `startDate` = 1st of month / Monday | No pro-ration; `period_target_hours = weeklyOrMonthlyHours` |
|
||||
| `startDate` = last day of period | `remaining_days = 1`; target is heavily reduced (e.g. 1/31 × hours) |
|
||||
| Working pattern has no matches in the partial first period | `elapsed_working_days = 0`; `expected_hours = 0`; balance = `tracked + corrections` |
|
||||
| Current period has zero elapsed working days | `expected_hours = 0`; balance = `tracked + corrections` (cannot divide by zero — guard required) |
|
||||
| `working_days_in_period = 0` | Impossible by validation (at least one day required), but system must guard: treat as `daily_rate_hours = 0` |
|
||||
| Today is not a working day | `elapsed_working_days` does not include today |
|
||||
| Correction date before `startDate` | Rejected with a validation error |
|
||||
| Correction date in future period | Accepted and stored; applied when that period is ongoing or completed |
|
||||
| User changes working days or period type | Must create a new target with a new `startDate`; old target history is frozen |
|
||||
| Two periods with the same client exist (old soft-deleted, new active) | Only the active target's periods contribute to the displayed balance |
|
||||
| A month with only partial working day coverage (e.g. all Mondays are public holidays) | No automatic holiday handling; user adds manual corrections to compensate |
|
||||
|
||||
---
|
||||
|
||||
## 9. Data Model Changes
|
||||
|
||||
### `ClientTarget` table — additions / changes
|
||||
|
||||
| Column | Change | Notes |
|
||||
|---|---|---|
|
||||
| `period_type` | **Add** | Enum: `WEEKLY`, `MONTHLY` |
|
||||
| `working_days` | **Add** | Array/bitmask of day names: `MON TUE WED THU FRI SAT SUN` |
|
||||
| `start_date` | **Modify** | Remove "must be Monday" validation constraint |
|
||||
| `weekly_hours` | **Rename** | → `target_hours` (represents hours per week or per month depending on `period_type`) |
|
||||
|
||||
### `BalanceCorrection` table — no structural changes
|
||||
|
||||
Date-to-period assignment is computed at query time, not stored.
|
||||
|
||||
---
|
||||
|
||||
## 10. API Changes
|
||||
|
||||
### `ClientTargetWithBalance` response shape
|
||||
|
||||
```typescript
|
||||
interface ClientTargetWithBalance {
|
||||
id: string
|
||||
clientId: string
|
||||
clientName: string
|
||||
userId: string
|
||||
periodType: "weekly" | "monthly"
|
||||
targetHours: number // renamed from weeklyHours
|
||||
workingDays: string[] // e.g. ["MON", "WED"]
|
||||
startDate: string // YYYY-MM-DD
|
||||
createdAt: string
|
||||
updatedAt: string
|
||||
corrections: BalanceCorrection[]
|
||||
totalBalanceSeconds: number // running total across all periods
|
||||
currentPeriodTrackedSeconds: number // replaces currentWeekTrackedSeconds
|
||||
currentPeriodTargetSeconds: number // replaces currentWeekTargetSeconds
|
||||
periods: PeriodBalance[] // replaces weeks[]
|
||||
}
|
||||
|
||||
interface PeriodBalance {
|
||||
periodStart: string // YYYY-MM-DD (Monday or 1st of month)
|
||||
periodEnd: string // YYYY-MM-DD (Sunday or last of month)
|
||||
targetHours: number // pro-rated for first period
|
||||
trackedSeconds: number
|
||||
correctionHours: number
|
||||
balanceSeconds: number
|
||||
isOngoing: boolean
|
||||
// only present when isOngoing = true
|
||||
dailyRateHours?: number
|
||||
workingDaysInPeriod?: number
|
||||
elapsedWorkingDays?: number
|
||||
expectedHours?: number
|
||||
}
|
||||
```
|
||||
|
||||
### Endpoint changes
|
||||
|
||||
| Method | Path | Change |
|
||||
|---|---|---|
|
||||
| `POST /client-targets` | Create | Accepts `periodType`, `workingDays`, `targetHours`; `startDate` unconstrained |
|
||||
| `PUT /client-targets/:id` | Update | Accepts same new fields |
|
||||
| `GET /client-targets` | List | Returns updated `ClientTargetWithBalance` shape |
|
||||
| `POST /client-targets/:id/corrections` | Add correction | No change to signature |
|
||||
| `DELETE /client-targets/:id/corrections/:corrId` | Delete correction | No change |
|
||||
|
||||
### Zod schema changes
|
||||
|
||||
- `CreateClientTargetSchema` / `UpdateClientTargetSchema`:
|
||||
- Add `periodType: z.enum(["weekly", "monthly"])`
|
||||
- Add `workingDays: z.array(z.enum(["MON","TUE","WED","THU","FRI","SAT","SUN"])).min(1)`
|
||||
- Rename `weeklyHours` → `targetHours`
|
||||
- Remove Monday-only regex constraint from `startDate`
|
||||
|
||||
---
|
||||
|
||||
## 11. Frontend Changes
|
||||
|
||||
### Types (`frontend/src/types/index.ts`)
|
||||
- `ClientTargetWithBalance` — add `periodType`, `workingDays`, `targetHours`; replace `weeks` → `periods: PeriodBalance[]`; replace `currentWeek*` → `currentPeriod*`
|
||||
- Add `PeriodBalance` interface
|
||||
- `CreateClientTargetInput` / `UpdateClientTargetInput` — same field additions
|
||||
|
||||
### Hook (`frontend/src/hooks/useClientTargets.ts`)
|
||||
- No structural changes; mutations pass through new fields
|
||||
|
||||
### API client (`frontend/src/api/clientTargets.ts`)
|
||||
- No structural changes; payload shapes updated
|
||||
|
||||
### `ClientsPage` — `ClientTargetPanel`
|
||||
- Working day selector (checkboxes: Mon–Sun, at least one required)
|
||||
- Period type selector (Weekly / Monthly)
|
||||
- Label for hours input updates dynamically: "Hours/week" or "Hours/month"
|
||||
- Start date picker: free date input (no week-picker)
|
||||
- Balance display: label changes from "this week" to "this week" or "this month" based on `periodType`
|
||||
- Expanded period list replaces the expanded week list
|
||||
|
||||
### `DashboardPage`
|
||||
- "Weekly Targets" widget renamed to "Targets"
|
||||
- "This week" label becomes "This week" / "This month" dynamically
|
||||
- `currentWeek*` fields replaced with `currentPeriod*`
|
||||
@@ -1,40 +1,23 @@
|
||||
<svg xmlns="http://www.w3.org/2000/svg" viewBox="48 48 416 416" width="100%" height="100%">
|
||||
<defs>
|
||||
<linearGradient id="bg" x1="0%" y1="0%" x2="100%" y2="100%">
|
||||
<stop offset="0%" stop-color="#818CF8" />
|
||||
<stop offset="100%" stop-color="#4F46E5" />
|
||||
<?xml version="1.0" encoding="UTF-8"?>
|
||||
<!-- Generated by Pixelmator Pro 4.0 -->
|
||||
<svg width="416" height="416" viewBox="0 0 416 416" xmlns="http://www.w3.org/2000/svg">
|
||||
<linearGradient id="linearGradient1" x1="0" y1="0" x2="416" y2="416" gradientUnits="userSpaceOnUse">
|
||||
<stop offset="1e-05" stop-color="#818cf8" stop-opacity="1"/>
|
||||
<stop offset="1" stop-color="#4f46e5" stop-opacity="1"/>
|
||||
</linearGradient>
|
||||
</defs>
|
||||
|
||||
<!-- App Icon Background -->
|
||||
<rect x="48" y="48" width="416" height="416" rx="96" fill="url(#bg)" />
|
||||
|
||||
<!-- Inner Icon Group -->
|
||||
<g stroke="#ffffff" fill="none" stroke-linecap="round" stroke-linejoin="round">
|
||||
<!-- Stopwatch Top Button -->
|
||||
<path d="M256 96 v44" stroke-width="28" />
|
||||
<path d="M224 88 h64" stroke-width="24" />
|
||||
|
||||
<!-- Stopwatch Side Button -->
|
||||
<path d="M352 176 l 24 -24" stroke-width="24" />
|
||||
<!-- Cap for side button -->
|
||||
<path d="M362 138 l 28 28" stroke-width="24" />
|
||||
|
||||
<!-- Outer Ring -->
|
||||
<circle cx="256" cy="272" r="132" stroke-width="28" />
|
||||
|
||||
<!-- Clock Hands -->
|
||||
<!-- Minute Hand -->
|
||||
<path d="M256 184 v 88" stroke-width="24" />
|
||||
<!-- Hour Hand -->
|
||||
<path d="M256 272 l 48 32" stroke-width="24" />
|
||||
|
||||
<!-- Dial Tick Marks -->
|
||||
<g stroke-width="12" opacity="0.6">
|
||||
<line x1="256" y1="172" x2="256" y2="188" />
|
||||
<line x1="256" y1="356" x2="256" y2="372" />
|
||||
<line x1="172" y1="272" x2="188" y2="272" />
|
||||
<line x1="340" y1="272" x2="324" y2="272" />
|
||||
<path id="Path" fill="url(#linearGradient1)" stroke="none" d="M 96 0 L 320 0 C 373.019348 0 416 42.980652 416 96 L 416 320 C 416 373.019348 373.019348 416 320 416 L 96 416 C 42.980667 416 0 373.019348 0 320 L 0 96 C 0 42.980652 42.980667 0 96 0 Z"/>
|
||||
<g id="Group">
|
||||
<path id="path1" fill="none" stroke="#ffffff" stroke-width="28" stroke-linecap="round" stroke-linejoin="round" d="M 208 48 L 208 92"/>
|
||||
<path id="path2" fill="none" stroke="#ffffff" stroke-width="24" stroke-linecap="round" stroke-linejoin="round" d="M 176 40 L 240 40"/>
|
||||
<path id="path3" fill="none" stroke="#ffffff" stroke-width="24" stroke-linecap="round" stroke-linejoin="round" d="M 304 128 L 328 104"/>
|
||||
<path id="path4" fill="none" stroke="#ffffff" stroke-width="24" stroke-linecap="round" stroke-linejoin="round" d="M 314 90 L 342 118"/>
|
||||
<path id="path5" fill="none" stroke="#ffffff" stroke-width="28" stroke-linecap="round" stroke-linejoin="round" d="M 340 224 C 340 296.901581 280.901581 356 208 356 C 135.098419 356 76 296.901581 76 224 C 76 151.098419 135.098419 92 208 92 C 280.901581 92 340 151.098419 340 224 Z"/>
|
||||
<path id="path6" fill="none" stroke="#ffffff" stroke-width="24" stroke-linecap="round" stroke-linejoin="round" d="M 208 136 L 208 224"/>
|
||||
<path id="path7" fill="none" stroke="#ffffff" stroke-width="24" stroke-linecap="round" stroke-linejoin="round" d="M 208 224 L 256 256"/>
|
||||
<g id="g1" opacity="0.6">
|
||||
<path id="path8" fill="none" stroke="#ffffff" stroke-width="12" stroke-linecap="round" stroke-linejoin="round" d="M 208 308 L 208 324"/>
|
||||
<path id="path9" fill="none" stroke="#ffffff" stroke-width="12" stroke-linecap="round" stroke-linejoin="round" d="M 124 224 L 140 224"/>
|
||||
<path id="path10" fill="none" stroke="#ffffff" stroke-width="12" stroke-linecap="round" stroke-linejoin="round" d="M 292 224 L 276 224"/>
|
||||
</g>
|
||||
</g>
|
||||
</g>
|
||||
</svg>
|
||||
|
||||
|
Before Width: | Height: | Size: 1.3 KiB After Width: | Height: | Size: 2.3 KiB |
@@ -1,43 +1,33 @@
|
||||
<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 512 512" width="100%" height="100%">
|
||||
<defs>
|
||||
<linearGradient id="bg" x1="0%" y1="0%" x2="100%" y2="100%">
|
||||
<stop offset="0%" stop-color="#818CF8" />
|
||||
<stop offset="100%" stop-color="#4F46E5" />
|
||||
<?xml version="1.0" encoding="UTF-8"?>
|
||||
<!-- Generated by Pixelmator Pro 4.0 -->
|
||||
<svg width="512" height="512" viewBox="0 0 512 512" xmlns="http://www.w3.org/2000/svg">
|
||||
<linearGradient id="linearGradient1" x1="48" y1="48" x2="464" y2="464" gradientUnits="userSpaceOnUse">
|
||||
<stop offset="1e-05" stop-color="#818cf8" stop-opacity="1"/>
|
||||
<stop offset="1" stop-color="#4f46e5" stop-opacity="1"/>
|
||||
</linearGradient>
|
||||
<filter id="shadow" x="-20%" y="-20%" width="140%" height="140%">
|
||||
<feDropShadow dx="0" dy="12" stdDeviation="16" flood-color="#4F46E5" flood-opacity="0.4" />
|
||||
<filter id="filter1" x="0" y="0" width="512" height="512" filterUnits="userSpaceOnUse" primitiveUnits="userSpaceOnUse" color-interpolation-filters="sRGB">
|
||||
<feGaussianBlur stdDeviation="16"/>
|
||||
<feOffset dx="0" dy="12" result="offsetblur"/>
|
||||
<feFlood flood-color="#4f46e5" flood-opacity="0.4"/>
|
||||
<feComposite in2="offsetblur" operator="in"/>
|
||||
<feMerge>
|
||||
<feMergeNode/>
|
||||
<feMergeNode in="SourceGraphic"/>
|
||||
</feMerge>
|
||||
</filter>
|
||||
</defs>
|
||||
|
||||
<!-- App Icon Background -->
|
||||
<rect x="48" y="48" width="416" height="416" rx="96" fill="url(#bg)" filter="url(#shadow)" />
|
||||
|
||||
<!-- Inner Icon Group -->
|
||||
<g stroke="#ffffff" fill="none" stroke-linecap="round" stroke-linejoin="round">
|
||||
<!-- Stopwatch Top Button -->
|
||||
<path d="M256 96 v44" stroke-width="28" />
|
||||
<path d="M224 88 h64" stroke-width="24" />
|
||||
|
||||
<!-- Stopwatch Side Button -->
|
||||
<path d="M352 176 l 24 -24" stroke-width="24" />
|
||||
<!-- Cap for side button -->
|
||||
<path d="M362 138 l 28 28" stroke-width="24" />
|
||||
|
||||
<!-- Outer Ring -->
|
||||
<circle cx="256" cy="272" r="132" stroke-width="28" />
|
||||
|
||||
<!-- Clock Hands -->
|
||||
<!-- Minute Hand -->
|
||||
<path d="M256 184 v 88" stroke-width="24" />
|
||||
<!-- Hour Hand -->
|
||||
<path d="M256 272 l 48 32" stroke-width="24" />
|
||||
|
||||
<!-- Dial Tick Marks -->
|
||||
<g stroke-width="12" opacity="0.6">
|
||||
<line x1="256" y1="172" x2="256" y2="188" />
|
||||
<line x1="256" y1="356" x2="256" y2="372" />
|
||||
<line x1="172" y1="272" x2="188" y2="272" />
|
||||
<line x1="340" y1="272" x2="324" y2="272" />
|
||||
<path id="Path" fill="url(#linearGradient1)" stroke="none" filter="url(#filter1)" d="M 144 48 L 368 48 C 421.019348 48 464 90.980652 464 144 L 464 368 C 464 421.019348 421.019348 464 368 464 L 144 464 C 90.980667 464 48 421.019348 48 368 L 48 144 C 48 90.980652 90.980667 48 144 48 Z"/>
|
||||
<g id="Group">
|
||||
<path id="path1" fill="none" stroke="#ffffff" stroke-width="28" stroke-linecap="round" stroke-linejoin="round" d="M 256 96 L 256 140"/>
|
||||
<path id="path2" fill="none" stroke="#ffffff" stroke-width="24" stroke-linecap="round" stroke-linejoin="round" d="M 224 88 L 288 88"/>
|
||||
<path id="path3" fill="none" stroke="#ffffff" stroke-width="24" stroke-linecap="round" stroke-linejoin="round" d="M 352 176 L 376 152"/>
|
||||
<path id="path4" fill="none" stroke="#ffffff" stroke-width="24" stroke-linecap="round" stroke-linejoin="round" d="M 362 138 L 390 166"/>
|
||||
<path id="path5" fill="none" stroke="#ffffff" stroke-width="28" stroke-linecap="round" stroke-linejoin="round" d="M 388 272 C 388 344.901581 328.901581 404 256 404 C 183.098419 404 124 344.901581 124 272 C 124 199.098419 183.098419 140 256 140 C 328.901581 140 388 199.098419 388 272 Z"/>
|
||||
<path id="path6" fill="none" stroke="#ffffff" stroke-width="24" stroke-linecap="round" stroke-linejoin="round" d="M 256 184 L 256 272"/>
|
||||
<path id="path7" fill="none" stroke="#ffffff" stroke-width="24" stroke-linecap="round" stroke-linejoin="round" d="M 256 272 L 304 304"/>
|
||||
<g id="g1" opacity="0.6">
|
||||
<path id="path8" fill="none" stroke="#ffffff" stroke-width="12" stroke-linecap="round" stroke-linejoin="round" d="M 256 356 L 256 372"/>
|
||||
<path id="path9" fill="none" stroke="#ffffff" stroke-width="12" stroke-linecap="round" stroke-linejoin="round" d="M 172 272 L 188 272"/>
|
||||
<path id="path10" fill="none" stroke="#ffffff" stroke-width="12" stroke-linecap="round" stroke-linejoin="round" d="M 340 272 L 324 272"/>
|
||||
</g>
|
||||
</g>
|
||||
</g>
|
||||
</svg>
|
||||
|
||||
|
Before Width: | Height: | Size: 1.5 KiB After Width: | Height: | Size: 2.8 KiB |
@@ -1,6 +1,11 @@
|
||||
import apiClient from './client';
|
||||
import type { OngoingTimer, TimeEntry } from '@/types';
|
||||
|
||||
export interface UpdateTimerPayload {
|
||||
projectId?: string | null;
|
||||
startTime?: string;
|
||||
}
|
||||
|
||||
export const timerApi = {
|
||||
getOngoing: async (): Promise<OngoingTimer | null> => {
|
||||
const { data } = await apiClient.get<OngoingTimer | null>('/timer');
|
||||
@@ -14,10 +19,8 @@ export const timerApi = {
|
||||
return data;
|
||||
},
|
||||
|
||||
update: async (projectId?: string | null): Promise<OngoingTimer> => {
|
||||
const { data } = await apiClient.put<OngoingTimer>('/timer', {
|
||||
projectId,
|
||||
});
|
||||
update: async (payload: UpdateTimerPayload): Promise<OngoingTimer> => {
|
||||
const { data } = await apiClient.put<OngoingTimer>('/timer', payload);
|
||||
return data;
|
||||
},
|
||||
|
||||
@@ -27,4 +30,8 @@ export const timerApi = {
|
||||
});
|
||||
return data;
|
||||
},
|
||||
|
||||
cancel: async (): Promise<void> => {
|
||||
await apiClient.delete('/timer');
|
||||
},
|
||||
};
|
||||
@@ -4,9 +4,9 @@ import { TimerWidget } from './TimerWidget';
|
||||
|
||||
export function Layout() {
|
||||
return (
|
||||
<div className="min-h-screen bg-gray-50">
|
||||
<div className="h-[100vh] w-[100vw] flex flex-col bg-gray-50">
|
||||
<Navbar />
|
||||
<main className="pt-4 pb-24">
|
||||
<main className="pt-4 pb-8 grow overflow-auto">
|
||||
<div className="max-w-7xl mx-auto px-4 sm:px-6 lg:px-8">
|
||||
<Outlet />
|
||||
</div>
|
||||
|
||||
@@ -47,20 +47,27 @@ export function Navbar() {
|
||||
<div className="max-w-7xl mx-auto px-4 sm:px-6 lg:px-8">
|
||||
<div className="flex justify-between h-16">
|
||||
<div className="flex">
|
||||
<div className="flex-shrink-0 flex items-center">
|
||||
<Clock className="h-8 w-8 text-primary-600" />
|
||||
<NavLink
|
||||
className="flex-shrink-0 flex items-center"
|
||||
to={"/dashboard"}
|
||||
>
|
||||
<img
|
||||
src="/icon.svg"
|
||||
alt="TimeTracker Logo"
|
||||
className="h-8 w-8 drop-shadow-sm"
|
||||
/>
|
||||
<span className="ml-2 text-xl font-bold text-gray-900">
|
||||
TimeTracker
|
||||
</span>
|
||||
</div>
|
||||
<div className="hidden sm:ml-8 sm:flex sm:space-x-4">
|
||||
</NavLink>
|
||||
<div className="hidden sm:ml-8 sm:flex sm:space-x-4 items-center">
|
||||
{/* Main Navigation Items */}
|
||||
{mainNavItems.map((item) => (
|
||||
<NavLink
|
||||
key={item.to}
|
||||
to={item.to}
|
||||
className={({ isActive }) =>
|
||||
`inline-flex items-center px-3 py-2 text-sm font-medium rounded-md transition-colors ${
|
||||
`inline-flex items-center px-3 py-2 text-sm font-medium rounded-md transition-colors h-min ${
|
||||
isActive
|
||||
? "text-primary-600 bg-primary-50"
|
||||
: "text-gray-600 hover:text-gray-900 hover:bg-gray-50"
|
||||
|
||||
@@ -3,25 +3,35 @@ interface StatCardProps {
|
||||
label: string;
|
||||
value: string;
|
||||
color: 'blue' | 'green' | 'purple' | 'orange';
|
||||
/** When true, renders a pulsing green dot to signal a live/active state. */
|
||||
indicator?: boolean;
|
||||
}
|
||||
|
||||
const colorClasses: Record<StatCardProps['color'], string> = {
|
||||
const colorClasses: Record<NonNullable<StatCardProps['color']>, string> = {
|
||||
blue: 'bg-blue-50 text-blue-600',
|
||||
green: 'bg-green-50 text-green-600',
|
||||
purple: 'bg-purple-50 text-purple-600',
|
||||
orange: 'bg-orange-50 text-orange-600',
|
||||
};
|
||||
|
||||
export function StatCard({ icon: Icon, label, value, color }: StatCardProps) {
|
||||
export function StatCard({ icon: Icon, label, value, color, indicator }: StatCardProps) {
|
||||
return (
|
||||
<div className="card p-4">
|
||||
<div className="flex items-center">
|
||||
<div className={`p-3 rounded-lg ${colorClasses[color]}`}>
|
||||
<Icon className="h-6 w-6" />
|
||||
</div>
|
||||
<div className="ml-4">
|
||||
<div className="ml-4 flex-1">
|
||||
<p className="text-sm font-medium text-gray-600">{label}</p>
|
||||
<p className="text-2xl font-bold text-gray-900">{value}</p>
|
||||
<div className="flex items-center gap-2">
|
||||
{indicator && (
|
||||
<span
|
||||
className="inline-block h-2.5 w-2.5 rounded-full bg-red-500 animate-pulse"
|
||||
title="Timer running"
|
||||
/>
|
||||
)}
|
||||
<p className="text-2xl font-bold text-gray-900">{value}</p>
|
||||
</div>
|
||||
</div>
|
||||
</div>
|
||||
</div>
|
||||
|
||||
@@ -20,6 +20,7 @@ export function TimeEntryFormModal({ entry, onClose, createTimeEntry, updateTime
|
||||
return {
|
||||
startTime: getLocalISOString(new Date(entry.startTime)),
|
||||
endTime: getLocalISOString(new Date(entry.endTime)),
|
||||
breakMinutes: entry.breakMinutes,
|
||||
description: entry.description || '',
|
||||
projectId: entry.projectId,
|
||||
};
|
||||
@@ -29,6 +30,7 @@ export function TimeEntryFormModal({ entry, onClose, createTimeEntry, updateTime
|
||||
return {
|
||||
startTime: getLocalISOString(oneHourAgo),
|
||||
endTime: getLocalISOString(now),
|
||||
breakMinutes: 0,
|
||||
description: '',
|
||||
projectId: projects?.[0]?.id || '',
|
||||
};
|
||||
@@ -97,6 +99,16 @@ export function TimeEntryFormModal({ entry, onClose, createTimeEntry, updateTime
|
||||
/>
|
||||
</div>
|
||||
</div>
|
||||
<div>
|
||||
<label className="label">Break (minutes)</label>
|
||||
<input
|
||||
type="number"
|
||||
min="0"
|
||||
value={formData.breakMinutes ?? 0}
|
||||
onChange={(e) => setFormData({ ...formData, breakMinutes: parseInt(e.target.value) || 0 })}
|
||||
className="input"
|
||||
/>
|
||||
</div>
|
||||
<div>
|
||||
<label className="label">Description</label>
|
||||
<textarea
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
import { useState } from "react";
|
||||
import { Play, Square, ChevronDown } from "lucide-react";
|
||||
import { useState, useRef } from "react";
|
||||
import { Play, Square, ChevronDown, Pencil, Check, X, Trash2 } from "lucide-react";
|
||||
import { useTimer } from "@/contexts/TimerContext";
|
||||
import { useProjects } from "@/hooks/useProjects";
|
||||
import { ProjectColorDot } from "@/components/ProjectColorDot";
|
||||
@@ -27,6 +27,21 @@ function TimerDisplay({ totalSeconds }: { totalSeconds: number }) {
|
||||
);
|
||||
}
|
||||
|
||||
/** Converts a HH:mm string to an ISO datetime, inferring the correct date.
|
||||
* If the resulting time would be in the future, it is assumed to belong to the previous day.
|
||||
*/
|
||||
function timeInputToIso(timeValue: string): string {
|
||||
const [hours, minutes] = timeValue.split(":").map(Number);
|
||||
const now = new Date();
|
||||
const candidate = new Date(now);
|
||||
candidate.setHours(hours, minutes, 0, 0);
|
||||
// If the candidate is in the future, roll back one day
|
||||
if (candidate > now) {
|
||||
candidate.setDate(candidate.getDate() - 1);
|
||||
}
|
||||
return candidate.toISOString();
|
||||
}
|
||||
|
||||
export function TimerWidget() {
|
||||
const {
|
||||
ongoingTimer,
|
||||
@@ -34,12 +49,19 @@ export function TimerWidget() {
|
||||
elapsedSeconds,
|
||||
startTimer,
|
||||
stopTimer,
|
||||
cancelTimer,
|
||||
updateTimerProject,
|
||||
updateTimerStartTime,
|
||||
} = useTimer();
|
||||
const { projects } = useProjects();
|
||||
const [showProjectSelect, setShowProjectSelect] = useState(false);
|
||||
const [error, setError] = useState<string | null>(null);
|
||||
|
||||
// Start time editing state
|
||||
const [editingStartTime, setEditingStartTime] = useState(false);
|
||||
const [startTimeInput, setStartTimeInput] = useState("");
|
||||
const startTimeInputRef = useRef<HTMLInputElement>(null);
|
||||
|
||||
const handleStart = async () => {
|
||||
setError(null);
|
||||
try {
|
||||
@@ -58,6 +80,15 @@ export function TimerWidget() {
|
||||
}
|
||||
};
|
||||
|
||||
const handleCancelTimer = async () => {
|
||||
setError(null);
|
||||
try {
|
||||
await cancelTimer();
|
||||
} catch (err) {
|
||||
setError(err instanceof Error ? err.message : "Failed to cancel timer");
|
||||
}
|
||||
};
|
||||
|
||||
const handleProjectChange = async (projectId: string) => {
|
||||
setError(null);
|
||||
try {
|
||||
@@ -78,9 +109,45 @@ export function TimerWidget() {
|
||||
}
|
||||
};
|
||||
|
||||
const handleStartEditStartTime = () => {
|
||||
if (!ongoingTimer) return;
|
||||
const start = new Date(ongoingTimer.startTime);
|
||||
const hh = start.getHours().toString().padStart(2, "0");
|
||||
const mm = start.getMinutes().toString().padStart(2, "0");
|
||||
setStartTimeInput(`${hh}:${mm}`);
|
||||
setEditingStartTime(true);
|
||||
// Focus the input on next render
|
||||
setTimeout(() => startTimeInputRef.current?.focus(), 0);
|
||||
};
|
||||
|
||||
const handleCancelEditStartTime = () => {
|
||||
setEditingStartTime(false);
|
||||
setError(null);
|
||||
};
|
||||
|
||||
const handleConfirmStartTime = async () => {
|
||||
if (!startTimeInput) return;
|
||||
setError(null);
|
||||
try {
|
||||
const iso = timeInputToIso(startTimeInput);
|
||||
await updateTimerStartTime(iso);
|
||||
setEditingStartTime(false);
|
||||
} catch (err) {
|
||||
setError(err instanceof Error ? err.message : "Failed to update start time");
|
||||
}
|
||||
};
|
||||
|
||||
const handleStartTimeKeyDown = (e: React.KeyboardEvent<HTMLInputElement>) => {
|
||||
if (e.key === "Enter") {
|
||||
void handleConfirmStartTime();
|
||||
} else if (e.key === "Escape") {
|
||||
handleCancelEditStartTime();
|
||||
}
|
||||
};
|
||||
|
||||
if (isLoading) {
|
||||
return (
|
||||
<div className="fixed bottom-0 left-0 right-0 bg-white border-t border-gray-200 py-4 shadow-lg">
|
||||
<div className="bg-white border-t border-gray-200 py-4 shadow-lg">
|
||||
<div className="max-w-7xl mx-auto px-4 sm:px-6 lg:px-8 flex items-center justify-center">
|
||||
<div className="animate-spin rounded-full h-6 w-6 border-b-2 border-primary-600"></div>
|
||||
</div>
|
||||
@@ -89,26 +156,74 @@ export function TimerWidget() {
|
||||
}
|
||||
|
||||
return (
|
||||
<div className="fixed bottom-0 left-0 right-0 bg-white border-t border-gray-200 py-4 shadow-lg z-50">
|
||||
<div className="bg-white border-t border-gray-200 py-4 shadow-lg">
|
||||
<div className="max-w-7xl mx-auto px-4 sm:px-6 lg:px-8 flex flex-wrap sm:flex-nowrap items-center gap-2 sm:justify-between">
|
||||
{ongoingTimer ? (
|
||||
<>
|
||||
{/* Row 1 (mobile): timer + stop side by side. On sm+ dissolves into the parent flex row via contents. */}
|
||||
<div className="flex items-center justify-between w-full sm:contents">
|
||||
{/* Timer Display */}
|
||||
{/* Timer Display + Start Time Editor */}
|
||||
<div className="flex items-center space-x-2 shrink-0">
|
||||
<div className="w-3 h-3 bg-red-500 rounded-full animate-pulse"></div>
|
||||
<TimerDisplay totalSeconds={elapsedSeconds} />
|
||||
{editingStartTime ? (
|
||||
<div className="flex items-center space-x-1">
|
||||
<span className="text-xs text-gray-500 mr-1">Started at</span>
|
||||
<input
|
||||
ref={startTimeInputRef}
|
||||
type="time"
|
||||
value={startTimeInput}
|
||||
onChange={(e) => setStartTimeInput(e.target.value)}
|
||||
onKeyDown={handleStartTimeKeyDown}
|
||||
className="font-mono text-lg font-bold text-gray-900 border border-primary-400 rounded px-2 py-0.5 focus:outline-none focus:ring-2 focus:ring-primary-500 w-28"
|
||||
/>
|
||||
<button
|
||||
onClick={() => void handleConfirmStartTime()}
|
||||
title="Confirm"
|
||||
className="p-1 text-green-600 hover:text-green-700 hover:bg-green-50 rounded"
|
||||
>
|
||||
<Check className="h-4 w-4" />
|
||||
</button>
|
||||
<button
|
||||
onClick={handleCancelEditStartTime}
|
||||
title="Cancel"
|
||||
className="p-1 text-gray-400 hover:text-gray-600 hover:bg-gray-100 rounded"
|
||||
>
|
||||
<X className="h-4 w-4" />
|
||||
</button>
|
||||
</div>
|
||||
) : (
|
||||
<div className="flex items-center space-x-2">
|
||||
<TimerDisplay totalSeconds={elapsedSeconds} />
|
||||
<button
|
||||
onClick={handleStartEditStartTime}
|
||||
title="Adjust start time"
|
||||
className="p-1 text-gray-400 hover:text-gray-600 hover:bg-gray-100 rounded transition-colors"
|
||||
>
|
||||
<Pencil className="h-3.5 w-3.5" />
|
||||
</button>
|
||||
</div>
|
||||
)}
|
||||
</div>
|
||||
|
||||
{/* Stop Button */}
|
||||
<button
|
||||
onClick={handleStop}
|
||||
className="flex items-center space-x-2 px-6 py-3 bg-red-600 text-white rounded-lg font-medium hover:bg-red-700 transition-colors shrink-0 sm:order-last"
|
||||
>
|
||||
<Square className="h-5 w-5 fill-current" />
|
||||
<span>Stop</span>
|
||||
</button>
|
||||
{/* Stop + Cancel Buttons */}
|
||||
<div className="flex items-center space-x-2 shrink-0 sm:order-last">
|
||||
<button
|
||||
onClick={() => void handleCancelTimer()}
|
||||
title="Discard timer"
|
||||
className="p-2 text-gray-400 hover:text-gray-600 hover:bg-gray-100 rounded-lg transition-colors"
|
||||
>
|
||||
<Trash2 className="h-5 w-5" />
|
||||
</button>
|
||||
<button
|
||||
onClick={handleStop}
|
||||
disabled={!ongoingTimer.project}
|
||||
title={!ongoingTimer.project ? "Select a project to stop the timer" : undefined}
|
||||
className="flex items-center space-x-2 px-6 py-3 bg-red-600 text-white rounded-lg font-medium hover:bg-red-700 transition-colors disabled:opacity-40 disabled:cursor-not-allowed disabled:hover:bg-red-600"
|
||||
>
|
||||
<Square className="h-5 w-5 fill-current" />
|
||||
<span>Stop</span>
|
||||
</button>
|
||||
</div>
|
||||
</div>
|
||||
|
||||
{/* Project Selector — full width on mobile, auto on desktop */}
|
||||
|
||||
@@ -17,6 +17,8 @@ interface TimerContextType {
|
||||
elapsedSeconds: number;
|
||||
startTimer: (projectId?: string) => Promise<void>;
|
||||
updateTimerProject: (projectId?: string | null) => Promise<void>;
|
||||
updateTimerStartTime: (startTime: string) => Promise<void>;
|
||||
cancelTimer: () => Promise<void>;
|
||||
stopTimer: (projectId?: string) => Promise<TimeEntry | null>;
|
||||
}
|
||||
|
||||
@@ -84,6 +86,14 @@ export function TimerProvider({ children }: { children: ReactNode }) {
|
||||
},
|
||||
});
|
||||
|
||||
// Cancel timer mutation
|
||||
const cancelMutation = useMutation({
|
||||
mutationFn: timerApi.cancel,
|
||||
onSuccess: () => {
|
||||
queryClient.invalidateQueries({ queryKey: ["ongoingTimer"] });
|
||||
},
|
||||
});
|
||||
|
||||
// Stop timer mutation
|
||||
const stopMutation = useMutation({
|
||||
mutationFn: timerApi.stop,
|
||||
@@ -102,11 +112,22 @@ export function TimerProvider({ children }: { children: ReactNode }) {
|
||||
|
||||
const updateTimerProject = useCallback(
|
||||
async (projectId?: string | null) => {
|
||||
await updateMutation.mutateAsync(projectId);
|
||||
await updateMutation.mutateAsync({ projectId });
|
||||
},
|
||||
[updateMutation],
|
||||
);
|
||||
|
||||
const updateTimerStartTime = useCallback(
|
||||
async (startTime: string) => {
|
||||
await updateMutation.mutateAsync({ startTime });
|
||||
},
|
||||
[updateMutation],
|
||||
);
|
||||
|
||||
const cancelTimer = useCallback(async () => {
|
||||
await cancelMutation.mutateAsync();
|
||||
}, [cancelMutation]);
|
||||
|
||||
const stopTimer = useCallback(
|
||||
async (projectId?: string): Promise<TimeEntry | null> => {
|
||||
try {
|
||||
@@ -127,6 +148,8 @@ export function TimerProvider({ children }: { children: ReactNode }) {
|
||||
elapsedSeconds,
|
||||
startTimer,
|
||||
updateTimerProject,
|
||||
updateTimerStartTime,
|
||||
cancelTimer,
|
||||
stopTimer,
|
||||
}}
|
||||
>
|
||||
|
||||
@@ -1,5 +1,6 @@
|
||||
import { useQuery, useMutation, useQueryClient } from '@tanstack/react-query';
|
||||
import { clientTargetsApi } from '@/api/clientTargets';
|
||||
import { useTimer } from '@/contexts/TimerContext';
|
||||
import type {
|
||||
CreateClientTargetInput,
|
||||
UpdateClientTargetInput,
|
||||
@@ -8,10 +9,13 @@ import type {
|
||||
|
||||
export function useClientTargets() {
|
||||
const queryClient = useQueryClient();
|
||||
const { ongoingTimer } = useTimer();
|
||||
|
||||
const { data: targets, isLoading, error } = useQuery({
|
||||
queryKey: ['clientTargets'],
|
||||
queryFn: clientTargetsApi.getAll,
|
||||
// Poll every 30 s while a timer is running so the balance stays current
|
||||
refetchInterval: ongoingTimer ? 30_000 : false,
|
||||
});
|
||||
|
||||
const createTarget = useMutation({
|
||||
|
||||
@@ -14,33 +14,10 @@ import type {
|
||||
CreateCorrectionInput,
|
||||
} from '@/types';
|
||||
|
||||
// Convert a <input type="week"> value like "2026-W07" to the Monday date "2026-02-16"
|
||||
function weekInputToMonday(weekValue: string): string {
|
||||
const [yearStr, weekStr] = weekValue.split('-W');
|
||||
const year = parseInt(yearStr, 10);
|
||||
const week = parseInt(weekStr, 10);
|
||||
// ISO week 1 is the week containing the first Thursday of January
|
||||
const jan4 = new Date(Date.UTC(year, 0, 4));
|
||||
const jan4Day = jan4.getUTCDay() || 7; // Mon=1..Sun=7
|
||||
const monday = new Date(jan4);
|
||||
monday.setUTCDate(jan4.getUTCDate() - jan4Day + 1 + (week - 1) * 7);
|
||||
return monday.toISOString().split('T')[0];
|
||||
}
|
||||
|
||||
// Convert a YYYY-MM-DD Monday to "YYYY-Www" for the week input
|
||||
function mondayToWeekInput(dateStr: string): string {
|
||||
const date = new Date(dateStr + 'T00:00:00Z');
|
||||
// ISO week number calculation
|
||||
const jan4 = new Date(Date.UTC(date.getUTCFullYear(), 0, 4));
|
||||
const jan4Day = jan4.getUTCDay() || 7;
|
||||
const firstMonday = new Date(jan4);
|
||||
firstMonday.setUTCDate(jan4.getUTCDate() - jan4Day + 1);
|
||||
const diff = date.getTime() - firstMonday.getTime();
|
||||
const week = Math.floor(diff / (7 * 24 * 3600 * 1000)) + 1;
|
||||
// Handle year boundary: if week > 52 we might be in week 1 of next year
|
||||
const year = date.getUTCFullYear();
|
||||
return `${year}-W${week.toString().padStart(2, '0')}`;
|
||||
}
|
||||
const ALL_DAYS = ['MON', 'TUE', 'WED', 'THU', 'FRI', 'SAT', 'SUN'] as const;
|
||||
const DAY_LABELS: Record<string, string> = {
|
||||
MON: 'Mon', TUE: 'Tue', WED: 'Wed', THU: 'Thu', FRI: 'Fri', SAT: 'Sat', SUN: 'Sun',
|
||||
};
|
||||
|
||||
function balanceLabel(seconds: number): { text: string; color: string } {
|
||||
if (seconds === 0) return { text: '±0', color: 'text-gray-500' };
|
||||
@@ -58,7 +35,12 @@ function ClientTargetPanel({
|
||||
}: {
|
||||
client: Client;
|
||||
target: ClientTargetWithBalance | undefined;
|
||||
onCreated: (weeklyHours: number, startDate: string) => Promise<void>;
|
||||
onCreated: (input: {
|
||||
targetHours: number;
|
||||
periodType: 'weekly' | 'monthly';
|
||||
workingDays: string[];
|
||||
startDate: string;
|
||||
}) => Promise<void>;
|
||||
onDeleted: () => Promise<void>;
|
||||
}) {
|
||||
const { addCorrection, deleteCorrection, updateTarget } = useClientTargets();
|
||||
@@ -69,7 +51,9 @@ function ClientTargetPanel({
|
||||
|
||||
// Create/edit form state
|
||||
const [formHours, setFormHours] = useState('');
|
||||
const [formWeek, setFormWeek] = useState('');
|
||||
const [formPeriodType, setFormPeriodType] = useState<'weekly' | 'monthly'>('weekly');
|
||||
const [formWorkingDays, setFormWorkingDays] = useState<string[]>(['MON', 'TUE', 'WED', 'THU', 'FRI']);
|
||||
const [formStartDate, setFormStartDate] = useState('');
|
||||
const [formError, setFormError] = useState<string | null>(null);
|
||||
const [formSaving, setFormSaving] = useState(false);
|
||||
|
||||
@@ -81,13 +65,13 @@ function ClientTargetPanel({
|
||||
const [corrError, setCorrError] = useState<string | null>(null);
|
||||
const [corrSaving, setCorrSaving] = useState(false);
|
||||
|
||||
const todayIso = new Date().toISOString().split('T')[0];
|
||||
|
||||
const openCreate = () => {
|
||||
setFormHours('');
|
||||
const today = new Date();
|
||||
const day = today.getUTCDay() || 7;
|
||||
const monday = new Date(today);
|
||||
monday.setUTCDate(today.getUTCDate() - day + 1);
|
||||
setFormWeek(mondayToWeekInput(monday.toISOString().split('T')[0]));
|
||||
setFormPeriodType('weekly');
|
||||
setFormWorkingDays(['MON', 'TUE', 'WED', 'THU', 'FRI']);
|
||||
setFormStartDate(todayIso);
|
||||
setFormError(null);
|
||||
setEditing(false);
|
||||
setShowForm(true);
|
||||
@@ -95,32 +79,56 @@ function ClientTargetPanel({
|
||||
|
||||
const openEdit = () => {
|
||||
if (!target) return;
|
||||
setFormHours(String(target.weeklyHours));
|
||||
setFormWeek(mondayToWeekInput(target.startDate));
|
||||
setFormHours(String(target.targetHours));
|
||||
setFormPeriodType(target.periodType);
|
||||
setFormWorkingDays([...target.workingDays]);
|
||||
setFormStartDate(target.startDate);
|
||||
setFormError(null);
|
||||
setEditing(true);
|
||||
setShowForm(true);
|
||||
};
|
||||
|
||||
const toggleDay = (day: string) => {
|
||||
setFormWorkingDays(prev =>
|
||||
prev.includes(day) ? prev.filter(d => d !== day) : [...prev, day],
|
||||
);
|
||||
};
|
||||
|
||||
const handleFormSubmit = async (e: React.FormEvent) => {
|
||||
e.preventDefault();
|
||||
setFormError(null);
|
||||
const hours = parseFloat(formHours);
|
||||
if (isNaN(hours) || hours <= 0 || hours > 168) {
|
||||
setFormError('Weekly hours must be between 0 and 168');
|
||||
setFormError(`${formPeriodType === 'weekly' ? 'Weekly' : 'Monthly'} hours must be between 0 and 168`);
|
||||
return;
|
||||
}
|
||||
if (!formWeek) {
|
||||
setFormError('Please select a start week');
|
||||
if (formWorkingDays.length === 0) {
|
||||
setFormError('Select at least one working day');
|
||||
return;
|
||||
}
|
||||
if (!formStartDate) {
|
||||
setFormError('Please select a start date');
|
||||
return;
|
||||
}
|
||||
const startDate = weekInputToMonday(formWeek);
|
||||
setFormSaving(true);
|
||||
try {
|
||||
if (editing && target) {
|
||||
await updateTarget.mutateAsync({ id: target.id, input: { weeklyHours: hours, startDate } });
|
||||
await updateTarget.mutateAsync({
|
||||
id: target.id,
|
||||
input: {
|
||||
targetHours: hours,
|
||||
periodType: formPeriodType,
|
||||
workingDays: formWorkingDays,
|
||||
startDate: formStartDate,
|
||||
},
|
||||
});
|
||||
} else {
|
||||
await onCreated(hours, startDate);
|
||||
await onCreated({
|
||||
targetHours: hours,
|
||||
periodType: formPeriodType,
|
||||
workingDays: formWorkingDays,
|
||||
startDate: formStartDate,
|
||||
});
|
||||
}
|
||||
setShowForm(false);
|
||||
} catch (err) {
|
||||
@@ -185,23 +193,46 @@ function ClientTargetPanel({
|
||||
className="flex items-center gap-1.5 text-xs text-primary-600 hover:text-primary-700 font-medium"
|
||||
>
|
||||
<Target className="h-3.5 w-3.5" />
|
||||
Set weekly target
|
||||
Set target
|
||||
</button>
|
||||
</div>
|
||||
);
|
||||
}
|
||||
|
||||
if (showForm) {
|
||||
const hoursLabel = formPeriodType === 'weekly' ? 'Hours/week' : 'Hours/month';
|
||||
return (
|
||||
<div className="mt-3 pt-3 border-t border-gray-100">
|
||||
<p className="text-xs font-medium text-gray-700 mb-2">
|
||||
{editing ? 'Edit target' : 'Set weekly target'}
|
||||
{editing ? 'Edit target' : 'Set target'}
|
||||
</p>
|
||||
<form onSubmit={handleFormSubmit} className="space-y-2">
|
||||
{formError && <p className="text-xs text-red-600">{formError}</p>}
|
||||
|
||||
{/* Period type */}
|
||||
<div>
|
||||
<label className="block text-xs text-gray-500 mb-0.5">Period</label>
|
||||
<div className="flex gap-2">
|
||||
{(['weekly', 'monthly'] as const).map(pt => (
|
||||
<label key={pt} className="flex items-center gap-1 text-xs cursor-pointer">
|
||||
<input
|
||||
type="radio"
|
||||
name="periodType"
|
||||
value={pt}
|
||||
checked={formPeriodType === pt}
|
||||
onChange={() => setFormPeriodType(pt)}
|
||||
className="accent-primary-600"
|
||||
/>
|
||||
{pt.charAt(0).toUpperCase() + pt.slice(1)}
|
||||
</label>
|
||||
))}
|
||||
</div>
|
||||
</div>
|
||||
|
||||
{/* Hours + Start Date */}
|
||||
<div className="flex gap-2">
|
||||
<div className="flex-1">
|
||||
<label className="block text-xs text-gray-500 mb-0.5">Hours/week</label>
|
||||
<label className="block text-xs text-gray-500 mb-0.5">{hoursLabel}</label>
|
||||
<input
|
||||
type="number"
|
||||
value={formHours}
|
||||
@@ -215,16 +246,41 @@ function ClientTargetPanel({
|
||||
/>
|
||||
</div>
|
||||
<div className="flex-1">
|
||||
<label className="block text-xs text-gray-500 mb-0.5">Starting week</label>
|
||||
<label className="block text-xs text-gray-500 mb-0.5">Start date</label>
|
||||
<input
|
||||
type="week"
|
||||
value={formWeek}
|
||||
onChange={e => setFormWeek(e.target.value)}
|
||||
type="date"
|
||||
value={formStartDate}
|
||||
onChange={e => setFormStartDate(e.target.value)}
|
||||
className="input text-sm py-1"
|
||||
required
|
||||
/>
|
||||
</div>
|
||||
</div>
|
||||
|
||||
{/* Working days */}
|
||||
<div>
|
||||
<label className="block text-xs text-gray-500 mb-0.5">Working days</label>
|
||||
<div className="flex gap-1 flex-wrap">
|
||||
{ALL_DAYS.map(day => {
|
||||
const active = formWorkingDays.includes(day);
|
||||
return (
|
||||
<button
|
||||
key={day}
|
||||
type="button"
|
||||
onClick={() => toggleDay(day)}
|
||||
className={`text-xs px-2 py-0.5 rounded border font-medium transition-colors ${
|
||||
active
|
||||
? 'bg-primary-600 border-primary-600 text-white'
|
||||
: 'bg-white border-gray-300 text-gray-600 hover:border-primary-400'
|
||||
}`}
|
||||
>
|
||||
{DAY_LABELS[day]}
|
||||
</button>
|
||||
);
|
||||
})}
|
||||
</div>
|
||||
</div>
|
||||
|
||||
<div className="flex gap-2 justify-end">
|
||||
<button
|
||||
type="button"
|
||||
@@ -248,6 +304,7 @@ function ClientTargetPanel({
|
||||
|
||||
// Target exists — show summary + expandable details
|
||||
const balance = balanceLabel(target!.totalBalanceSeconds);
|
||||
const periodLabel = target!.periodType === 'weekly' ? 'week' : 'month';
|
||||
|
||||
return (
|
||||
<div className="mt-3 pt-3 border-t border-gray-100">
|
||||
@@ -256,9 +313,15 @@ function ClientTargetPanel({
|
||||
<div className="flex items-center gap-2">
|
||||
<Target className="h-3.5 w-3.5 text-gray-400 shrink-0" />
|
||||
<span className="text-xs text-gray-600">
|
||||
<span className="font-medium">{target!.weeklyHours}h</span>/week
|
||||
<span className="font-medium">{target!.targetHours}h</span>/{periodLabel}
|
||||
</span>
|
||||
<span className={`text-xs font-semibold ${balance.color}`}>{balance.text}</span>
|
||||
{target!.hasOngoingTimer && (
|
||||
<span
|
||||
className="inline-block h-2 w-2 rounded-full bg-green-500 animate-pulse"
|
||||
title="Timer running — balance updates every 30 s"
|
||||
/>
|
||||
)}
|
||||
</div>
|
||||
<div className="flex items-center gap-1">
|
||||
<button
|
||||
@@ -531,8 +594,14 @@ export function ClientsPage() {
|
||||
<ClientTargetPanel
|
||||
client={client}
|
||||
target={target}
|
||||
onCreated={async (weeklyHours, startDate) => {
|
||||
await createTarget.mutateAsync({ clientId: client.id, weeklyHours, startDate });
|
||||
onCreated={async ({ targetHours, periodType, workingDays, startDate }) => {
|
||||
await createTarget.mutateAsync({
|
||||
clientId: client.id,
|
||||
targetHours,
|
||||
periodType,
|
||||
workingDays,
|
||||
startDate,
|
||||
});
|
||||
}}
|
||||
onDeleted={async () => {
|
||||
if (target) await deleteTarget.mutateAsync(target.id);
|
||||
|
||||
@@ -3,6 +3,7 @@ import { Link } from "react-router-dom";
|
||||
import { Clock, Calendar, Briefcase, TrendingUp, Target, Edit2, Trash2 } from "lucide-react";
|
||||
import { useTimeEntries } from "@/hooks/useTimeEntries";
|
||||
import { useClientTargets } from "@/hooks/useClientTargets";
|
||||
import { useTimer } from "@/contexts/TimerContext";
|
||||
import { ProjectColorDot } from "@/components/ProjectColorDot";
|
||||
import { StatCard } from "@/components/StatCard";
|
||||
import { TimeEntryFormModal } from "@/components/TimeEntryFormModal";
|
||||
@@ -30,6 +31,7 @@ export function DashboardPage() {
|
||||
});
|
||||
|
||||
const { targets } = useClientTargets();
|
||||
const { ongoingTimer, elapsedSeconds } = useTimer();
|
||||
|
||||
const [isModalOpen, setIsModalOpen] = useState(false);
|
||||
const [editingEntry, setEditingEntry] = useState<TimeEntry | null>(null);
|
||||
@@ -54,12 +56,19 @@ export function DashboardPage() {
|
||||
}
|
||||
};
|
||||
|
||||
const totalTodaySeconds =
|
||||
const completedTodaySeconds =
|
||||
todayEntries?.entries.reduce((total, entry) => {
|
||||
return total + calculateDuration(entry.startTime, entry.endTime);
|
||||
}, 0) || 0;
|
||||
return total + calculateDuration(entry.startTime, entry.endTime, entry.breakMinutes);
|
||||
}, 0) ?? 0;
|
||||
|
||||
const targetsWithData = targets?.filter(t => t.weeks.length > 0) ?? [];
|
||||
// Only add the running timer if it started today (not a timer left running from yesterday)
|
||||
const timerStartedToday =
|
||||
ongoingTimer !== null &&
|
||||
new Date(ongoingTimer.startTime) >= startOfDay(today);
|
||||
|
||||
const totalTodaySeconds = completedTodaySeconds + (timerStartedToday ? elapsedSeconds : 0);
|
||||
|
||||
const targetsWithData = targets?.filter(t => t.periods.length > 0) ?? [];
|
||||
|
||||
return (
|
||||
<div className="space-y-6">
|
||||
@@ -78,6 +87,7 @@ export function DashboardPage() {
|
||||
label="Today"
|
||||
value={formatDurationHoursMinutes(totalTodaySeconds)}
|
||||
color="blue"
|
||||
indicator={timerStartedToday}
|
||||
/>
|
||||
<StatCard
|
||||
icon={Calendar}
|
||||
@@ -108,7 +118,7 @@ export function DashboardPage() {
|
||||
<div className="card">
|
||||
<div className="flex items-center gap-2 mb-4">
|
||||
<Target className="h-5 w-5 text-primary-600" />
|
||||
<h2 className="text-lg font-semibold text-gray-900">Weekly Targets</h2>
|
||||
<h2 className="text-lg font-semibold text-gray-900">Targets</h2>
|
||||
</div>
|
||||
<div className="space-y-3">
|
||||
{targetsWithData.map(target => {
|
||||
@@ -116,8 +126,9 @@ export function DashboardPage() {
|
||||
const absBalance = Math.abs(balance);
|
||||
const isOver = balance > 0;
|
||||
const isEven = balance === 0;
|
||||
const currentWeekTracked = formatDurationHoursMinutes(target.currentWeekTrackedSeconds);
|
||||
const currentWeekTarget = formatDurationHoursMinutes(target.currentWeekTargetSeconds);
|
||||
const currentPeriodTracked = formatDurationHoursMinutes(target.currentPeriodTrackedSeconds);
|
||||
const currentPeriodTarget = formatDurationHoursMinutes(target.currentPeriodTargetSeconds);
|
||||
const periodLabel = target.periodType === 'weekly' ? 'This week' : 'This month';
|
||||
|
||||
return (
|
||||
<div
|
||||
@@ -127,23 +138,31 @@ export function DashboardPage() {
|
||||
<div>
|
||||
<p className="text-sm font-medium text-gray-900">{target.clientName}</p>
|
||||
<p className="text-xs text-gray-500 mt-0.5">
|
||||
This week: {currentWeekTracked} / {currentWeekTarget}
|
||||
</p>
|
||||
{periodLabel}: {currentPeriodTracked} / {currentPeriodTarget}
|
||||
</p>
|
||||
</div>
|
||||
<div className="text-right">
|
||||
<p
|
||||
className={`text-sm font-bold ${
|
||||
isEven
|
||||
? 'text-gray-500'
|
||||
: isOver
|
||||
? 'text-green-600'
|
||||
: 'text-red-600'
|
||||
}`}
|
||||
>
|
||||
{isEven
|
||||
? '±0'
|
||||
: (isOver ? '+' : '−') + formatDurationHoursMinutes(absBalance)}
|
||||
</p>
|
||||
<div className="flex items-center justify-end gap-1.5">
|
||||
{target.hasOngoingTimer && (
|
||||
<span
|
||||
className="inline-block h-2 w-2 rounded-full bg-red-500 animate-pulse"
|
||||
title="Timer running — balance updates every 30 s"
|
||||
/>
|
||||
)}
|
||||
<p
|
||||
className={`text-sm font-bold ${
|
||||
isEven
|
||||
? 'text-gray-500'
|
||||
: isOver
|
||||
? 'text-green-600'
|
||||
: 'text-red-600'
|
||||
}`}
|
||||
>
|
||||
{isEven
|
||||
? '±0'
|
||||
: (isOver ? '+' : '−') + formatDurationHoursMinutes(absBalance)}
|
||||
</p>
|
||||
</div>
|
||||
<p className="text-xs text-gray-400">running balance</p>
|
||||
</div>
|
||||
</div>
|
||||
@@ -216,7 +235,10 @@ export function DashboardPage() {
|
||||
<div className="text-xs text-gray-400">{formatTime(entry.startTime)} – {formatTime(entry.endTime)}</div>
|
||||
</td>
|
||||
<td className="px-4 py-3 whitespace-nowrap text-sm text-gray-900 font-mono">
|
||||
{formatDurationFromDatesHoursMinutes(entry.startTime, entry.endTime)}
|
||||
{formatDurationFromDatesHoursMinutes(entry.startTime, entry.endTime, entry.breakMinutes)}
|
||||
{entry.breakMinutes > 0 && (
|
||||
<span className="text-xs text-gray-400 ml-1">(−{entry.breakMinutes}m break)</span>
|
||||
)}
|
||||
</td>
|
||||
<td className="px-4 py-3 whitespace-nowrap text-right">
|
||||
<button onClick={() => handleOpenModal(entry)} className="p-1.5 text-gray-400 hover:text-gray-600 mr-1"><Edit2 className="h-4 w-4" /></button>
|
||||
|
||||
@@ -1,4 +1,3 @@
|
||||
import { Clock } from 'lucide-react';
|
||||
import { useAuth } from '@/contexts/AuthContext';
|
||||
|
||||
export function LoginPage() {
|
||||
@@ -8,8 +7,8 @@ export function LoginPage() {
|
||||
<div className="min-h-screen flex items-center justify-center bg-gray-50">
|
||||
<div className="max-w-md w-full space-y-8 p-8">
|
||||
<div className="text-center">
|
||||
<div className="mx-auto h-16 w-16 bg-primary-100 rounded-full flex items-center justify-center">
|
||||
<Clock className="h-8 w-8 text-primary-600" />
|
||||
<div className="mx-auto h-16 w-16 flex items-center justify-center drop-shadow-sm">
|
||||
<img src="/icon.svg" alt="TimeTracker Logo" className="h-16 w-16" />
|
||||
</div>
|
||||
<h2 className="mt-6 text-3xl font-bold text-gray-900">TimeTracker</h2>
|
||||
<p className="mt-2 text-sm text-gray-600">
|
||||
|
||||
@@ -78,7 +78,10 @@ export function TimeEntriesPage() {
|
||||
</div>
|
||||
</td>
|
||||
<td className="px-4 py-3 whitespace-nowrap text-sm font-mono text-gray-900">
|
||||
{formatDurationFromDatesHoursMinutes(entry.startTime, entry.endTime)}
|
||||
{formatDurationFromDatesHoursMinutes(entry.startTime, entry.endTime, entry.breakMinutes)}
|
||||
{entry.breakMinutes > 0 && (
|
||||
<span className="text-xs text-gray-400 ml-1">(−{entry.breakMinutes}m)</span>
|
||||
)}
|
||||
</td>
|
||||
<td className="px-4 py-3 whitespace-nowrap text-right">
|
||||
<button onClick={() => handleOpenModal(entry)} className="p-1.5 text-gray-400 hover:text-gray-600 mr-1"><Edit2 className="h-4 w-4" /></button>
|
||||
|
||||
@@ -11,6 +11,7 @@ export interface Client {
|
||||
description: string | null;
|
||||
createdAt: string;
|
||||
updatedAt: string;
|
||||
deletedAt: string | null;
|
||||
}
|
||||
|
||||
export interface Project {
|
||||
@@ -22,12 +23,14 @@ export interface Project {
|
||||
client: Pick<Client, 'id' | 'name'>;
|
||||
createdAt: string;
|
||||
updatedAt: string;
|
||||
deletedAt: string | null;
|
||||
}
|
||||
|
||||
export interface TimeEntry {
|
||||
id: string;
|
||||
startTime: string;
|
||||
endTime: string;
|
||||
breakMinutes: number;
|
||||
description: string | null;
|
||||
projectId: string;
|
||||
project: Pick<Project, 'id' | 'name' | 'color'> & {
|
||||
@@ -35,6 +38,7 @@ export interface TimeEntry {
|
||||
};
|
||||
createdAt: string;
|
||||
updatedAt: string;
|
||||
deletedAt: string | null;
|
||||
}
|
||||
|
||||
export interface OngoingTimer {
|
||||
@@ -129,6 +133,7 @@ export interface UpdateProjectInput {
|
||||
export interface CreateTimeEntryInput {
|
||||
startTime: string;
|
||||
endTime: string;
|
||||
breakMinutes?: number;
|
||||
description?: string;
|
||||
projectId: string;
|
||||
}
|
||||
@@ -136,6 +141,7 @@ export interface CreateTimeEntryInput {
|
||||
export interface UpdateTimeEntryInput {
|
||||
startTime?: string;
|
||||
endTime?: string;
|
||||
breakMinutes?: number;
|
||||
description?: string;
|
||||
projectId?: string;
|
||||
}
|
||||
@@ -146,15 +152,22 @@ export interface BalanceCorrection {
|
||||
hours: number;
|
||||
description: string | null;
|
||||
createdAt: string;
|
||||
deletedAt: string | null;
|
||||
}
|
||||
|
||||
export interface WeekBalance {
|
||||
weekStart: string; // YYYY-MM-DD (Monday)
|
||||
weekEnd: string; // YYYY-MM-DD (Sunday)
|
||||
export interface PeriodBalance {
|
||||
periodStart: string; // YYYY-MM-DD
|
||||
periodEnd: string; // YYYY-MM-DD
|
||||
targetHours: number; // pro-rated for first period
|
||||
trackedSeconds: number;
|
||||
targetSeconds: number;
|
||||
correctionHours: number;
|
||||
balanceSeconds: number;
|
||||
isOngoing: boolean;
|
||||
// only present when isOngoing = true
|
||||
dailyRateHours?: number;
|
||||
workingDaysInPeriod?: number;
|
||||
elapsedWorkingDays?: number;
|
||||
expectedHours?: number;
|
||||
}
|
||||
|
||||
export interface ClientTargetWithBalance {
|
||||
@@ -162,26 +175,34 @@ export interface ClientTargetWithBalance {
|
||||
clientId: string;
|
||||
clientName: string;
|
||||
userId: string;
|
||||
weeklyHours: number;
|
||||
startDate: string; // YYYY-MM-DD
|
||||
periodType: "weekly" | "monthly";
|
||||
targetHours: number;
|
||||
workingDays: string[]; // e.g. ["MON","WED"]
|
||||
startDate: string; // YYYY-MM-DD
|
||||
createdAt: string;
|
||||
updatedAt: string;
|
||||
corrections: BalanceCorrection[];
|
||||
totalBalanceSeconds: number;
|
||||
currentWeekTrackedSeconds: number;
|
||||
currentWeekTargetSeconds: number;
|
||||
weeks: WeekBalance[];
|
||||
currentPeriodTrackedSeconds: number;
|
||||
currentPeriodTargetSeconds: number;
|
||||
periods: PeriodBalance[];
|
||||
/** True when an active timer for a project belonging to this client is running. */
|
||||
hasOngoingTimer: boolean;
|
||||
}
|
||||
|
||||
export interface CreateClientTargetInput {
|
||||
clientId: string;
|
||||
weeklyHours: number;
|
||||
startDate: string; // YYYY-MM-DD
|
||||
targetHours: number;
|
||||
periodType: "weekly" | "monthly";
|
||||
workingDays: string[]; // e.g. ["MON","WED","FRI"]
|
||||
startDate: string; // YYYY-MM-DD
|
||||
}
|
||||
|
||||
export interface UpdateClientTargetInput {
|
||||
weeklyHours?: number;
|
||||
startDate?: string;
|
||||
targetHours?: number;
|
||||
periodType?: "weekly" | "monthly";
|
||||
workingDays?: string[];
|
||||
startDate?: string; // YYYY-MM-DD
|
||||
}
|
||||
|
||||
export interface CreateCorrectionInput {
|
||||
|
||||
@@ -43,7 +43,14 @@ export function formatDurationHoursMinutes(totalSeconds: number): string {
|
||||
return `${hours}h ${minutes}m`;
|
||||
}
|
||||
|
||||
export function calculateDuration(startTime: string, endTime: string): number {
|
||||
export function calculateDuration(startTime: string, endTime: string, breakMinutes: number = 0): number {
|
||||
const start = parseISO(startTime);
|
||||
const end = parseISO(endTime);
|
||||
const totalSeconds = differenceInSeconds(end, start);
|
||||
return totalSeconds - (breakMinutes * 60);
|
||||
}
|
||||
|
||||
export function calculateGrossDuration(startTime: string, endTime: string): number {
|
||||
const start = parseISO(startTime);
|
||||
const end = parseISO(endTime);
|
||||
return differenceInSeconds(end, start);
|
||||
@@ -52,16 +59,18 @@ export function calculateDuration(startTime: string, endTime: string): number {
|
||||
export function formatDurationFromDates(
|
||||
startTime: string,
|
||||
endTime: string,
|
||||
breakMinutes: number = 0,
|
||||
): string {
|
||||
const seconds = calculateDuration(startTime, endTime);
|
||||
const seconds = calculateDuration(startTime, endTime, breakMinutes);
|
||||
return formatDuration(seconds);
|
||||
}
|
||||
|
||||
export function formatDurationFromDatesHoursMinutes(
|
||||
startTime: string,
|
||||
endTime: string,
|
||||
breakMinutes: number = 0,
|
||||
): string {
|
||||
const seconds = calculateDuration(startTime, endTime);
|
||||
const seconds = calculateDuration(startTime, endTime, breakMinutes);
|
||||
return formatDurationHoursMinutes(seconds);
|
||||
}
|
||||
|
||||
|
||||
@@ -25,6 +25,14 @@ enum APIEndpoint {
|
||||
static let timer = "/timer"
|
||||
static let timerStart = "/timer/start"
|
||||
static let timerStop = "/timer/stop"
|
||||
|
||||
// Client Targets
|
||||
static let clientTargets = "/client-targets"
|
||||
static func clientTarget(id: String) -> String { "/client-targets/\(id)" }
|
||||
static func clientTargetCorrections(targetId: String) -> String { "/client-targets/\(targetId)/corrections" }
|
||||
static func clientTargetCorrection(targetId: String, correctionId: String) -> String {
|
||||
"/client-targets/\(targetId)/corrections/\(correctionId)"
|
||||
}
|
||||
}
|
||||
|
||||
struct APIEndpoints {
|
||||
|
||||
@@ -1,134 +1,3 @@
|
||||
import SwiftUI
|
||||
|
||||
struct ClientsView: View {
|
||||
@StateObject private var viewModel = ClientsViewModel()
|
||||
@State private var showAddClient = false
|
||||
@State private var clientToDelete: Client?
|
||||
@State private var showDeleteConfirmation = false
|
||||
|
||||
var body: some View {
|
||||
NavigationStack {
|
||||
Group {
|
||||
if viewModel.isLoading && viewModel.clients.isEmpty {
|
||||
LoadingView()
|
||||
} else if let error = viewModel.error, viewModel.clients.isEmpty {
|
||||
ErrorView(message: error) {
|
||||
Task { await viewModel.loadClients() }
|
||||
}
|
||||
} else if viewModel.clients.isEmpty {
|
||||
EmptyView(
|
||||
icon: "person.2",
|
||||
title: "No Clients",
|
||||
message: "Create a client to organize your projects."
|
||||
)
|
||||
} else {
|
||||
clientsList
|
||||
}
|
||||
}
|
||||
.navigationTitle("Clients")
|
||||
.toolbar {
|
||||
ToolbarItem(placement: .primaryAction) {
|
||||
Button {
|
||||
showAddClient = true
|
||||
} label: {
|
||||
Image(systemName: "plus")
|
||||
}
|
||||
}
|
||||
}
|
||||
.task {
|
||||
await viewModel.loadClients()
|
||||
}
|
||||
.sheet(isPresented: $showAddClient) {
|
||||
ClientFormView(onSave: { name, description in
|
||||
Task {
|
||||
await viewModel.createClient(name: name, description: description)
|
||||
}
|
||||
})
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
private var clientsList: some View {
|
||||
List {
|
||||
ForEach(viewModel.clients) { client in
|
||||
ClientRow(client: client)
|
||||
}
|
||||
.onDelete { indexSet in
|
||||
if let index = indexSet.first {
|
||||
clientToDelete = viewModel.clients[index]
|
||||
showDeleteConfirmation = true
|
||||
}
|
||||
}
|
||||
}
|
||||
.alert("Delete Client?", isPresented: $showDeleteConfirmation, presenting: clientToDelete) { client in
|
||||
Button("Cancel", role: .cancel) {}
|
||||
Button("Delete", role: .destructive) {
|
||||
Task {
|
||||
await viewModel.deleteClient(client)
|
||||
}
|
||||
}
|
||||
} message: { client in
|
||||
Text("This will permanently delete '\(client.name)' and all related projects and time entries. This action cannot be undone.")
|
||||
}
|
||||
.refreshable {
|
||||
await viewModel.loadClients()
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
struct ClientRow: View {
|
||||
let client: Client
|
||||
|
||||
var body: some View {
|
||||
VStack(alignment: .leading, spacing: 4) {
|
||||
Text(client.name)
|
||||
.font(.headline)
|
||||
if let description = client.description {
|
||||
Text(description)
|
||||
.font(.subheadline)
|
||||
.foregroundStyle(.secondary)
|
||||
.lineLimit(2)
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
struct ClientFormView: View {
|
||||
@Environment(\.dismiss) private var dismiss
|
||||
|
||||
let onSave: (String, String?) -> Void
|
||||
|
||||
@State private var name = ""
|
||||
@State private var description = ""
|
||||
|
||||
var body: some View {
|
||||
NavigationStack {
|
||||
Form {
|
||||
Section("Name") {
|
||||
TextField("Client name", text: $name)
|
||||
}
|
||||
|
||||
Section("Description (Optional)") {
|
||||
TextField("Description", text: $description, axis: .vertical)
|
||||
.lineLimit(3...6)
|
||||
}
|
||||
}
|
||||
.navigationTitle("New Client")
|
||||
.navigationBarTitleDisplayMode(.inline)
|
||||
.toolbar {
|
||||
ToolbarItem(placement: .cancellationAction) {
|
||||
Button("Cancel") {
|
||||
dismiss()
|
||||
}
|
||||
}
|
||||
ToolbarItem(placement: .confirmationAction) {
|
||||
Button("Save") {
|
||||
onSave(name, description.isEmpty ? nil : description)
|
||||
dismiss()
|
||||
}
|
||||
.disabled(name.isEmpty)
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
// ClientsView.swift — replaced by Features/Settings/ClientsListView.swift
|
||||
// This file is intentionally empty; ClientsViewModel is no longer used directly.
|
||||
import Foundation
|
||||
|
||||
@@ -2,94 +2,82 @@ import SwiftUI
|
||||
|
||||
struct DashboardView: View {
|
||||
@StateObject private var viewModel = DashboardViewModel()
|
||||
|
||||
|
||||
var body: some View {
|
||||
NavigationStack {
|
||||
ScrollView {
|
||||
VStack(spacing: 24) {
|
||||
// Active Timer Card
|
||||
timerCard
|
||||
|
||||
// Weekly Stats
|
||||
if let stats = viewModel.statistics {
|
||||
statsSection(stats)
|
||||
}
|
||||
|
||||
// Recent Entries
|
||||
recentEntriesSection
|
||||
Group {
|
||||
if viewModel.isLoading && viewModel.statistics == nil && viewModel.recentEntries.isEmpty {
|
||||
LoadingView()
|
||||
} else {
|
||||
scrollContent
|
||||
}
|
||||
.padding()
|
||||
}
|
||||
.navigationTitle("Dashboard")
|
||||
.refreshable {
|
||||
await viewModel.loadData()
|
||||
}
|
||||
.task {
|
||||
await viewModel.loadData()
|
||||
}
|
||||
.refreshable { await viewModel.loadData() }
|
||||
.task { await viewModel.loadData() }
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
// MARK: - Main scroll content
|
||||
|
||||
private var scrollContent: some View {
|
||||
ScrollView {
|
||||
VStack(spacing: 24) {
|
||||
timerCard
|
||||
if let stats = viewModel.statistics { weeklyStatsSection(stats) }
|
||||
if !viewModel.clientTargets.isEmpty { workBalanceSection }
|
||||
recentEntriesSection
|
||||
}
|
||||
.padding()
|
||||
}
|
||||
}
|
||||
|
||||
// MARK: - Active Timer Card
|
||||
|
||||
private var timerCard: some View {
|
||||
VStack(spacing: 16) {
|
||||
if let timer = viewModel.activeTimer {
|
||||
HStack {
|
||||
VStack(alignment: .leading, spacing: 4) {
|
||||
Text("Timer Running")
|
||||
.font(.subheadline)
|
||||
.foregroundStyle(.secondary)
|
||||
|
||||
Text(viewModel.elapsedTime.formattedDuration)
|
||||
.font(.system(size: 32, weight: .medium, design: .monospaced))
|
||||
|
||||
if let project = timer.project {
|
||||
ProjectColorBadge(color: project.color, name: project.name)
|
||||
}
|
||||
}
|
||||
|
||||
Spacer()
|
||||
|
||||
Image(systemName: "timer")
|
||||
.font(.title)
|
||||
.foregroundStyle(.green)
|
||||
}
|
||||
} else {
|
||||
HStack {
|
||||
VStack(alignment: .leading, spacing: 4) {
|
||||
Text("No Active Timer")
|
||||
.font(.subheadline)
|
||||
.foregroundStyle(.secondary)
|
||||
|
||||
Text("Start tracking to see your time")
|
||||
.font(.headline)
|
||||
}
|
||||
|
||||
Spacer()
|
||||
|
||||
Image(systemName: "timer")
|
||||
.font(.title)
|
||||
HStack {
|
||||
VStack(alignment: .leading, spacing: 4) {
|
||||
if let timer = viewModel.activeTimer {
|
||||
Text("Timer Running")
|
||||
.font(.subheadline)
|
||||
.foregroundStyle(.secondary)
|
||||
Text(viewModel.elapsedTime.formattedDuration)
|
||||
.font(.system(size: 32, weight: .medium, design: .monospaced))
|
||||
if let project = timer.project {
|
||||
ProjectColorBadge(color: project.color, name: project.name)
|
||||
}
|
||||
} else {
|
||||
Text("No Active Timer")
|
||||
.font(.subheadline)
|
||||
.foregroundStyle(.secondary)
|
||||
Text("Start tracking to see your time")
|
||||
.font(.headline)
|
||||
}
|
||||
}
|
||||
Spacer()
|
||||
Image(systemName: "timer")
|
||||
.font(.title)
|
||||
.foregroundStyle(viewModel.activeTimer != nil ? .green : .secondary)
|
||||
}
|
||||
.padding()
|
||||
.background(Color(.systemGray6))
|
||||
.cornerRadius(12)
|
||||
}
|
||||
|
||||
private func statsSection(_ stats: TimeStatistics) -> some View {
|
||||
|
||||
// MARK: - Weekly Stats
|
||||
|
||||
private func weeklyStatsSection(_ stats: TimeStatistics) -> some View {
|
||||
VStack(alignment: .leading, spacing: 12) {
|
||||
Text("This Week")
|
||||
.font(.headline)
|
||||
|
||||
|
||||
HStack(spacing: 12) {
|
||||
StatCard(
|
||||
title: "Hours Tracked",
|
||||
value: TimeInterval(stats.totalSeconds).formattedHours,
|
||||
value: TimeInterval(stats.totalSeconds).formattedShortDuration,
|
||||
icon: "clock.fill",
|
||||
color: .blue
|
||||
)
|
||||
|
||||
StatCard(
|
||||
title: "Entries",
|
||||
value: "\(stats.entryCount)",
|
||||
@@ -97,13 +85,12 @@ struct DashboardView: View {
|
||||
color: .green
|
||||
)
|
||||
}
|
||||
|
||||
|
||||
if !stats.byProject.isEmpty {
|
||||
VStack(alignment: .leading, spacing: 8) {
|
||||
Text("By Project")
|
||||
.font(.subheadline)
|
||||
.foregroundStyle(.secondary)
|
||||
|
||||
ForEach(stats.byProject.prefix(5)) { projectStat in
|
||||
HStack {
|
||||
if let color = projectStat.projectColor {
|
||||
@@ -112,21 +99,39 @@ struct DashboardView: View {
|
||||
Text(projectStat.projectName)
|
||||
.font(.subheadline)
|
||||
Spacer()
|
||||
Text(TimeInterval(projectStat.totalSeconds).formattedHours)
|
||||
Text(TimeInterval(projectStat.totalSeconds).formattedShortDuration)
|
||||
.font(.subheadline)
|
||||
.foregroundStyle(.secondary)
|
||||
}
|
||||
}
|
||||
}
|
||||
.padding()
|
||||
.background(Color(.systemGray6))
|
||||
.cornerRadius(12)
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
// MARK: - Work Balance Section
|
||||
|
||||
private var workBalanceSection: some View {
|
||||
VStack(alignment: .leading, spacing: 12) {
|
||||
Text("Work Time Balance")
|
||||
.font(.headline)
|
||||
|
||||
ForEach(viewModel.clientTargets) { target in
|
||||
WorkBalanceCard(target: target)
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
// MARK: - Recent Entries
|
||||
|
||||
private var recentEntriesSection: some View {
|
||||
VStack(alignment: .leading, spacing: 12) {
|
||||
Text("Recent Entries")
|
||||
.font(.headline)
|
||||
|
||||
|
||||
if viewModel.recentEntries.isEmpty {
|
||||
Text("No entries yet")
|
||||
.font(.subheadline)
|
||||
@@ -134,22 +139,27 @@ struct DashboardView: View {
|
||||
.frame(maxWidth: .infinity, alignment: .center)
|
||||
.padding()
|
||||
} else {
|
||||
ForEach(viewModel.recentEntries) { entry in
|
||||
HStack {
|
||||
ProjectColorDot(color: entry.project.color, size: 10)
|
||||
VStack(alignment: .leading, spacing: 2) {
|
||||
Text(entry.project.name)
|
||||
VStack(spacing: 0) {
|
||||
ForEach(Array(viewModel.recentEntries.enumerated()), id: \.element.id) { index, entry in
|
||||
HStack {
|
||||
ProjectColorDot(color: entry.project.color, size: 10)
|
||||
VStack(alignment: .leading, spacing: 2) {
|
||||
Text(entry.project.name)
|
||||
.font(.subheadline)
|
||||
Text(formatDate(entry.startTime))
|
||||
.font(.caption)
|
||||
.foregroundStyle(.secondary)
|
||||
}
|
||||
Spacer()
|
||||
Text(entry.duration.formattedShortDuration)
|
||||
.font(.subheadline)
|
||||
Text(formatDate(entry.startTime))
|
||||
.font(.caption)
|
||||
.foregroundStyle(.secondary)
|
||||
}
|
||||
Spacer()
|
||||
Text(entry.duration.formattedHours)
|
||||
.font(.subheadline)
|
||||
.foregroundStyle(.secondary)
|
||||
.padding(.vertical, 8)
|
||||
if index < viewModel.recentEntries.count - 1 {
|
||||
Divider()
|
||||
}
|
||||
}
|
||||
.padding(.vertical, 4)
|
||||
}
|
||||
}
|
||||
}
|
||||
@@ -157,9 +167,144 @@ struct DashboardView: View {
|
||||
.background(Color(.systemGray6))
|
||||
.cornerRadius(12)
|
||||
}
|
||||
|
||||
|
||||
private func formatDate(_ isoString: String) -> String {
|
||||
guard let date = Date.fromISO8601(isoString) else { return "" }
|
||||
return date.formattedDateTime()
|
||||
}
|
||||
}
|
||||
|
||||
// MARK: - Work Balance Card
|
||||
|
||||
struct WorkBalanceCard: View {
|
||||
let target: ClientTarget
|
||||
@State private var expanded = false
|
||||
|
||||
private var totalBalance: TimeInterval { TimeInterval(target.totalBalanceSeconds) }
|
||||
private var currentWeekTracked: TimeInterval { TimeInterval(target.currentWeekTrackedSeconds) }
|
||||
private var currentWeekTarget: TimeInterval { TimeInterval(target.currentWeekTargetSeconds) }
|
||||
|
||||
private var balanceColor: Color {
|
||||
if target.totalBalanceSeconds >= 0 { return .green }
|
||||
return .red
|
||||
}
|
||||
|
||||
private var balanceLabel: String {
|
||||
let abs = abs(totalBalance)
|
||||
return target.totalBalanceSeconds >= 0
|
||||
? "+\(abs.formattedShortDuration) overtime"
|
||||
: "-\(abs.formattedShortDuration) undertime"
|
||||
}
|
||||
|
||||
var body: some View {
|
||||
VStack(alignment: .leading, spacing: 12) {
|
||||
// Header row
|
||||
HStack {
|
||||
Text(target.clientName)
|
||||
.font(.subheadline)
|
||||
.fontWeight(.semibold)
|
||||
Spacer()
|
||||
Text(balanceLabel)
|
||||
.font(.subheadline)
|
||||
.fontWeight(.medium)
|
||||
.foregroundStyle(balanceColor)
|
||||
}
|
||||
|
||||
// This-week progress
|
||||
VStack(alignment: .leading, spacing: 4) {
|
||||
HStack {
|
||||
Text("This week")
|
||||
.font(.caption)
|
||||
.foregroundStyle(.secondary)
|
||||
Spacer()
|
||||
Text("\(currentWeekTracked.formattedShortDuration) / \(currentWeekTarget.formattedShortDuration)")
|
||||
.font(.caption)
|
||||
.foregroundStyle(.secondary)
|
||||
}
|
||||
if currentWeekTarget > 0 {
|
||||
ProgressView(value: min(currentWeekTracked / currentWeekTarget, 1.0))
|
||||
.tint(currentWeekTracked >= currentWeekTarget ? .green : .blue)
|
||||
}
|
||||
}
|
||||
|
||||
// Weekly breakdown (expandable)
|
||||
if !target.weeks.isEmpty {
|
||||
Button {
|
||||
withAnimation(.easeInOut(duration: 0.2)) { expanded.toggle() }
|
||||
} label: {
|
||||
HStack(spacing: 4) {
|
||||
Text(expanded ? "Hide weeks" : "Show weeks")
|
||||
.font(.caption)
|
||||
Image(systemName: expanded ? "chevron.up" : "chevron.down")
|
||||
.font(.caption2)
|
||||
}
|
||||
.foregroundStyle(.secondary)
|
||||
}
|
||||
.buttonStyle(.plain)
|
||||
|
||||
if expanded {
|
||||
VStack(spacing: 6) {
|
||||
ForEach(target.weeks.suffix(8).reversed()) { week in
|
||||
WeekBalanceRow(week: week)
|
||||
}
|
||||
}
|
||||
.transition(.opacity.combined(with: .move(edge: .top)))
|
||||
}
|
||||
}
|
||||
}
|
||||
.padding()
|
||||
.background(Color(.systemGray6))
|
||||
.cornerRadius(12)
|
||||
}
|
||||
}
|
||||
|
||||
// MARK: - Week Balance Row
|
||||
|
||||
struct WeekBalanceRow: View {
|
||||
let week: WeekBalance
|
||||
|
||||
private var balance: TimeInterval { TimeInterval(week.balanceSeconds) }
|
||||
private var tracked: TimeInterval { TimeInterval(week.trackedSeconds) }
|
||||
private var target: TimeInterval { TimeInterval(week.targetSeconds) }
|
||||
private var balanceColor: Color { week.balanceSeconds >= 0 ? .green : .red }
|
||||
|
||||
var body: some View {
|
||||
HStack {
|
||||
Text(weekLabel)
|
||||
.font(.caption)
|
||||
.foregroundStyle(.secondary)
|
||||
Spacer()
|
||||
Text(tracked.formattedShortDuration)
|
||||
.font(.caption)
|
||||
Text("/")
|
||||
.font(.caption)
|
||||
.foregroundStyle(.secondary)
|
||||
Text(target.formattedShortDuration)
|
||||
.font(.caption)
|
||||
.foregroundStyle(.secondary)
|
||||
Text(balanceText)
|
||||
.font(.caption)
|
||||
.fontWeight(.medium)
|
||||
.foregroundStyle(balanceColor)
|
||||
.frame(width: 70, alignment: .trailing)
|
||||
}
|
||||
}
|
||||
|
||||
private var weekLabel: String {
|
||||
guard let date = parseDate(week.weekStart) else { return week.weekStart }
|
||||
let formatter = DateFormatter()
|
||||
formatter.dateFormat = "MMM d"
|
||||
return formatter.string(from: date)
|
||||
}
|
||||
|
||||
private var balanceText: String {
|
||||
let abs = Swift.abs(balance)
|
||||
return week.balanceSeconds >= 0 ? "+\(abs.formattedShortDuration)" : "-\(abs.formattedShortDuration)"
|
||||
}
|
||||
|
||||
private func parseDate(_ string: String) -> Date? {
|
||||
let f = DateFormatter()
|
||||
f.dateFormat = "yyyy-MM-dd"
|
||||
return f.date(from: string)
|
||||
}
|
||||
}
|
||||
|
||||
@@ -6,40 +6,42 @@ final class DashboardViewModel: ObservableObject {
|
||||
@Published var activeTimer: OngoingTimer?
|
||||
@Published var statistics: TimeStatistics?
|
||||
@Published var recentEntries: [TimeEntry] = []
|
||||
@Published var clientTargets: [ClientTarget] = []
|
||||
@Published var isLoading = false
|
||||
@Published var error: String?
|
||||
@Published var elapsedTime: TimeInterval = 0
|
||||
|
||||
|
||||
private let apiClient = APIClient()
|
||||
private let database = DatabaseService.shared
|
||||
private var timerTask: Task<Void, Never>?
|
||||
|
||||
|
||||
init() {
|
||||
startElapsedTimeUpdater()
|
||||
}
|
||||
|
||||
|
||||
deinit {
|
||||
timerTask?.cancel()
|
||||
}
|
||||
|
||||
|
||||
func loadData() async {
|
||||
isLoading = true
|
||||
error = nil
|
||||
|
||||
|
||||
do {
|
||||
// Fetch active timer
|
||||
activeTimer = try await apiClient.request(
|
||||
endpoint: APIEndpoint.timer,
|
||||
authenticated: true
|
||||
)
|
||||
|
||||
// Get statistics for this week
|
||||
|
||||
// Statistics for this week
|
||||
let calendar = Calendar.current
|
||||
let today = Date()
|
||||
let startOfWeek = calendar.date(from: calendar.dateComponents([.yearForWeekOfYear, .weekOfYear], from: today))!
|
||||
let startOfWeek = calendar.date(
|
||||
from: calendar.dateComponents([.yearForWeekOfYear, .weekOfYear], from: today)
|
||||
)!
|
||||
let endOfWeek = calendar.date(byAdding: .day, value: 6, to: startOfWeek)!
|
||||
|
||||
let statsInput = StatisticsFiltersInput(startDate: startOfWeek, endDate: endOfWeek)
|
||||
|
||||
statistics = try await apiClient.request(
|
||||
endpoint: APIEndpoint.timeEntriesStatistics,
|
||||
queryItems: [
|
||||
@@ -48,41 +50,43 @@ final class DashboardViewModel: ObservableObject {
|
||||
],
|
||||
authenticated: true
|
||||
)
|
||||
|
||||
// Fetch recent entries
|
||||
|
||||
// Recent entries (last 5)
|
||||
let entriesResponse: TimeEntryListResponse = try await apiClient.request(
|
||||
endpoint: APIEndpoint.timeEntries,
|
||||
queryItems: [
|
||||
URLQueryItem(name: "limit", value: "5")
|
||||
],
|
||||
queryItems: [URLQueryItem(name: "limit", value: "5")],
|
||||
authenticated: true
|
||||
)
|
||||
recentEntries = entriesResponse.entries
|
||||
|
||||
|
||||
// Client targets (for overtime/undertime)
|
||||
clientTargets = try await apiClient.request(
|
||||
endpoint: APIEndpoint.clientTargets,
|
||||
authenticated: true
|
||||
)
|
||||
|
||||
if let timer = activeTimer {
|
||||
elapsedTime = timer.elapsedTime
|
||||
}
|
||||
|
||||
|
||||
isLoading = false
|
||||
} catch {
|
||||
isLoading = false
|
||||
self.error = error.localizedDescription
|
||||
|
||||
// Try to load cached data
|
||||
|
||||
// Fallback to cached timer
|
||||
if let cachedTimer = try? await database.getCachedTimer() {
|
||||
activeTimer = cachedTimer
|
||||
elapsedTime = cachedTimer.elapsedTime
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
private func startElapsedTimeUpdater() {
|
||||
timerTask = Task { [weak self] in
|
||||
while !Task.isCancelled {
|
||||
try? await Task.sleep(nanoseconds: 1_000_000_000)
|
||||
|
||||
guard let self = self, self.activeTimer != nil else { continue }
|
||||
|
||||
await MainActor.run {
|
||||
self.elapsedTime = self.activeTimer?.elapsedTime ?? 0
|
||||
}
|
||||
|
||||
@@ -1,178 +1,3 @@
|
||||
import SwiftUI
|
||||
|
||||
struct ProjectsView: View {
|
||||
@StateObject private var viewModel = ProjectsViewModel()
|
||||
@State private var showAddProject = false
|
||||
@State private var projectToDelete: Project?
|
||||
@State private var showDeleteConfirmation = false
|
||||
|
||||
var body: some View {
|
||||
NavigationStack {
|
||||
Group {
|
||||
if viewModel.isLoading && viewModel.projects.isEmpty {
|
||||
LoadingView()
|
||||
} else if let error = viewModel.error, viewModel.projects.isEmpty {
|
||||
ErrorView(message: error) {
|
||||
Task { await viewModel.loadData() }
|
||||
}
|
||||
} else if viewModel.projects.isEmpty {
|
||||
EmptyView(
|
||||
icon: "folder",
|
||||
title: "No Projects",
|
||||
message: "Create a project to start tracking time."
|
||||
)
|
||||
} else {
|
||||
projectsList
|
||||
}
|
||||
}
|
||||
.navigationTitle("Projects")
|
||||
.toolbar {
|
||||
ToolbarItem(placement: .primaryAction) {
|
||||
Button {
|
||||
showAddProject = true
|
||||
} label: {
|
||||
Image(systemName: "plus")
|
||||
}
|
||||
}
|
||||
}
|
||||
.task {
|
||||
await viewModel.loadData()
|
||||
}
|
||||
.sheet(isPresented: $showAddProject) {
|
||||
ProjectFormView(
|
||||
clients: viewModel.clients,
|
||||
onSave: { name, description, color, clientId in
|
||||
Task {
|
||||
await viewModel.createProject(
|
||||
name: name,
|
||||
description: description,
|
||||
color: color,
|
||||
clientId: clientId
|
||||
)
|
||||
}
|
||||
}
|
||||
)
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
private var projectsList: some View {
|
||||
List {
|
||||
ForEach(viewModel.projects) { project in
|
||||
ProjectRow(project: project)
|
||||
}
|
||||
.onDelete { indexSet in
|
||||
if let index = indexSet.first {
|
||||
projectToDelete = viewModel.projects[index]
|
||||
showDeleteConfirmation = true
|
||||
}
|
||||
}
|
||||
}
|
||||
.alert("Delete Project?", isPresented: $showDeleteConfirmation, presenting: projectToDelete) { project in
|
||||
Button("Cancel", role: .cancel) {}
|
||||
Button("Delete", role: .destructive) {
|
||||
Task {
|
||||
await viewModel.deleteProject(project)
|
||||
}
|
||||
}
|
||||
} message: { project in
|
||||
Text("This will permanently delete '\(project.name)' and all related time entries. This action cannot be undone.")
|
||||
}
|
||||
.refreshable {
|
||||
await viewModel.loadData()
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
struct ProjectRow: View {
|
||||
let project: Project
|
||||
|
||||
var body: some View {
|
||||
HStack {
|
||||
ProjectColorDot(color: project.color, size: 16)
|
||||
|
||||
VStack(alignment: .leading, spacing: 2) {
|
||||
Text(project.name)
|
||||
.font(.headline)
|
||||
Text(project.client.name)
|
||||
.font(.subheadline)
|
||||
.foregroundStyle(.secondary)
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
struct ProjectFormView: View {
|
||||
@Environment(\.dismiss) private var dismiss
|
||||
|
||||
let clients: [Client]
|
||||
let onSave: (String, String?, String?, String) -> Void
|
||||
|
||||
@State private var name = ""
|
||||
@State private var description = ""
|
||||
@State private var selectedColor: String = "#3B82F6"
|
||||
@State private var selectedClient: Client?
|
||||
|
||||
private let colors = ["#EF4444", "#F97316", "#EAB308", "#22C55E", "#14B8A6",
|
||||
"#06B6D4", "#3B82F6", "#6366F1", "#A855F7", "#EC4899"]
|
||||
|
||||
var body: some View {
|
||||
NavigationStack {
|
||||
Form {
|
||||
Section("Name") {
|
||||
TextField("Project name", text: $name)
|
||||
}
|
||||
|
||||
Section("Description (Optional)") {
|
||||
TextField("Description", text: $description, axis: .vertical)
|
||||
.lineLimit(3...6)
|
||||
}
|
||||
|
||||
Section("Color") {
|
||||
LazyVGrid(columns: Array(repeating: GridItem(.flexible()), count: 5), spacing: 16) {
|
||||
ForEach(colors, id: \.self) { color in
|
||||
Circle()
|
||||
.fill(Color(hex: color))
|
||||
.frame(width: 44, height: 44)
|
||||
.overlay(
|
||||
Circle()
|
||||
.strokeBorder(Color.primary, lineWidth: selectedColor == color ? 3 : 0)
|
||||
)
|
||||
.onTapGesture {
|
||||
selectedColor = color
|
||||
}
|
||||
}
|
||||
}
|
||||
.padding(.vertical, 8)
|
||||
}
|
||||
|
||||
Section("Client") {
|
||||
Picker("Client", selection: $selectedClient) {
|
||||
Text("Select Client").tag(nil as Client?)
|
||||
ForEach(clients) { client in
|
||||
Text(client.name)
|
||||
.tag(client as Client?)
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
.navigationTitle("New Project")
|
||||
.navigationBarTitleDisplayMode(.inline)
|
||||
.toolbar {
|
||||
ToolbarItem(placement: .cancellationAction) {
|
||||
Button("Cancel") {
|
||||
dismiss()
|
||||
}
|
||||
}
|
||||
ToolbarItem(placement: .confirmationAction) {
|
||||
Button("Save") {
|
||||
guard let client = selectedClient else { return }
|
||||
onSave(name, description.isEmpty ? nil : description, selectedColor, client.id)
|
||||
dismiss()
|
||||
}
|
||||
.disabled(name.isEmpty || selectedClient == nil)
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
// ProjectsView.swift — replaced by Features/Settings/ProjectsListView.swift
|
||||
// This file is intentionally empty; ProjectsViewModel is no longer used directly.
|
||||
import Foundation
|
||||
|
||||
@@ -0,0 +1,608 @@
|
||||
import SwiftUI
|
||||
|
||||
// MARK: - Clients List
|
||||
|
||||
struct ClientsListView: View {
|
||||
@State private var clients: [Client] = []
|
||||
@State private var isLoading = false
|
||||
@State private var error: String?
|
||||
@State private var showAddClient = false
|
||||
@State private var clientToDelete: Client?
|
||||
@State private var showDeleteConfirmation = false
|
||||
|
||||
private let apiClient = APIClient()
|
||||
|
||||
var body: some View {
|
||||
Group {
|
||||
if isLoading && clients.isEmpty {
|
||||
LoadingView()
|
||||
} else if let err = error, clients.isEmpty {
|
||||
ErrorView(message: err) { Task { await loadClients() } }
|
||||
} else if clients.isEmpty {
|
||||
EmptyView(icon: "person.2", title: "No Clients",
|
||||
message: "Create a client to organise your projects.")
|
||||
} else {
|
||||
List {
|
||||
ForEach(clients) { client in
|
||||
NavigationLink {
|
||||
ClientDetailView(client: client, onUpdate: { Task { await loadClients() } })
|
||||
} label: {
|
||||
ClientRow(client: client)
|
||||
}
|
||||
}
|
||||
.onDelete { indexSet in
|
||||
if let i = indexSet.first {
|
||||
clientToDelete = clients[i]
|
||||
showDeleteConfirmation = true
|
||||
}
|
||||
}
|
||||
}
|
||||
.refreshable { await loadClients() }
|
||||
}
|
||||
}
|
||||
.navigationTitle("Clients")
|
||||
.navigationBarTitleDisplayMode(.large)
|
||||
.toolbar {
|
||||
ToolbarItem(placement: .primaryAction) {
|
||||
Button { showAddClient = true } label: { Image(systemName: "plus") }
|
||||
}
|
||||
}
|
||||
.task { await loadClients() }
|
||||
.sheet(isPresented: $showAddClient) {
|
||||
ClientFormSheet(mode: .create) { Task { await loadClients() } }
|
||||
}
|
||||
.alert("Delete Client?", isPresented: $showDeleteConfirmation, presenting: clientToDelete) { client in
|
||||
Button("Cancel", role: .cancel) {}
|
||||
Button("Delete", role: .destructive) { Task { await deleteClient(client) } }
|
||||
} message: { client in
|
||||
Text("Deleting '\(client.name)' will also delete all its projects and time entries. This cannot be undone.")
|
||||
}
|
||||
}
|
||||
|
||||
private func loadClients() async {
|
||||
isLoading = true; error = nil
|
||||
do {
|
||||
clients = try await apiClient.request(endpoint: APIEndpoint.clients, authenticated: true)
|
||||
} catch { self.error = error.localizedDescription }
|
||||
isLoading = false
|
||||
}
|
||||
|
||||
private func deleteClient(_ client: Client) async {
|
||||
do {
|
||||
try await apiClient.requestVoid(endpoint: APIEndpoint.client(id: client.id),
|
||||
method: .delete, authenticated: true)
|
||||
clients.removeAll { $0.id == client.id }
|
||||
} catch { self.error = error.localizedDescription }
|
||||
}
|
||||
}
|
||||
|
||||
struct ClientRow: View {
|
||||
let client: Client
|
||||
var body: some View {
|
||||
VStack(alignment: .leading, spacing: 3) {
|
||||
Text(client.name).font(.headline)
|
||||
if let desc = client.description {
|
||||
Text(desc).font(.subheadline).foregroundStyle(.secondary).lineLimit(2)
|
||||
}
|
||||
}
|
||||
.padding(.vertical, 2)
|
||||
}
|
||||
}
|
||||
|
||||
// MARK: - Client Detail / Edit + Target Management
|
||||
|
||||
struct ClientDetailView: View {
|
||||
let client: Client
|
||||
let onUpdate: () -> Void
|
||||
|
||||
// Edit client fields
|
||||
@State private var name: String
|
||||
@State private var clientDescription: String
|
||||
@State private var isSavingClient = false
|
||||
@State private var clientSaveError: String?
|
||||
@State private var clientSaveSuccess = false
|
||||
|
||||
// Client targets
|
||||
@State private var target: ClientTarget?
|
||||
@State private var isLoadingTarget = false
|
||||
@State private var targetError: String?
|
||||
|
||||
// New target form
|
||||
@State private var showNewTargetForm = false
|
||||
@State private var newWeeklyHours = 40.0
|
||||
@State private var newStartDate = Date().nextMonday()
|
||||
@State private var isSavingTarget = false
|
||||
|
||||
// Edit target inline
|
||||
@State private var editingWeeklyHours: Double?
|
||||
@State private var editingStartDate: Date?
|
||||
@State private var isEditingTarget = false
|
||||
|
||||
// Balance correction
|
||||
@State private var showAddCorrection = false
|
||||
@State private var correctionDate = Date()
|
||||
@State private var correctionHours = 0.0
|
||||
@State private var correctionDescription = ""
|
||||
@State private var isSavingCorrection = false
|
||||
@State private var correctionToDelete: BalanceCorrection?
|
||||
@State private var showDeleteCorrection = false
|
||||
|
||||
private let apiClient = APIClient()
|
||||
|
||||
init(client: Client, onUpdate: @escaping () -> Void) {
|
||||
self.client = client
|
||||
self.onUpdate = onUpdate
|
||||
_name = State(initialValue: client.name)
|
||||
_clientDescription = State(initialValue: client.description ?? "")
|
||||
}
|
||||
|
||||
var body: some View {
|
||||
Form {
|
||||
clientEditSection
|
||||
targetSection
|
||||
if let target { correctionsSection(target) }
|
||||
}
|
||||
.navigationTitle(client.name)
|
||||
.navigationBarTitleDisplayMode(.inline)
|
||||
.task { await loadTarget() }
|
||||
.sheet(isPresented: $showAddCorrection) {
|
||||
addCorrectionSheet
|
||||
}
|
||||
}
|
||||
|
||||
// MARK: - Client edit
|
||||
|
||||
private var clientEditSection: some View {
|
||||
Section("Client Details") {
|
||||
TextField("Name", text: $name)
|
||||
TextField("Description (optional)", text: $clientDescription, axis: .vertical)
|
||||
.lineLimit(2...4)
|
||||
|
||||
if let err = clientSaveError {
|
||||
Text(err).font(.caption).foregroundStyle(.red)
|
||||
}
|
||||
if clientSaveSuccess {
|
||||
Label("Saved", systemImage: "checkmark.circle").foregroundStyle(.green).font(.caption)
|
||||
}
|
||||
|
||||
Button(isSavingClient ? "Saving…" : "Save Client Details") {
|
||||
Task { await saveClient() }
|
||||
}
|
||||
.disabled(name.isEmpty || isSavingClient)
|
||||
}
|
||||
}
|
||||
|
||||
private func saveClient() async {
|
||||
isSavingClient = true; clientSaveError = nil; clientSaveSuccess = false
|
||||
do {
|
||||
let input = UpdateClientInput(
|
||||
name: name,
|
||||
description: clientDescription.isEmpty ? nil : clientDescription
|
||||
)
|
||||
let _: Client = try await apiClient.request(
|
||||
endpoint: APIEndpoint.client(id: client.id),
|
||||
method: .put,
|
||||
body: input,
|
||||
authenticated: true
|
||||
)
|
||||
clientSaveSuccess = true
|
||||
onUpdate()
|
||||
} catch { clientSaveError = error.localizedDescription }
|
||||
isSavingClient = false
|
||||
}
|
||||
|
||||
// MARK: - Target section
|
||||
|
||||
private var targetSection: some View {
|
||||
Section {
|
||||
if isLoadingTarget {
|
||||
HStack { Spacer(); ProgressView(); Spacer() }
|
||||
} else if let err = targetError {
|
||||
Text(err).font(.caption).foregroundStyle(.red)
|
||||
} else if let target {
|
||||
// Show existing target + balance
|
||||
targetSummaryRows(target)
|
||||
if isEditingTarget {
|
||||
targetEditRows(target)
|
||||
} else {
|
||||
Button("Edit Target") { startEditingTarget(target) }
|
||||
}
|
||||
} else {
|
||||
// No target yet
|
||||
if showNewTargetForm {
|
||||
newTargetFormRows
|
||||
} else {
|
||||
Button("Set Up Work Time Target") { showNewTargetForm = true }
|
||||
}
|
||||
}
|
||||
} header: {
|
||||
Text("Work Time Target")
|
||||
}
|
||||
}
|
||||
|
||||
private func targetSummaryRows(_ t: ClientTarget) -> some View {
|
||||
Group {
|
||||
HStack {
|
||||
Text("Weekly hours")
|
||||
Spacer()
|
||||
Text("\(t.weeklyHours, specifier: "%.1f") h/week")
|
||||
.foregroundStyle(.secondary)
|
||||
}
|
||||
HStack {
|
||||
Text("Tracking since")
|
||||
Spacer()
|
||||
Text(formatDate(t.startDate))
|
||||
.foregroundStyle(.secondary)
|
||||
}
|
||||
HStack {
|
||||
Text("This week")
|
||||
Spacer()
|
||||
Text("\(TimeInterval(t.currentWeekTrackedSeconds).formattedShortDuration) / \(TimeInterval(t.currentWeekTargetSeconds).formattedShortDuration)")
|
||||
.foregroundStyle(.secondary)
|
||||
}
|
||||
HStack {
|
||||
Text("Total balance")
|
||||
Spacer()
|
||||
let balance = TimeInterval(abs(t.totalBalanceSeconds))
|
||||
Text(t.totalBalanceSeconds >= 0 ? "+\(balance.formattedShortDuration)" : "-\(balance.formattedShortDuration)")
|
||||
.fontWeight(.medium)
|
||||
.foregroundStyle(t.totalBalanceSeconds >= 0 ? .green : .red)
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
private func targetEditRows(_ t: ClientTarget) -> some View {
|
||||
Group {
|
||||
HStack {
|
||||
Text("Weekly hours")
|
||||
Spacer()
|
||||
TextField("Hours", value: Binding(
|
||||
get: { editingWeeklyHours ?? t.weeklyHours },
|
||||
set: { editingWeeklyHours = $0 }
|
||||
), format: .number)
|
||||
.keyboardType(.decimalPad)
|
||||
.multilineTextAlignment(.trailing)
|
||||
.frame(width: 80)
|
||||
}
|
||||
|
||||
DatePicker("Start date (Monday)",
|
||||
selection: Binding(
|
||||
get: { editingStartDate ?? parseDate(t.startDate) ?? Date() },
|
||||
set: { editingStartDate = $0 }
|
||||
),
|
||||
displayedComponents: .date)
|
||||
|
||||
HStack {
|
||||
Button("Cancel") { isEditingTarget = false; editingWeeklyHours = nil; editingStartDate = nil }
|
||||
.foregroundStyle(.secondary)
|
||||
Spacer()
|
||||
Button(isSavingTarget ? "Saving…" : "Save Target") {
|
||||
Task { await saveTarget(existingId: t.id) }
|
||||
}
|
||||
.disabled(isSavingTarget)
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
private var newTargetFormRows: some View {
|
||||
Group {
|
||||
HStack {
|
||||
Text("Weekly hours")
|
||||
Spacer()
|
||||
TextField("Hours", value: $newWeeklyHours, format: .number)
|
||||
.keyboardType(.decimalPad)
|
||||
.multilineTextAlignment(.trailing)
|
||||
.frame(width: 80)
|
||||
}
|
||||
DatePicker("Start date (Monday)", selection: $newStartDate, displayedComponents: .date)
|
||||
|
||||
HStack {
|
||||
Button("Cancel") { showNewTargetForm = false }
|
||||
.foregroundStyle(.secondary)
|
||||
Spacer()
|
||||
Button(isSavingTarget ? "Saving…" : "Create Target") {
|
||||
Task { await createTarget() }
|
||||
}
|
||||
.disabled(newWeeklyHours <= 0 || isSavingTarget)
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
private func startEditingTarget(_ t: ClientTarget) {
|
||||
editingWeeklyHours = t.weeklyHours
|
||||
editingStartDate = parseDate(t.startDate)
|
||||
isEditingTarget = true
|
||||
}
|
||||
|
||||
// MARK: - Corrections section
|
||||
|
||||
private func correctionsSection(_ t: ClientTarget) -> some View {
|
||||
Section {
|
||||
if t.corrections.isEmpty {
|
||||
Text("No corrections")
|
||||
.foregroundStyle(.secondary)
|
||||
.font(.subheadline)
|
||||
} else {
|
||||
ForEach(t.corrections) { correction in
|
||||
HStack {
|
||||
VStack(alignment: .leading, spacing: 2) {
|
||||
Text(formatDate(correction.date))
|
||||
.font(.subheadline)
|
||||
if let desc = correction.description {
|
||||
Text(desc).font(.caption).foregroundStyle(.secondary)
|
||||
}
|
||||
}
|
||||
Spacer()
|
||||
Text(correction.hours >= 0 ? "+\(correction.hours, specifier: "%.1f")h" : "\(correction.hours, specifier: "%.1f")h")
|
||||
.font(.subheadline)
|
||||
.fontWeight(.medium)
|
||||
.foregroundStyle(correction.hours >= 0 ? .green : .red)
|
||||
}
|
||||
}
|
||||
.onDelete { indexSet in
|
||||
if let i = indexSet.first {
|
||||
correctionToDelete = t.corrections[i]
|
||||
showDeleteCorrection = true
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
Button("Add Correction") { showAddCorrection = true }
|
||||
} header: {
|
||||
Text("Balance Corrections")
|
||||
}
|
||||
.alert("Delete Correction?", isPresented: $showDeleteCorrection, presenting: correctionToDelete) { correction in
|
||||
Button("Cancel", role: .cancel) {}
|
||||
Button("Delete", role: .destructive) {
|
||||
Task { await deleteCorrection(correction) }
|
||||
}
|
||||
} message: { correction in
|
||||
Text("Remove the \(correction.hours >= 0 ? "+" : "")\(correction.hours, specifier: "%.1f")h correction on \(formatDate(correction.date))?")
|
||||
}
|
||||
}
|
||||
|
||||
// MARK: - Add correction sheet
|
||||
|
||||
private var addCorrectionSheet: some View {
|
||||
NavigationStack {
|
||||
Form {
|
||||
Section("Date") {
|
||||
DatePicker("Date", selection: $correctionDate, displayedComponents: .date)
|
||||
}
|
||||
Section("Hours adjustment") {
|
||||
HStack {
|
||||
TextField("Hours (positive = bonus, negative = penalty)",
|
||||
value: $correctionHours, format: .number)
|
||||
.keyboardType(.numbersAndPunctuation)
|
||||
Text("h").foregroundStyle(.secondary)
|
||||
}
|
||||
Text("Positive values reduce the weekly target; negative values increase it.")
|
||||
.font(.caption)
|
||||
.foregroundStyle(.secondary)
|
||||
}
|
||||
Section("Description (optional)") {
|
||||
TextField("Note", text: $correctionDescription)
|
||||
}
|
||||
}
|
||||
.navigationTitle("Add Correction")
|
||||
.navigationBarTitleDisplayMode(.inline)
|
||||
.toolbar {
|
||||
ToolbarItem(placement: .cancellationAction) {
|
||||
Button("Cancel") { showAddCorrection = false }
|
||||
}
|
||||
ToolbarItem(placement: .confirmationAction) {
|
||||
Button(isSavingCorrection ? "Saving…" : "Add") {
|
||||
Task { await addCorrection() }
|
||||
}
|
||||
.disabled(correctionHours == 0 || isSavingCorrection)
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
// MARK: - API calls
|
||||
|
||||
private func loadTarget() async {
|
||||
isLoadingTarget = true; targetError = nil
|
||||
do {
|
||||
let allTargets: [ClientTarget] = try await apiClient.request(
|
||||
endpoint: APIEndpoint.clientTargets, authenticated: true
|
||||
)
|
||||
target = allTargets.first { $0.clientId == client.id }
|
||||
} catch { targetError = error.localizedDescription }
|
||||
isLoadingTarget = false
|
||||
}
|
||||
|
||||
private func createTarget() async {
|
||||
isSavingTarget = true
|
||||
do {
|
||||
let input = CreateClientTargetInput(
|
||||
clientId: client.id,
|
||||
weeklyHours: newWeeklyHours,
|
||||
startDate: newStartDate.iso8601FullDate
|
||||
)
|
||||
let created: ClientTarget = try await apiClient.request(
|
||||
endpoint: APIEndpoint.clientTargets,
|
||||
method: .post,
|
||||
body: input,
|
||||
authenticated: true
|
||||
)
|
||||
target = created
|
||||
showNewTargetForm = false
|
||||
} catch { targetError = error.localizedDescription }
|
||||
isSavingTarget = false
|
||||
}
|
||||
|
||||
private func saveTarget(existingId: String) async {
|
||||
isSavingTarget = true
|
||||
do {
|
||||
let input = UpdateClientTargetInput(
|
||||
weeklyHours: editingWeeklyHours,
|
||||
startDate: editingStartDate?.iso8601FullDate
|
||||
)
|
||||
let _: ClientTarget = try await apiClient.request(
|
||||
endpoint: APIEndpoint.clientTarget(id: existingId),
|
||||
method: .put,
|
||||
body: input,
|
||||
authenticated: true
|
||||
)
|
||||
isEditingTarget = false
|
||||
editingWeeklyHours = nil
|
||||
editingStartDate = nil
|
||||
await loadTarget() // reload to get fresh balance
|
||||
} catch { targetError = error.localizedDescription }
|
||||
isSavingTarget = false
|
||||
}
|
||||
|
||||
private func addCorrection() async {
|
||||
guard let t = target else { return }
|
||||
isSavingCorrection = true
|
||||
do {
|
||||
let input = CreateBalanceCorrectionInput(
|
||||
date: correctionDate.iso8601FullDate,
|
||||
hours: correctionHours,
|
||||
description: correctionDescription.isEmpty ? nil : correctionDescription
|
||||
)
|
||||
try await apiClient.requestVoid(
|
||||
endpoint: APIEndpoint.clientTargetCorrections(targetId: t.id),
|
||||
method: .post,
|
||||
body: input,
|
||||
authenticated: true
|
||||
)
|
||||
correctionHours = 0
|
||||
correctionDescription = ""
|
||||
showAddCorrection = false
|
||||
await loadTarget()
|
||||
} catch { targetError = error.localizedDescription }
|
||||
isSavingCorrection = false
|
||||
}
|
||||
|
||||
private func deleteCorrection(_ correction: BalanceCorrection) async {
|
||||
guard let t = target else { return }
|
||||
do {
|
||||
try await apiClient.requestVoid(
|
||||
endpoint: APIEndpoint.clientTargetCorrection(targetId: t.id, correctionId: correction.id),
|
||||
method: .delete,
|
||||
authenticated: true
|
||||
)
|
||||
await loadTarget()
|
||||
} catch { targetError = error.localizedDescription }
|
||||
}
|
||||
|
||||
// MARK: - Helpers
|
||||
|
||||
private func formatDate(_ string: String) -> String {
|
||||
let f = DateFormatter()
|
||||
f.dateFormat = "yyyy-MM-dd"
|
||||
guard let d = f.date(from: string) else { return string }
|
||||
let out = DateFormatter()
|
||||
out.dateStyle = .medium
|
||||
return out.string(from: d)
|
||||
}
|
||||
|
||||
private func parseDate(_ string: String) -> Date? {
|
||||
let f = DateFormatter()
|
||||
f.dateFormat = "yyyy-MM-dd"
|
||||
return f.date(from: string)
|
||||
}
|
||||
}
|
||||
|
||||
// MARK: - Client Form Sheet (create / edit)
|
||||
|
||||
struct ClientFormSheet: View {
|
||||
enum Mode {
|
||||
case create
|
||||
case edit(Client)
|
||||
}
|
||||
|
||||
@Environment(\.dismiss) private var dismiss
|
||||
|
||||
let mode: Mode
|
||||
let onSave: () -> Void
|
||||
|
||||
@State private var name = ""
|
||||
@State private var description = ""
|
||||
@State private var isSaving = false
|
||||
@State private var error: String?
|
||||
|
||||
private let apiClient = APIClient()
|
||||
|
||||
init(mode: Mode, onSave: @escaping () -> Void) {
|
||||
self.mode = mode
|
||||
self.onSave = onSave
|
||||
if case .edit(let client) = mode {
|
||||
_name = State(initialValue: client.name)
|
||||
_description = State(initialValue: client.description ?? "")
|
||||
}
|
||||
}
|
||||
|
||||
var body: some View {
|
||||
NavigationStack {
|
||||
Form {
|
||||
Section("Name") {
|
||||
TextField("Client name", text: $name)
|
||||
}
|
||||
Section("Description (optional)") {
|
||||
TextField("Description", text: $description, axis: .vertical)
|
||||
.lineLimit(3...6)
|
||||
}
|
||||
if let error {
|
||||
Section { Text(error).font(.caption).foregroundStyle(.red) }
|
||||
}
|
||||
}
|
||||
.navigationTitle(isEditing ? "Edit Client" : "New Client")
|
||||
.navigationBarTitleDisplayMode(.inline)
|
||||
.toolbar {
|
||||
ToolbarItem(placement: .cancellationAction) {
|
||||
Button("Cancel") { dismiss() }
|
||||
}
|
||||
ToolbarItem(placement: .confirmationAction) {
|
||||
Button(isSaving ? "Saving…" : "Save") {
|
||||
Task { await save() }
|
||||
}
|
||||
.disabled(name.isEmpty || isSaving)
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
private var isEditing: Bool {
|
||||
if case .edit = mode { return true }
|
||||
return false
|
||||
}
|
||||
|
||||
private func save() async {
|
||||
isSaving = true; error = nil
|
||||
do {
|
||||
switch mode {
|
||||
case .create:
|
||||
let input = CreateClientInput(name: name, description: description.isEmpty ? nil : description)
|
||||
let _: Client = try await apiClient.request(
|
||||
endpoint: APIEndpoint.clients, method: .post, body: input, authenticated: true
|
||||
)
|
||||
case .edit(let client):
|
||||
let input = UpdateClientInput(name: name, description: description.isEmpty ? nil : description)
|
||||
let _: Client = try await apiClient.request(
|
||||
endpoint: APIEndpoint.client(id: client.id), method: .put, body: input, authenticated: true
|
||||
)
|
||||
}
|
||||
isSaving = false
|
||||
dismiss()
|
||||
onSave()
|
||||
} catch {
|
||||
isSaving = false
|
||||
self.error = error.localizedDescription
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
// MARK: - Date extension
|
||||
|
||||
private extension Date {
|
||||
func nextMonday() -> Date {
|
||||
let cal = Calendar.current
|
||||
var comps = DateComponents()
|
||||
comps.weekday = 2 // Monday
|
||||
return cal.nextDate(after: self, matching: comps, matchingPolicy: .nextTime) ?? self
|
||||
}
|
||||
}
|
||||
@@ -0,0 +1,335 @@
|
||||
import SwiftUI
|
||||
|
||||
// MARK: - Projects List (under Settings)
|
||||
|
||||
struct ProjectsListView: View {
|
||||
@State private var projects: [Project] = []
|
||||
@State private var clients: [Client] = []
|
||||
@State private var isLoading = false
|
||||
@State private var error: String?
|
||||
@State private var showAddProject = false
|
||||
@State private var projectToDelete: Project?
|
||||
@State private var showDeleteConfirmation = false
|
||||
|
||||
private let apiClient = APIClient()
|
||||
|
||||
var body: some View {
|
||||
Group {
|
||||
if isLoading && projects.isEmpty {
|
||||
LoadingView()
|
||||
} else if let err = error, projects.isEmpty {
|
||||
ErrorView(message: err) { Task { await loadData() } }
|
||||
} else if projects.isEmpty {
|
||||
EmptyView(icon: "folder", title: "No Projects",
|
||||
message: "Create a project to start tracking time.")
|
||||
} else {
|
||||
projectList
|
||||
}
|
||||
}
|
||||
.navigationTitle("Projects")
|
||||
.navigationBarTitleDisplayMode(.large)
|
||||
.toolbar {
|
||||
ToolbarItem(placement: .primaryAction) {
|
||||
Button { showAddProject = true } label: { Image(systemName: "plus") }
|
||||
}
|
||||
}
|
||||
.task { await loadData() }
|
||||
.sheet(isPresented: $showAddProject) {
|
||||
ProjectFormSheet(mode: .create, clients: clients) {
|
||||
Task { await loadData() }
|
||||
}
|
||||
}
|
||||
.alert("Delete Project?", isPresented: $showDeleteConfirmation, presenting: projectToDelete) { project in
|
||||
Button("Cancel", role: .cancel) {}
|
||||
Button("Delete", role: .destructive) { Task { await deleteProject(project) } }
|
||||
} message: { project in
|
||||
Text("Deleting '\(project.name)' will also delete all its time entries. This cannot be undone.")
|
||||
}
|
||||
}
|
||||
|
||||
// Group projects by client
|
||||
private var projectsByClient: [(clientName: String, projects: [Project])] {
|
||||
let grouped = Dictionary(grouping: projects) { $0.client.name }
|
||||
return grouped.sorted { $0.key < $1.key }
|
||||
.map { (clientName: $0.key, projects: $0.value.sorted { $0.name < $1.name }) }
|
||||
}
|
||||
|
||||
private var projectList: some View {
|
||||
List {
|
||||
ForEach(projectsByClient, id: \.clientName) { group in
|
||||
Section(group.clientName) {
|
||||
ForEach(group.projects) { project in
|
||||
NavigationLink {
|
||||
ProjectDetailView(project: project, clients: clients) {
|
||||
Task { await loadData() }
|
||||
}
|
||||
} label: {
|
||||
ProjectListRow(project: project)
|
||||
}
|
||||
}
|
||||
.onDelete { indexSet in
|
||||
let deleteTargets = indexSet.map { group.projects[$0] }
|
||||
if let first = deleteTargets.first {
|
||||
projectToDelete = first
|
||||
showDeleteConfirmation = true
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
.refreshable { await loadData() }
|
||||
}
|
||||
|
||||
private func loadData() async {
|
||||
isLoading = true; error = nil
|
||||
do {
|
||||
async let fetchProjects: [Project] = apiClient.request(endpoint: APIEndpoint.projects, authenticated: true)
|
||||
async let fetchClients: [Client] = apiClient.request(endpoint: APIEndpoint.clients, authenticated: true)
|
||||
projects = try await fetchProjects
|
||||
clients = try await fetchClients
|
||||
} catch { self.error = error.localizedDescription }
|
||||
isLoading = false
|
||||
}
|
||||
|
||||
private func deleteProject(_ project: Project) async {
|
||||
do {
|
||||
try await apiClient.requestVoid(endpoint: APIEndpoint.project(id: project.id),
|
||||
method: .delete, authenticated: true)
|
||||
projects.removeAll { $0.id == project.id }
|
||||
} catch { self.error = error.localizedDescription }
|
||||
}
|
||||
}
|
||||
|
||||
// MARK: - Project list row
|
||||
|
||||
struct ProjectListRow: View {
|
||||
let project: Project
|
||||
var body: some View {
|
||||
HStack(spacing: 12) {
|
||||
ProjectColorDot(color: project.color, size: 14)
|
||||
VStack(alignment: .leading, spacing: 2) {
|
||||
Text(project.name).font(.headline)
|
||||
if let desc = project.description {
|
||||
Text(desc).font(.caption).foregroundStyle(.secondary).lineLimit(1)
|
||||
}
|
||||
}
|
||||
}
|
||||
.padding(.vertical, 2)
|
||||
}
|
||||
}
|
||||
|
||||
// MARK: - Project Detail / Edit
|
||||
|
||||
struct ProjectDetailView: View {
|
||||
let project: Project
|
||||
let clients: [Client]
|
||||
let onUpdate: () -> Void
|
||||
|
||||
@State private var name: String
|
||||
@State private var projectDescription: String
|
||||
@State private var selectedColor: String
|
||||
@State private var selectedClient: Client?
|
||||
@State private var isSaving = false
|
||||
@State private var saveError: String?
|
||||
@State private var saveSuccess = false
|
||||
|
||||
private let colorPalette = ["#EF4444", "#F97316", "#EAB308", "#22C55E", "#14B8A6",
|
||||
"#06B6D4", "#3B82F6", "#6366F1", "#A855F7", "#EC4899"]
|
||||
private let apiClient = APIClient()
|
||||
|
||||
init(project: Project, clients: [Client], onUpdate: @escaping () -> Void) {
|
||||
self.project = project
|
||||
self.clients = clients
|
||||
self.onUpdate = onUpdate
|
||||
_name = State(initialValue: project.name)
|
||||
_projectDescription = State(initialValue: project.description ?? "")
|
||||
_selectedColor = State(initialValue: project.color ?? "#3B82F6")
|
||||
_selectedClient = State(initialValue: clients.first { $0.id == project.clientId })
|
||||
}
|
||||
|
||||
var body: some View {
|
||||
Form {
|
||||
Section("Name") {
|
||||
TextField("Project name", text: $name)
|
||||
}
|
||||
|
||||
Section("Description (optional)") {
|
||||
TextField("Description", text: $projectDescription, axis: .vertical)
|
||||
.lineLimit(2...5)
|
||||
}
|
||||
|
||||
Section("Colour") {
|
||||
LazyVGrid(columns: Array(repeating: GridItem(.flexible()), count: 5), spacing: 16) {
|
||||
ForEach(colorPalette, id: \.self) { color in
|
||||
Circle()
|
||||
.fill(Color(hex: color))
|
||||
.frame(width: 44, height: 44)
|
||||
.overlay(
|
||||
Circle().strokeBorder(
|
||||
Color.primary,
|
||||
lineWidth: selectedColor == color ? 3 : 0
|
||||
)
|
||||
)
|
||||
.onTapGesture { selectedColor = color }
|
||||
}
|
||||
}
|
||||
.padding(.vertical, 8)
|
||||
}
|
||||
|
||||
Section("Client") {
|
||||
Picker("Client", selection: $selectedClient) {
|
||||
Text("Select Client").tag(nil as Client?)
|
||||
ForEach(clients) { client in
|
||||
Text(client.name).tag(client as Client?)
|
||||
}
|
||||
}
|
||||
.pickerStyle(.navigationLink)
|
||||
}
|
||||
|
||||
if let err = saveError {
|
||||
Section { Text(err).font(.caption).foregroundStyle(.red) }
|
||||
}
|
||||
if saveSuccess {
|
||||
Section { Label("Saved", systemImage: "checkmark.circle").foregroundStyle(.green) }
|
||||
}
|
||||
|
||||
Section {
|
||||
Button(isSaving ? "Saving…" : "Save Project") {
|
||||
Task { await save() }
|
||||
}
|
||||
.disabled(name.isEmpty || selectedClient == nil || isSaving)
|
||||
}
|
||||
}
|
||||
.navigationTitle(project.name)
|
||||
.navigationBarTitleDisplayMode(.inline)
|
||||
.onAppear {
|
||||
// Ensure selectedClient resolves correctly once clients are available
|
||||
if selectedClient == nil {
|
||||
selectedClient = clients.first { $0.id == project.clientId }
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
private func save() async {
|
||||
guard let client = selectedClient else { return }
|
||||
isSaving = true; saveError = nil; saveSuccess = false
|
||||
do {
|
||||
let input = UpdateProjectInput(
|
||||
name: name,
|
||||
description: projectDescription.isEmpty ? nil : projectDescription,
|
||||
color: selectedColor,
|
||||
clientId: client.id
|
||||
)
|
||||
let _: Project = try await apiClient.request(
|
||||
endpoint: APIEndpoint.project(id: project.id),
|
||||
method: .put,
|
||||
body: input,
|
||||
authenticated: true
|
||||
)
|
||||
saveSuccess = true
|
||||
onUpdate()
|
||||
} catch { saveError = error.localizedDescription }
|
||||
isSaving = false
|
||||
}
|
||||
}
|
||||
|
||||
// MARK: - Project Form Sheet (create)
|
||||
|
||||
struct ProjectFormSheet: View {
|
||||
enum Mode { case create }
|
||||
|
||||
@Environment(\.dismiss) private var dismiss
|
||||
|
||||
let mode: Mode
|
||||
let clients: [Client]
|
||||
let onSave: () -> Void
|
||||
|
||||
@State private var name = ""
|
||||
@State private var description = ""
|
||||
@State private var selectedColor = "#3B82F6"
|
||||
@State private var selectedClient: Client?
|
||||
@State private var isSaving = false
|
||||
@State private var error: String?
|
||||
|
||||
private let colorPalette = ["#EF4444", "#F97316", "#EAB308", "#22C55E", "#14B8A6",
|
||||
"#06B6D4", "#3B82F6", "#6366F1", "#A855F7", "#EC4899"]
|
||||
private let apiClient = APIClient()
|
||||
|
||||
var body: some View {
|
||||
NavigationStack {
|
||||
Form {
|
||||
Section("Name") {
|
||||
TextField("Project name", text: $name)
|
||||
}
|
||||
Section("Description (optional)") {
|
||||
TextField("Description", text: $description, axis: .vertical)
|
||||
.lineLimit(2...5)
|
||||
}
|
||||
Section("Colour") {
|
||||
LazyVGrid(columns: Array(repeating: GridItem(.flexible()), count: 5), spacing: 16) {
|
||||
ForEach(colorPalette, id: \.self) { color in
|
||||
Circle()
|
||||
.fill(Color(hex: color))
|
||||
.frame(width: 44, height: 44)
|
||||
.overlay(
|
||||
Circle().strokeBorder(
|
||||
Color.primary,
|
||||
lineWidth: selectedColor == color ? 3 : 0
|
||||
)
|
||||
)
|
||||
.onTapGesture { selectedColor = color }
|
||||
}
|
||||
}
|
||||
.padding(.vertical, 8)
|
||||
}
|
||||
Section("Client") {
|
||||
Picker("Client", selection: $selectedClient) {
|
||||
Text("Select Client").tag(nil as Client?)
|
||||
ForEach(clients) { client in
|
||||
Text(client.name).tag(client as Client?)
|
||||
}
|
||||
}
|
||||
.pickerStyle(.navigationLink)
|
||||
}
|
||||
if let error {
|
||||
Section { Text(error).font(.caption).foregroundStyle(.red) }
|
||||
}
|
||||
}
|
||||
.navigationTitle("New Project")
|
||||
.navigationBarTitleDisplayMode(.inline)
|
||||
.toolbar {
|
||||
ToolbarItem(placement: .cancellationAction) {
|
||||
Button("Cancel") { dismiss() }
|
||||
}
|
||||
ToolbarItem(placement: .confirmationAction) {
|
||||
Button(isSaving ? "Saving…" : "Save") {
|
||||
Task { await save() }
|
||||
}
|
||||
.disabled(name.isEmpty || selectedClient == nil || isSaving)
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
private func save() async {
|
||||
guard let client = selectedClient else { return }
|
||||
isSaving = true; error = nil
|
||||
do {
|
||||
let input = CreateProjectInput(
|
||||
name: name,
|
||||
description: description.isEmpty ? nil : description,
|
||||
color: selectedColor,
|
||||
clientId: client.id
|
||||
)
|
||||
let _: Project = try await apiClient.request(
|
||||
endpoint: APIEndpoint.projects, method: .post, body: input, authenticated: true
|
||||
)
|
||||
isSaving = false
|
||||
dismiss()
|
||||
onSave()
|
||||
} catch {
|
||||
isSaving = false
|
||||
self.error = error.localizedDescription
|
||||
}
|
||||
}
|
||||
}
|
||||
@@ -0,0 +1,74 @@
|
||||
import SwiftUI
|
||||
|
||||
struct SettingsView: View {
|
||||
@EnvironmentObject var authManager: AuthManager
|
||||
@State private var showLogoutConfirmation = false
|
||||
|
||||
var body: some View {
|
||||
NavigationStack {
|
||||
List {
|
||||
// User info header
|
||||
if let user = authManager.currentUser {
|
||||
Section {
|
||||
HStack(spacing: 14) {
|
||||
Circle()
|
||||
.fill(Color.accentColor.opacity(0.15))
|
||||
.frame(width: 50, height: 50)
|
||||
.overlay(
|
||||
Text(user.username.prefix(1).uppercased())
|
||||
.font(.title3)
|
||||
.fontWeight(.semibold)
|
||||
.foregroundStyle(Color.accentColor)
|
||||
)
|
||||
VStack(alignment: .leading, spacing: 2) {
|
||||
Text(user.fullName ?? user.username)
|
||||
.font(.headline)
|
||||
Text(user.email)
|
||||
.font(.subheadline)
|
||||
.foregroundStyle(.secondary)
|
||||
}
|
||||
}
|
||||
.padding(.vertical, 6)
|
||||
}
|
||||
}
|
||||
|
||||
// Navigation
|
||||
Section("Data") {
|
||||
NavigationLink {
|
||||
ClientsListView()
|
||||
} label: {
|
||||
Label("Clients", systemImage: "person.2")
|
||||
}
|
||||
|
||||
NavigationLink {
|
||||
ProjectsListView()
|
||||
} label: {
|
||||
Label("Projects", systemImage: "folder")
|
||||
}
|
||||
}
|
||||
|
||||
// Logout
|
||||
Section {
|
||||
Button(role: .destructive) {
|
||||
showLogoutConfirmation = true
|
||||
} label: {
|
||||
HStack {
|
||||
Spacer()
|
||||
Label("Sign Out", systemImage: "rectangle.portrait.and.arrow.right")
|
||||
Spacer()
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
.navigationTitle("Settings")
|
||||
.alert("Sign Out?", isPresented: $showLogoutConfirmation) {
|
||||
Button("Cancel", role: .cancel) {}
|
||||
Button("Sign Out", role: .destructive) {
|
||||
Task { try? await authManager.logout() }
|
||||
}
|
||||
} message: {
|
||||
Text("You will be signed out and need to sign in again to use the app.")
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
@@ -2,10 +2,19 @@ import SwiftUI
|
||||
|
||||
struct TimeEntriesView: View {
|
||||
@StateObject private var viewModel = TimeEntriesViewModel()
|
||||
|
||||
// dayOffset is the source of truth: 0 = today, -1 = yesterday, etc.
|
||||
@State private var dayOffset: Int = 0
|
||||
// tabSelection is always snapped back to 1 (middle) after each swipe.
|
||||
// Pages are: 0 = dayOffset-1, 1 = dayOffset, 2 = dayOffset+1
|
||||
@State private var tabSelection: Int = 1
|
||||
|
||||
@State private var showFilterSheet = false
|
||||
@State private var showAddEntry = false
|
||||
@State private var entryToEdit: TimeEntry?
|
||||
@State private var entryToDelete: TimeEntry?
|
||||
@State private var showDeleteConfirmation = false
|
||||
|
||||
|
||||
var body: some View {
|
||||
NavigationStack {
|
||||
Group {
|
||||
@@ -15,109 +24,338 @@ struct TimeEntriesView: View {
|
||||
ErrorView(message: error) {
|
||||
Task { await viewModel.loadEntries() }
|
||||
}
|
||||
} else if viewModel.entries.isEmpty {
|
||||
EmptyView(
|
||||
icon: "clock",
|
||||
title: "No Time Entries",
|
||||
message: "Start tracking your time to see entries here."
|
||||
)
|
||||
} else {
|
||||
entriesList
|
||||
mainContent
|
||||
}
|
||||
}
|
||||
.navigationTitle("Time Entries")
|
||||
.toolbar {
|
||||
ToolbarItem(placement: .primaryAction) {
|
||||
Button {
|
||||
showAddEntry = true
|
||||
} label: {
|
||||
Image(systemName: "plus")
|
||||
.navigationTitle("Entries")
|
||||
.navigationBarTitleDisplayMode(.inline)
|
||||
.toolbar { toolbarContent }
|
||||
.task { await viewModel.loadEntries() }
|
||||
.refreshable { await viewModel.loadEntries() }
|
||||
.sheet(isPresented: $showFilterSheet) {
|
||||
TimeEntriesFilterSheet(viewModel: viewModel) {
|
||||
Task { await viewModel.loadEntries() }
|
||||
}
|
||||
}
|
||||
.sheet(isPresented: $showAddEntry) {
|
||||
TimeEntryDetailSheet(entry: nil) {
|
||||
Task { await viewModel.loadEntries() }
|
||||
}
|
||||
}
|
||||
.sheet(item: $entryToEdit) { entry in
|
||||
TimeEntryDetailSheet(entry: entry) {
|
||||
Task { await viewModel.loadEntries() }
|
||||
}
|
||||
}
|
||||
.alert("Delete Entry?", isPresented: $showDeleteConfirmation, presenting: entryToDelete) { entry in
|
||||
Button("Cancel", role: .cancel) {}
|
||||
Button("Delete", role: .destructive) {
|
||||
Task { await viewModel.deleteEntry(entry) }
|
||||
}
|
||||
} message: { entry in
|
||||
Text("Delete the time entry for '\(entry.project.name)'? This cannot be undone.")
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
// MARK: - Toolbar
|
||||
|
||||
@ToolbarContentBuilder
|
||||
private var toolbarContent: some ToolbarContent {
|
||||
ToolbarItem(placement: .primaryAction) {
|
||||
Button { showAddEntry = true } label: { Image(systemName: "plus") }
|
||||
}
|
||||
ToolbarItem(placement: .topBarLeading) {
|
||||
Button {
|
||||
Task { await viewModel.loadFilterSupportData() }
|
||||
showFilterSheet = true
|
||||
} label: {
|
||||
Image(systemName: viewModel.activeFilters.isEmpty ? "line.3.horizontal.decrease.circle" : "line.3.horizontal.decrease.circle.fill")
|
||||
}
|
||||
}
|
||||
|
||||
// Place the DatePicker in the principal placement (center of nav bar)
|
||||
ToolbarItem(placement: .principal) {
|
||||
DatePicker(
|
||||
"",
|
||||
selection: Binding(
|
||||
get: {
|
||||
Calendar.current.date(byAdding: .day, value: dayOffset, to: Calendar.current.startOfDay(for: Date())) ?? Date()
|
||||
},
|
||||
set: { newDate in
|
||||
let today = Calendar.current.startOfDay(for: Date())
|
||||
let normalizedNewDate = Calendar.current.startOfDay(for: newDate)
|
||||
let components = Calendar.current.dateComponents([.day], from: today, to: normalizedNewDate)
|
||||
if let dayDifference = components.day {
|
||||
dayOffset = dayDifference
|
||||
}
|
||||
}
|
||||
),
|
||||
displayedComponents: .date
|
||||
)
|
||||
.datePickerStyle(.compact)
|
||||
.labelsHidden()
|
||||
.environment(\.locale, Locale.current) // Ensure correct start of week
|
||||
}
|
||||
}
|
||||
|
||||
// MARK: - Main content
|
||||
|
||||
private var mainContent: some View {
|
||||
// Only 3 pages exist at any time: previous, current, next.
|
||||
// After each swipe settles, we reset tabSelection to 1 and shift
|
||||
// dayOffset, so the carousel appears infinite while staying cheap.
|
||||
TabView(selection: $tabSelection) {
|
||||
ForEach(0..<3, id: \.self) { page in
|
||||
let offset = dayOffset + (page - 1)
|
||||
let day = Calendar.current.date(
|
||||
byAdding: .day,
|
||||
value: offset,
|
||||
to: Calendar.current.startOfDay(for: Date())
|
||||
) ?? Date()
|
||||
|
||||
ScrollView {
|
||||
dayEntriesSection(for: day)
|
||||
}
|
||||
.tag(page)
|
||||
}
|
||||
}
|
||||
.tabViewStyle(.page(indexDisplayMode: .never))
|
||||
.onChange(of: tabSelection) { _, newPage in
|
||||
guard newPage != 1 else { return }
|
||||
// Shift the logical day offset by how many pages we moved.
|
||||
dayOffset += newPage - 1
|
||||
// Snap back to the middle page without animation so the
|
||||
// surrounding pages are refreshed invisibly.
|
||||
var tx = Transaction()
|
||||
tx.disablesAnimations = true
|
||||
withTransaction(tx) {
|
||||
tabSelection = 1
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
// MARK: - Day entries section
|
||||
|
||||
private func dayEntriesSection(for day: Date) -> some View {
|
||||
let dayEntries = viewModel.entries(for: day)
|
||||
return VStack(alignment: .leading, spacing: 0) {
|
||||
|
||||
// Optional: A small summary header for the day
|
||||
HStack {
|
||||
Text(dayTitle(day))
|
||||
.font(.subheadline)
|
||||
.fontWeight(.semibold)
|
||||
.foregroundStyle(.primary)
|
||||
Spacer()
|
||||
Text(dayEntries.isEmpty ? "No entries" : "\(dayEntries.count) \(dayEntries.count == 1 ? "entry" : "entries")")
|
||||
.font(.caption)
|
||||
.foregroundStyle(.secondary)
|
||||
}
|
||||
.padding(.horizontal)
|
||||
.padding(.vertical, 12)
|
||||
|
||||
if dayEntries.isEmpty {
|
||||
Text("No entries for this day")
|
||||
.font(.subheadline)
|
||||
.foregroundStyle(.secondary)
|
||||
.frame(maxWidth: .infinity, alignment: .center)
|
||||
.padding(.vertical, 32)
|
||||
} else {
|
||||
ForEach(Array(dayEntries.enumerated()), id: \.element.id) { index, entry in
|
||||
EntryRow(entry: entry)
|
||||
.contentShape(Rectangle())
|
||||
.onTapGesture { entryToEdit = entry }
|
||||
.swipeActions(edge: .trailing, allowsFullSwipe: false) {
|
||||
Button(role: .destructive) {
|
||||
entryToDelete = entry
|
||||
showDeleteConfirmation = true
|
||||
} label: {
|
||||
Label("Delete", systemImage: "trash")
|
||||
}
|
||||
}
|
||||
if index < dayEntries.count - 1 {
|
||||
Divider().padding(.leading, 56)
|
||||
}
|
||||
}
|
||||
}
|
||||
.task {
|
||||
await viewModel.loadEntries()
|
||||
}
|
||||
.sheet(isPresented: $showAddEntry) {
|
||||
TimeEntryFormView(onSave: {
|
||||
Task { await viewModel.loadEntries() }
|
||||
})
|
||||
}
|
||||
}
|
||||
.padding(.bottom, 40) // Give some breathing room at the bottom of the scroll
|
||||
}
|
||||
|
||||
private var entriesList: some View {
|
||||
List {
|
||||
ForEach(viewModel.entries) { entry in
|
||||
TimeEntryRow(entry: entry)
|
||||
}
|
||||
.onDelete { indexSet in
|
||||
if let index = indexSet.first {
|
||||
entryToDelete = viewModel.entries[index]
|
||||
showDeleteConfirmation = true
|
||||
}
|
||||
}
|
||||
}
|
||||
.alert("Delete Time Entry?", isPresented: $showDeleteConfirmation, presenting: entryToDelete) { entry in
|
||||
Button("Cancel", role: .cancel) {}
|
||||
Button("Delete", role: .destructive) {
|
||||
Task {
|
||||
await viewModel.deleteEntry(entry)
|
||||
}
|
||||
}
|
||||
} message: { entry in
|
||||
Text("This will permanently delete the time entry for '\(entry.project.name)'. This action cannot be undone.")
|
||||
}
|
||||
.refreshable {
|
||||
await viewModel.loadEntries()
|
||||
}
|
||||
|
||||
// MARK: - Helpers
|
||||
|
||||
private func dayTitle(_ date: Date) -> String {
|
||||
let cal = Calendar.current
|
||||
if cal.isDateInToday(date) { return "Today" }
|
||||
if cal.isDateInYesterday(date) { return "Yesterday" }
|
||||
if cal.isDateInTomorrow(date) { return "Tomorrow" }
|
||||
let formatter = DateFormatter()
|
||||
formatter.dateFormat = "EEEE, MMM d"
|
||||
return formatter.string(from: date)
|
||||
}
|
||||
}
|
||||
|
||||
struct TimeEntryRow: View {
|
||||
// MARK: - Entry Row
|
||||
|
||||
struct EntryRow: View {
|
||||
let entry: TimeEntry
|
||||
|
||||
|
||||
var body: some View {
|
||||
VStack(alignment: .leading, spacing: 8) {
|
||||
HStack {
|
||||
ProjectColorDot(color: entry.project.color)
|
||||
HStack(alignment: .top, spacing: 12) {
|
||||
// Color dot
|
||||
ProjectColorDot(color: entry.project.color, size: 12)
|
||||
.padding(.top, 4)
|
||||
|
||||
VStack(alignment: .leading, spacing: 3) {
|
||||
Text(entry.project.name)
|
||||
.font(.headline)
|
||||
Spacer()
|
||||
Text(entry.duration.formattedHours)
|
||||
.font(.subheadline)
|
||||
.foregroundStyle(.secondary)
|
||||
}
|
||||
|
||||
HStack {
|
||||
Text(formatDateRange(start: entry.startTime, end: entry.endTime))
|
||||
.font(.subheadline)
|
||||
.foregroundStyle(.secondary)
|
||||
Spacer()
|
||||
Text(entry.project.client.name)
|
||||
.font(.caption)
|
||||
.foregroundStyle(.secondary)
|
||||
}
|
||||
|
||||
if let description = entry.description {
|
||||
Text(description)
|
||||
.font(.caption)
|
||||
.foregroundStyle(.secondary)
|
||||
.lineLimit(2)
|
||||
.fontWeight(.medium)
|
||||
|
||||
HStack(spacing: 6) {
|
||||
Text(timeRange)
|
||||
.font(.caption)
|
||||
.foregroundStyle(.secondary)
|
||||
Text("·")
|
||||
.font(.caption)
|
||||
.foregroundStyle(.secondary)
|
||||
Text(entry.project.client.name)
|
||||
.font(.caption)
|
||||
.foregroundStyle(.secondary)
|
||||
}
|
||||
|
||||
if let desc = entry.description, !desc.isEmpty {
|
||||
Text(desc)
|
||||
.font(.caption)
|
||||
.foregroundStyle(.secondary)
|
||||
.lineLimit(2)
|
||||
}
|
||||
}
|
||||
|
||||
Spacer()
|
||||
|
||||
Text(entry.duration.formattedShortDuration)
|
||||
.font(.subheadline)
|
||||
.foregroundStyle(.secondary)
|
||||
}
|
||||
.padding(.vertical, 4)
|
||||
.padding(.horizontal)
|
||||
.padding(.vertical, 10)
|
||||
}
|
||||
|
||||
private func formatDateRange(start: String, end: String) -> String {
|
||||
let formatter = DateFormatter()
|
||||
formatter.dateFormat = "MMM d, HH:mm"
|
||||
|
||||
guard let startDate = Date.fromISO8601(start),
|
||||
let endDate = Date.fromISO8601(end) else {
|
||||
return ""
|
||||
}
|
||||
|
||||
return "\(formatter.string(from: startDate)) - \(formatter.string(from: endDate))"
|
||||
|
||||
private var timeRange: String {
|
||||
let fmt = DateFormatter()
|
||||
fmt.dateFormat = "HH:mm"
|
||||
let start = Date.fromISO8601(entry.startTime).map { fmt.string(from: $0) } ?? ""
|
||||
let end = Date.fromISO8601(entry.endTime).map { fmt.string(from: $0) } ?? ""
|
||||
return "\(start) – \(end)"
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
// MARK: - Filter Sheet
|
||||
|
||||
struct TimeEntriesFilterSheet: View {
|
||||
@Environment(\.dismiss) private var dismiss
|
||||
@ObservedObject var viewModel: TimeEntriesViewModel
|
||||
let onApply: () -> Void
|
||||
|
||||
@State private var startDate: Date = Calendar.current.date(byAdding: .month, value: -1, to: Date()) ?? Date()
|
||||
@State private var endDate: Date = Date()
|
||||
@State private var useStartDate = false
|
||||
@State private var useEndDate = false
|
||||
@State private var selectedProjectId: String?
|
||||
@State private var selectedProjectName: String?
|
||||
@State private var selectedClientId: String?
|
||||
@State private var selectedClientName: String?
|
||||
|
||||
var body: some View {
|
||||
NavigationStack {
|
||||
Form {
|
||||
Section("Date Range") {
|
||||
Toggle("From", isOn: $useStartDate)
|
||||
if useStartDate {
|
||||
DatePicker("", selection: $startDate, displayedComponents: .date)
|
||||
.labelsHidden()
|
||||
}
|
||||
Toggle("To", isOn: $useEndDate)
|
||||
if useEndDate {
|
||||
DatePicker("", selection: $endDate, displayedComponents: .date)
|
||||
.labelsHidden()
|
||||
}
|
||||
}
|
||||
|
||||
Section("Project") {
|
||||
Picker("Project", selection: $selectedProjectId) {
|
||||
Text("Any Project").tag(nil as String?)
|
||||
ForEach(viewModel.projects) { project in
|
||||
HStack {
|
||||
ProjectColorDot(color: project.color, size: 10)
|
||||
Text(project.name)
|
||||
}
|
||||
.tag(project.id as String?)
|
||||
}
|
||||
}
|
||||
.pickerStyle(.navigationLink)
|
||||
.onChange(of: selectedProjectId) { _, newId in
|
||||
selectedProjectName = viewModel.projects.first { $0.id == newId }?.name
|
||||
}
|
||||
}
|
||||
|
||||
Section("Client") {
|
||||
Picker("Client", selection: $selectedClientId) {
|
||||
Text("Any Client").tag(nil as String?)
|
||||
ForEach(viewModel.clients) { client in
|
||||
Text(client.name).tag(client.id as String?)
|
||||
}
|
||||
}
|
||||
.pickerStyle(.navigationLink)
|
||||
.onChange(of: selectedClientId) { _, newId in
|
||||
selectedClientName = viewModel.clients.first { $0.id == newId }?.name
|
||||
}
|
||||
}
|
||||
|
||||
Section {
|
||||
Button("Clear All Filters", role: .destructive) {
|
||||
useStartDate = false
|
||||
useEndDate = false
|
||||
selectedProjectId = nil
|
||||
selectedClientId = nil
|
||||
}
|
||||
}
|
||||
}
|
||||
.navigationTitle("Filter Entries")
|
||||
.navigationBarTitleDisplayMode(.inline)
|
||||
.toolbar {
|
||||
ToolbarItem(placement: .cancellationAction) {
|
||||
Button("Cancel") { dismiss() }
|
||||
}
|
||||
ToolbarItem(placement: .confirmationAction) {
|
||||
Button("Apply") { applyAndDismiss() }
|
||||
}
|
||||
}
|
||||
.onAppear { loadCurrentFilters() }
|
||||
}
|
||||
}
|
||||
|
||||
private func loadCurrentFilters() {
|
||||
let f = viewModel.activeFilters
|
||||
if let s = f.startDate { startDate = s; useStartDate = true }
|
||||
if let e = f.endDate { endDate = e; useEndDate = true }
|
||||
selectedProjectId = f.projectId
|
||||
selectedClientId = f.clientId
|
||||
}
|
||||
|
||||
private func applyAndDismiss() {
|
||||
viewModel.activeFilters = TimeEntryActiveFilters(
|
||||
startDate: useStartDate ? startDate : nil,
|
||||
endDate: useEndDate ? endDate : nil,
|
||||
projectId: selectedProjectId,
|
||||
projectName: selectedProjectName,
|
||||
clientId: selectedClientId,
|
||||
clientName: selectedClientName
|
||||
)
|
||||
dismiss()
|
||||
onApply()
|
||||
}
|
||||
}
|
||||
|
||||
@@ -1,45 +1,54 @@
|
||||
import Foundation
|
||||
import SwiftUI
|
||||
|
||||
// MARK: - Active Filters (UI state)
|
||||
|
||||
struct TimeEntryActiveFilters: Equatable {
|
||||
var startDate: Date?
|
||||
var endDate: Date?
|
||||
var projectId: String?
|
||||
var projectName: String?
|
||||
var clientId: String?
|
||||
var clientName: String?
|
||||
|
||||
var isEmpty: Bool {
|
||||
startDate == nil && endDate == nil && projectId == nil && clientId == nil
|
||||
}
|
||||
}
|
||||
|
||||
// MARK: - ViewModel
|
||||
|
||||
@MainActor
|
||||
final class TimeEntriesViewModel: ObservableObject {
|
||||
// All loaded entries (flat list, accumulated across pages)
|
||||
@Published var entries: [TimeEntry] = []
|
||||
@Published var pagination: Pagination?
|
||||
@Published var isLoading = false
|
||||
@Published var isLoadingMore = false
|
||||
@Published var error: String?
|
||||
@Published var filters = TimeEntryFilters()
|
||||
|
||||
|
||||
// Active filters driving the current fetch
|
||||
@Published var activeFilters = TimeEntryActiveFilters()
|
||||
|
||||
// Projects and clients needed for filter sheet pickers
|
||||
@Published var projects: [Project] = []
|
||||
@Published var clients: [Client] = []
|
||||
|
||||
private let apiClient = APIClient()
|
||||
|
||||
func loadEntries() async {
|
||||
|
||||
// MARK: - Fetch
|
||||
|
||||
func loadEntries(resetPage: Bool = true) async {
|
||||
if resetPage { entries = [] }
|
||||
isLoading = true
|
||||
error = nil
|
||||
|
||||
|
||||
do {
|
||||
var queryItems: [URLQueryItem] = []
|
||||
|
||||
if let startDate = filters.startDate {
|
||||
queryItems.append(URLQueryItem(name: "startDate", value: startDate))
|
||||
}
|
||||
if let endDate = filters.endDate {
|
||||
queryItems.append(URLQueryItem(name: "endDate", value: endDate))
|
||||
}
|
||||
if let projectId = filters.projectId {
|
||||
queryItems.append(URLQueryItem(name: "projectId", value: projectId))
|
||||
}
|
||||
if let page = filters.page {
|
||||
queryItems.append(URLQueryItem(name: "page", value: String(page)))
|
||||
}
|
||||
if let limit = filters.limit {
|
||||
queryItems.append(URLQueryItem(name: "limit", value: String(limit)))
|
||||
}
|
||||
|
||||
let response: TimeEntryListResponse = try await apiClient.request(
|
||||
endpoint: APIEndpoint.timeEntries,
|
||||
queryItems: queryItems.isEmpty ? nil : queryItems,
|
||||
queryItems: buildQueryItems(page: 1),
|
||||
authenticated: true
|
||||
)
|
||||
|
||||
entries = response.entries
|
||||
pagination = response.pagination
|
||||
isLoading = false
|
||||
@@ -48,23 +57,30 @@ final class TimeEntriesViewModel: ObservableObject {
|
||||
self.error = error.localizedDescription
|
||||
}
|
||||
}
|
||||
|
||||
func nextPage() async {
|
||||
guard let pagination = pagination,
|
||||
|
||||
func loadMoreIfNeeded(currentEntry entry: TimeEntry) async {
|
||||
guard let pagination, !isLoadingMore,
|
||||
pagination.page < pagination.totalPages else { return }
|
||||
|
||||
filters.page = pagination.page + 1
|
||||
await loadEntries()
|
||||
// Trigger when the last entry in the list becomes visible
|
||||
guard entries.last?.id == entry.id else { return }
|
||||
|
||||
isLoadingMore = true
|
||||
let nextPage = pagination.page + 1
|
||||
|
||||
do {
|
||||
let response: TimeEntryListResponse = try await apiClient.request(
|
||||
endpoint: APIEndpoint.timeEntries,
|
||||
queryItems: buildQueryItems(page: nextPage),
|
||||
authenticated: true
|
||||
)
|
||||
entries.append(contentsOf: response.entries)
|
||||
self.pagination = response.pagination
|
||||
} catch {
|
||||
self.error = error.localizedDescription
|
||||
}
|
||||
isLoadingMore = false
|
||||
}
|
||||
|
||||
func previousPage() async {
|
||||
guard let pagination = pagination,
|
||||
pagination.page > 1 else { return }
|
||||
|
||||
filters.page = pagination.page - 1
|
||||
await loadEntries()
|
||||
}
|
||||
|
||||
|
||||
func deleteEntry(_ entry: TimeEntry) async {
|
||||
do {
|
||||
try await apiClient.requestVoid(
|
||||
@@ -72,10 +88,83 @@ final class TimeEntriesViewModel: ObservableObject {
|
||||
method: .delete,
|
||||
authenticated: true
|
||||
)
|
||||
|
||||
entries.removeAll { $0.id == entry.id }
|
||||
} catch {
|
||||
self.error = error.localizedDescription
|
||||
}
|
||||
}
|
||||
|
||||
// MARK: - Supporting data for filter sheet
|
||||
|
||||
func loadFilterSupportData() async {
|
||||
async let fetchProjects: [Project] = apiClient.request(
|
||||
endpoint: APIEndpoint.projects,
|
||||
authenticated: true
|
||||
)
|
||||
async let fetchClients: [Client] = apiClient.request(
|
||||
endpoint: APIEndpoint.clients,
|
||||
authenticated: true
|
||||
)
|
||||
projects = (try? await fetchProjects) ?? []
|
||||
clients = (try? await fetchClients) ?? []
|
||||
}
|
||||
|
||||
// MARK: - Entries grouped by calendar day
|
||||
|
||||
var entriesByDay: [(date: Date, entries: [TimeEntry])] {
|
||||
let calendar = Calendar.current
|
||||
let grouped = Dictionary(grouping: entries) { entry -> Date in
|
||||
guard let d = Date.fromISO8601(entry.startTime) else { return Date() }
|
||||
return calendar.startOfDay(for: d)
|
||||
}
|
||||
return grouped
|
||||
.sorted { $0.key > $1.key }
|
||||
.map { (date: $0.key, entries: $0.value.sorted {
|
||||
(Date.fromISO8601($0.startTime) ?? Date()) > (Date.fromISO8601($1.startTime) ?? Date())
|
||||
}) }
|
||||
}
|
||||
|
||||
/// All calendar days that have at least one entry (for dot decorations)
|
||||
var daysWithEntries: Set<Date> {
|
||||
let calendar = Calendar.current
|
||||
return Set(entries.compactMap { entry in
|
||||
guard let d = Date.fromISO8601(entry.startTime) else { return nil }
|
||||
return calendar.startOfDay(for: d)
|
||||
})
|
||||
}
|
||||
|
||||
/// Entries for a specific calendar day
|
||||
func entries(for day: Date) -> [TimeEntry] {
|
||||
let calendar = Calendar.current
|
||||
return entries.filter { entry in
|
||||
guard let d = Date.fromISO8601(entry.startTime) else { return false }
|
||||
return calendar.isDate(d, inSameDayAs: day)
|
||||
}.sorted {
|
||||
(Date.fromISO8601($0.startTime) ?? Date()) < (Date.fromISO8601($1.startTime) ?? Date())
|
||||
}
|
||||
}
|
||||
|
||||
// MARK: - Helpers
|
||||
|
||||
private func buildQueryItems(page: Int) -> [URLQueryItem] {
|
||||
var items: [URLQueryItem] = [
|
||||
URLQueryItem(name: "page", value: "\(page)"),
|
||||
URLQueryItem(name: "limit", value: "100")
|
||||
]
|
||||
if let start = activeFilters.startDate {
|
||||
items.append(URLQueryItem(name: "startDate", value: start.iso8601String))
|
||||
}
|
||||
if let end = activeFilters.endDate {
|
||||
// Push to end-of-day so the full day is included
|
||||
let endOfDay = Calendar.current.date(bySettingHour: 23, minute: 59, second: 59, of: end) ?? end
|
||||
items.append(URLQueryItem(name: "endDate", value: endOfDay.iso8601String))
|
||||
}
|
||||
if let pid = activeFilters.projectId {
|
||||
items.append(URLQueryItem(name: "projectId", value: pid))
|
||||
}
|
||||
if let cid = activeFilters.clientId {
|
||||
items.append(URLQueryItem(name: "clientId", value: cid))
|
||||
}
|
||||
return items
|
||||
}
|
||||
}
|
||||
|
||||
@@ -0,0 +1,212 @@
|
||||
import SwiftUI
|
||||
|
||||
/// Detail/edit sheet for a single time entry. Used both for creating new entries
|
||||
/// (pass `entry: nil`) and editing existing ones.
|
||||
struct TimeEntryDetailSheet: View {
|
||||
@Environment(\.dismiss) private var dismiss
|
||||
|
||||
// Pass an existing entry to edit it; pass nil to create a new one.
|
||||
let entry: TimeEntry?
|
||||
let onSave: () -> Void
|
||||
|
||||
// Form state
|
||||
@State private var startDateTime = Date()
|
||||
@State private var endDateTime = Date()
|
||||
@State private var description = ""
|
||||
@State private var selectedProject: Project?
|
||||
|
||||
// Supporting data
|
||||
@State private var projects: [Project] = []
|
||||
@State private var isLoading = false
|
||||
@State private var isSaving = false
|
||||
@State private var error: String?
|
||||
|
||||
private let apiClient = APIClient()
|
||||
|
||||
init(entry: TimeEntry? = nil, onSave: @escaping () -> Void) {
|
||||
self.entry = entry
|
||||
self.onSave = onSave
|
||||
}
|
||||
|
||||
// MARK: - Body
|
||||
|
||||
var body: some View {
|
||||
NavigationStack {
|
||||
Form {
|
||||
projectSection
|
||||
timeSection
|
||||
descriptionSection
|
||||
if let error { errorSection(error) }
|
||||
}
|
||||
.navigationTitle(entry == nil ? "New Entry" : "Edit Entry")
|
||||
.navigationBarTitleDisplayMode(.inline)
|
||||
.toolbar {
|
||||
ToolbarItem(placement: .cancellationAction) {
|
||||
Button("Cancel") { dismiss() }
|
||||
}
|
||||
ToolbarItem(placement: .confirmationAction) {
|
||||
if isSaving {
|
||||
ProgressView().controlSize(.small)
|
||||
} else {
|
||||
Button("Save") { Task { await save() } }
|
||||
.disabled(selectedProject == nil || endDateTime <= startDateTime)
|
||||
}
|
||||
}
|
||||
}
|
||||
.task {
|
||||
await loadProjects()
|
||||
populateFromEntry()
|
||||
}
|
||||
.overlay { if isLoading { LoadingView() } }
|
||||
}
|
||||
}
|
||||
|
||||
// MARK: - Sections
|
||||
|
||||
private var projectSection: some View {
|
||||
Section("Project") {
|
||||
Picker("Project", selection: $selectedProject) {
|
||||
Text("Select Project").tag(nil as Project?)
|
||||
ForEach(projects) { project in
|
||||
HStack {
|
||||
ProjectColorDot(color: project.color, size: 10)
|
||||
Text(project.name)
|
||||
Text("· \(project.client.name)")
|
||||
.foregroundStyle(.secondary)
|
||||
}
|
||||
.tag(project as Project?)
|
||||
}
|
||||
}
|
||||
.pickerStyle(.navigationLink)
|
||||
}
|
||||
}
|
||||
|
||||
private var timeSection: some View {
|
||||
Section {
|
||||
DatePicker("Start", selection: $startDateTime)
|
||||
DatePicker("End", selection: $endDateTime, in: startDateTime...)
|
||||
if endDateTime > startDateTime {
|
||||
HStack {
|
||||
Text("Duration")
|
||||
.foregroundStyle(.secondary)
|
||||
Spacer()
|
||||
Text(endDateTime.timeIntervalSince(startDateTime).formattedShortDuration)
|
||||
.foregroundStyle(.secondary)
|
||||
}
|
||||
}
|
||||
} header: {
|
||||
Text("Time")
|
||||
}
|
||||
}
|
||||
|
||||
private var descriptionSection: some View {
|
||||
Section("Description") {
|
||||
TextField("Optional notes…", text: $description, axis: .vertical)
|
||||
.lineLimit(3...8)
|
||||
}
|
||||
}
|
||||
|
||||
private func errorSection(_ message: String) -> some View {
|
||||
Section {
|
||||
Text(message)
|
||||
.font(.caption)
|
||||
.foregroundStyle(.red)
|
||||
}
|
||||
}
|
||||
|
||||
// MARK: - Data loading
|
||||
|
||||
private func loadProjects() async {
|
||||
isLoading = true
|
||||
do {
|
||||
projects = try await apiClient.request(
|
||||
endpoint: APIEndpoint.projects,
|
||||
authenticated: true
|
||||
)
|
||||
// Re-apply project selection now that the list is populated
|
||||
matchProjectAfterLoad()
|
||||
} catch {
|
||||
self.error = error.localizedDescription
|
||||
}
|
||||
isLoading = false
|
||||
}
|
||||
|
||||
private func populateFromEntry() {
|
||||
guard let entry else {
|
||||
// Default: now rounded to minute, 1-hour window
|
||||
let now = Date().roundedToMinute()
|
||||
startDateTime = now
|
||||
endDateTime = now.addingTimeInterval(3600)
|
||||
return
|
||||
}
|
||||
startDateTime = Date.fromISO8601(entry.startTime) ?? Date()
|
||||
endDateTime = Date.fromISO8601(entry.endTime) ?? Date()
|
||||
description = entry.description ?? ""
|
||||
// Pre-select the project once projects are loaded
|
||||
if !projects.isEmpty {
|
||||
selectedProject = projects.first { $0.id == entry.projectId }
|
||||
}
|
||||
}
|
||||
|
||||
// Called after projects load — re-apply the project selection if it wasn't
|
||||
// set yet (projects may have loaded after populateFromEntry ran).
|
||||
private func matchProjectAfterLoad() {
|
||||
guard let entry, selectedProject == nil else { return }
|
||||
selectedProject = projects.first { $0.id == entry.projectId }
|
||||
}
|
||||
|
||||
// MARK: - Save
|
||||
|
||||
private func save() async {
|
||||
guard let project = selectedProject else { return }
|
||||
isSaving = true
|
||||
error = nil
|
||||
|
||||
do {
|
||||
if let existingEntry = entry {
|
||||
let input = UpdateTimeEntryInput(
|
||||
startTime: startDateTime.iso8601String,
|
||||
endTime: endDateTime.iso8601String,
|
||||
description: description.isEmpty ? nil : description,
|
||||
projectId: project.id
|
||||
)
|
||||
try await apiClient.requestVoid(
|
||||
endpoint: APIEndpoint.timeEntry(id: existingEntry.id),
|
||||
method: .put,
|
||||
body: input,
|
||||
authenticated: true
|
||||
)
|
||||
} else {
|
||||
let input = CreateTimeEntryInput(
|
||||
startTime: startDateTime,
|
||||
endTime: endDateTime,
|
||||
description: description.isEmpty ? nil : description,
|
||||
projectId: project.id
|
||||
)
|
||||
try await apiClient.requestVoid(
|
||||
endpoint: APIEndpoint.timeEntries,
|
||||
method: .post,
|
||||
body: input,
|
||||
authenticated: true
|
||||
)
|
||||
}
|
||||
isSaving = false
|
||||
dismiss()
|
||||
onSave()
|
||||
} catch {
|
||||
isSaving = false
|
||||
self.error = error.localizedDescription
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
// MARK: - Date rounding helper
|
||||
|
||||
private extension Date {
|
||||
func roundedToMinute() -> Date {
|
||||
let cal = Calendar.current
|
||||
var comps = cal.dateComponents([.year, .month, .day, .hour, .minute], from: self)
|
||||
comps.second = 0
|
||||
return cal.date(from: comps) ?? self
|
||||
}
|
||||
}
|
||||
@@ -1,171 +1,2 @@
|
||||
import SwiftUI
|
||||
|
||||
struct TimeEntryFormView: View {
|
||||
@Environment(\.dismiss) private var dismiss
|
||||
|
||||
let entry: TimeEntry?
|
||||
let onSave: () -> Void
|
||||
|
||||
@State private var startDate = Date()
|
||||
@State private var startTime = Date()
|
||||
@State private var endDate = Date()
|
||||
@State private var endTime = Date()
|
||||
@State private var description = ""
|
||||
@State private var selectedProject: Project?
|
||||
@State private var projects: [Project] = []
|
||||
@State private var isLoading = false
|
||||
@State private var error: String?
|
||||
|
||||
private let apiClient = APIClient()
|
||||
|
||||
init(entry: TimeEntry? = nil, onSave: @escaping () -> Void) {
|
||||
self.entry = entry
|
||||
self.onSave = onSave
|
||||
}
|
||||
|
||||
var body: some View {
|
||||
NavigationStack {
|
||||
Form {
|
||||
Section("Project") {
|
||||
Picker("Project", selection: $selectedProject) {
|
||||
Text("Select Project").tag(nil as Project?)
|
||||
ForEach(projects) { project in
|
||||
HStack {
|
||||
ProjectColorDot(color: project.color)
|
||||
Text(project.name)
|
||||
}
|
||||
.tag(project as Project?)
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
Section("Start Time") {
|
||||
DatePicker("Date", selection: $startDate, displayedComponents: .date)
|
||||
DatePicker("Time", selection: $startTime, displayedComponents: .hourAndMinute)
|
||||
}
|
||||
|
||||
Section("End Time") {
|
||||
DatePicker("Date", selection: $endDate, displayedComponents: .date)
|
||||
DatePicker("Time", selection: $endTime, displayedComponents: .hourAndMinute)
|
||||
}
|
||||
|
||||
Section("Description (Optional)") {
|
||||
TextField("Description", text: $description, axis: .vertical)
|
||||
.lineLimit(3...6)
|
||||
}
|
||||
}
|
||||
.navigationTitle(entry == nil ? "New Entry" : "Edit Entry")
|
||||
.navigationBarTitleDisplayMode(.inline)
|
||||
.toolbar {
|
||||
ToolbarItem(placement: .cancellationAction) {
|
||||
Button("Cancel") {
|
||||
dismiss()
|
||||
}
|
||||
}
|
||||
ToolbarItem(placement: .confirmationAction) {
|
||||
Button("Save") {
|
||||
save()
|
||||
}
|
||||
.disabled(selectedProject == nil || isLoading)
|
||||
}
|
||||
}
|
||||
.task {
|
||||
await loadProjects()
|
||||
if let entry = entry {
|
||||
await loadEntry(entry)
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
private func loadProjects() async {
|
||||
do {
|
||||
projects = try await apiClient.request(
|
||||
endpoint: APIEndpoint.projects,
|
||||
authenticated: true
|
||||
)
|
||||
} catch {
|
||||
self.error = error.localizedDescription
|
||||
}
|
||||
}
|
||||
|
||||
private func loadEntry(_ entry: TimeEntry) async {
|
||||
let startFormatter = DateFormatter()
|
||||
startFormatter.dateFormat = "yyyy-MM-dd"
|
||||
let timeFormatter = DateFormatter()
|
||||
timeFormatter.dateFormat = "HH:mm"
|
||||
|
||||
if let startDateObj = Date.fromISO8601(entry.startTime) {
|
||||
startDate = startDateObj
|
||||
startTime = startDateObj
|
||||
}
|
||||
|
||||
if let endDateObj = Date.fromISO8601(entry.endTime) {
|
||||
endDate = endDateObj
|
||||
endTime = endDateObj
|
||||
}
|
||||
|
||||
description = entry.description ?? ""
|
||||
selectedProject = projects.first { $0.id == entry.projectId }
|
||||
}
|
||||
|
||||
private func save() {
|
||||
guard let project = selectedProject else { return }
|
||||
|
||||
isLoading = true
|
||||
|
||||
let calendar = Calendar.current
|
||||
let startDateTime = calendar.date(bySettingHour: calendar.component(.hour, from: startTime),
|
||||
minute: calendar.component(.minute, from: startTime),
|
||||
second: 0,
|
||||
of: startDate) ?? startDate
|
||||
let endDateTime = calendar.date(bySettingHour: calendar.component(.hour, from: endTime),
|
||||
minute: calendar.component(.minute, from: endTime),
|
||||
second: 0,
|
||||
of: endDate) ?? endDate
|
||||
|
||||
Task {
|
||||
do {
|
||||
if let existingEntry = entry {
|
||||
let input = UpdateTimeEntryInput(
|
||||
startTime: startDateTime.iso8601String,
|
||||
endTime: endDateTime.iso8601String,
|
||||
description: description.isEmpty ? nil : description,
|
||||
projectId: project.id
|
||||
)
|
||||
try await apiClient.requestVoid(
|
||||
endpoint: APIEndpoint.timeEntry(id: existingEntry.id),
|
||||
method: .put,
|
||||
body: input,
|
||||
authenticated: true
|
||||
)
|
||||
} else {
|
||||
let input = CreateTimeEntryInput(
|
||||
startTime: startDateTime,
|
||||
endTime: endDateTime,
|
||||
description: description.isEmpty ? nil : description,
|
||||
projectId: project.id
|
||||
)
|
||||
try await apiClient.requestVoid(
|
||||
endpoint: APIEndpoint.timeEntries,
|
||||
method: .post,
|
||||
body: input,
|
||||
authenticated: true
|
||||
)
|
||||
}
|
||||
|
||||
await MainActor.run {
|
||||
isLoading = false
|
||||
dismiss()
|
||||
onSave()
|
||||
}
|
||||
} catch {
|
||||
let errorMessage = error.localizedDescription
|
||||
await MainActor.run {
|
||||
isLoading = false
|
||||
self.error = errorMessage
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
// TimeEntryFormView.swift — replaced by TimeEntryDetailSheet.swift
|
||||
import Foundation
|
||||
|
||||
62
ios/TimeTracker/TimeTracker/Models/ClientTarget.swift
Normal file
62
ios/TimeTracker/TimeTracker/Models/ClientTarget.swift
Normal file
@@ -0,0 +1,62 @@
|
||||
import Foundation
|
||||
|
||||
// MARK: - Client Target
|
||||
|
||||
struct ClientTarget: Codable, Identifiable, Equatable {
|
||||
let id: String
|
||||
let clientId: String
|
||||
let clientName: String
|
||||
let userId: String
|
||||
let weeklyHours: Double
|
||||
let startDate: String // "YYYY-MM-DD"
|
||||
let createdAt: String
|
||||
let updatedAt: String
|
||||
let corrections: [BalanceCorrection]
|
||||
|
||||
// Computed balance fields returned by the API
|
||||
let totalBalanceSeconds: Int
|
||||
let currentWeekTrackedSeconds: Int
|
||||
let currentWeekTargetSeconds: Int
|
||||
let weeks: [WeekBalance]
|
||||
}
|
||||
|
||||
// MARK: - Week Balance
|
||||
|
||||
struct WeekBalance: Codable, Identifiable, Equatable {
|
||||
var id: String { weekStart }
|
||||
let weekStart: String // "YYYY-MM-DD"
|
||||
let weekEnd: String
|
||||
let trackedSeconds: Int
|
||||
let targetSeconds: Int
|
||||
let correctionHours: Double
|
||||
let balanceSeconds: Int
|
||||
}
|
||||
|
||||
// MARK: - Balance Correction
|
||||
|
||||
struct BalanceCorrection: Codable, Identifiable, Equatable {
|
||||
let id: String
|
||||
let date: String // "YYYY-MM-DD"
|
||||
let hours: Double
|
||||
let description: String?
|
||||
let createdAt: String
|
||||
}
|
||||
|
||||
// MARK: - Input Types
|
||||
|
||||
struct CreateClientTargetInput: Codable {
|
||||
let clientId: String
|
||||
let weeklyHours: Double
|
||||
let startDate: String // "YYYY-MM-DD", must be a Monday
|
||||
}
|
||||
|
||||
struct UpdateClientTargetInput: Codable {
|
||||
let weeklyHours: Double?
|
||||
let startDate: String?
|
||||
}
|
||||
|
||||
struct CreateBalanceCorrectionInput: Codable {
|
||||
let date: String // "YYYY-MM-DD"
|
||||
let hours: Double
|
||||
let description: String?
|
||||
}
|
||||
@@ -1,6 +1,29 @@
|
||||
{
|
||||
"images" : [
|
||||
{
|
||||
"filename" : "app_icon.png",
|
||||
"idiom" : "universal",
|
||||
"platform" : "ios",
|
||||
"size" : "1024x1024"
|
||||
},
|
||||
{
|
||||
"appearances" : [
|
||||
{
|
||||
"appearance" : "luminosity",
|
||||
"value" : "dark"
|
||||
}
|
||||
],
|
||||
"idiom" : "universal",
|
||||
"platform" : "ios",
|
||||
"size" : "1024x1024"
|
||||
},
|
||||
{
|
||||
"appearances" : [
|
||||
{
|
||||
"appearance" : "luminosity",
|
||||
"value" : "tinted"
|
||||
}
|
||||
],
|
||||
"idiom" : "universal",
|
||||
"platform" : "ios",
|
||||
"size" : "1024x1024"
|
||||
|
||||
Binary file not shown.
|
After Width: | Height: | Size: 918 KiB |
@@ -29,18 +29,38 @@ struct StatCard: View {
|
||||
}
|
||||
|
||||
extension TimeInterval {
|
||||
/// Formats as a clock string used for the live timer display: "1:23:45" or "05:30".
|
||||
var formattedDuration: String {
|
||||
let hours = Int(self) / 3600
|
||||
let minutes = (Int(self) % 3600) / 60
|
||||
let seconds = Int(self) % 60
|
||||
|
||||
let totalSeconds = Int(self)
|
||||
let hours = totalSeconds / 3600
|
||||
let minutes = (totalSeconds % 3600) / 60
|
||||
let seconds = totalSeconds % 60
|
||||
|
||||
if hours > 0 {
|
||||
return String(format: "%d:%02d:%02d", hours, minutes, seconds)
|
||||
} else {
|
||||
return String(format: "%02d:%02d", minutes, seconds)
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
/// Human-readable duration used in lists and cards: "3h 48min", "45min", "< 1min".
|
||||
var formattedShortDuration: String {
|
||||
let totalSeconds = Int(self)
|
||||
let hours = totalSeconds / 3600
|
||||
let minutes = (totalSeconds % 3600) / 60
|
||||
|
||||
if hours > 0 && minutes > 0 {
|
||||
return "\(hours)h \(minutes)min"
|
||||
} else if hours > 0 {
|
||||
return "\(hours)h"
|
||||
} else if minutes > 0 {
|
||||
return "\(minutes)min"
|
||||
} else {
|
||||
return "< 1min"
|
||||
}
|
||||
}
|
||||
|
||||
/// Formats as hours with one decimal place, e.g. "3.8h". Used by the widget.
|
||||
var formattedHours: String {
|
||||
let hours = self / 3600
|
||||
return String(format: "%.1fh", hours)
|
||||
|
||||
@@ -4,7 +4,7 @@
|
||||
<dict>
|
||||
<key>com.apple.security.application-groups</key>
|
||||
<array>
|
||||
<string>group.simonfranken.timetracker</string>
|
||||
<string>group.com.simonfranken.timetracker.app</string>
|
||||
</array>
|
||||
</dict>
|
||||
</plist>
|
||||
|
||||
@@ -31,38 +31,24 @@ struct RootView: View {
|
||||
|
||||
struct MainTabView: View {
|
||||
@State private var selectedTab = 0
|
||||
|
||||
|
||||
var body: some View {
|
||||
TabView(selection: $selectedTab) {
|
||||
DashboardView()
|
||||
.tabItem {
|
||||
Label("Dashboard", systemImage: "chart.bar")
|
||||
}
|
||||
.tabItem { Label("Dashboard", systemImage: "chart.bar") }
|
||||
.tag(0)
|
||||
|
||||
|
||||
TimerView()
|
||||
.tabItem {
|
||||
Label("Timer", systemImage: "timer")
|
||||
}
|
||||
.tabItem { Label("Timer", systemImage: "timer") }
|
||||
.tag(1)
|
||||
|
||||
|
||||
TimeEntriesView()
|
||||
.tabItem {
|
||||
Label("Entries", systemImage: "clock")
|
||||
}
|
||||
.tabItem { Label("Entries", systemImage: "calendar") }
|
||||
.tag(2)
|
||||
|
||||
ProjectsView()
|
||||
.tabItem {
|
||||
Label("Projects", systemImage: "folder")
|
||||
}
|
||||
|
||||
SettingsView()
|
||||
.tabItem { Label("Settings", systemImage: "gearshape") }
|
||||
.tag(3)
|
||||
|
||||
ClientsView()
|
||||
.tabItem {
|
||||
Label("Clients", systemImage: "person.2")
|
||||
}
|
||||
.tag(4)
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
@@ -4,7 +4,7 @@
|
||||
<dict>
|
||||
<key>com.apple.security.application-groups</key>
|
||||
<array>
|
||||
<string>group.simonfranken.timetracker</string>
|
||||
<string>group.com.simonfranken.timetracker.app</string>
|
||||
</array>
|
||||
</dict>
|
||||
</plist>
|
||||
|
||||
Reference in New Issue
Block a user