Skip to content
This repository was archived by the owner on Aug 15, 2025. It is now read-only.

Conversation

@NathanEdg
Copy link
Member

@NathanEdg NathanEdg commented Aug 12, 2025

Summary

Addition of docs, admin starter, attendance system

Changes

  • Code changes
  • Documentation updates
  • Tests added or updated
  • Other (describe):

Checklist

  • I have read the CONTRIBUTING.md
  • Linting passes locally
  • No breaking changes unless documented
  • Documentation updated

PillageDev and others added 30 commits August 10, 2025 12:58
Update staging branch to production
- Unify and clarify environment variable usage across apps - Replace
subdomain-based path config with explicit app URLs - Update Next.js app
ports for landing, docs, learn, and scout - Add .env.example for easier
setup - Improve cookie and cross-domain config in auth - Fix login
redirect and magic link callback handling - Update dependencies and
lockfile
Copilot AI review requested due to automatic review settings August 12, 2025 00:34
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This pull request adds a comprehensive attendance tracking system along with documentation and admin functionality. The PR introduces a new attendance application as a Progressive Web App (PWA) with user check-in/check-out capabilities, administrative dashboard for managing attendance records, and configurable shop days.

  • Adds attendance tracking PWA with user authentication integration
  • Implements admin dashboard with user management and attendance editing capabilities
  • Introduces automated attendance processing via cron jobs

Reviewed Changes

Copilot reviewed 80 out of 82 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
packages/env/src/env.shared.ts Updates environment configuration to support multiple service URLs
packages/db/src/db/platforms/attendance.ts Adds database schema for attendance tracking and shop days
apps/attendance/ Complete new attendance tracking application with PWA capabilities
apps/auth/components/login-form.tsx Updates login flow to work with new URL structure
packages/config/src/paths.config.ts Refactors path configuration to use explicit app URLs
Comments suppressed due to low confidence (1)

apps/attendance/app/admin/dashboard/_components/user-detail/user-detail-sheet.tsx:67

  • Using @ts-expect-error to suppress type checking for the 'role' field indicates a type definition issue. Consider updating the database schema types to include the 'role' field instead of suppressing the error.
    }

@PillageDev PillageDev merged commit 8190214 into production Aug 12, 2025
12 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants