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 11, 2025

This pull request introduces the initial setup for the new attendance app within the monorepo, including configuration, dependencies, environment variables, and support for PWA features. The changes establish the foundational structure for the app, integrating it with shared packages and setting up essential build and runtime settings.

New App Initialization and Configuration

  • Added apps/attendance/package.json to define the app's dependencies, scripts, and integration with internal workspace packages.
  • Created apps/attendance/next.config.ts with custom Next.js configuration, including support for PWA, optimized imports, image remote patterns, and experimental features.
  • Added .env.example entry for NEXT_PUBLIC_ATTENDANCE_BASE_URL to support local development of the attendance app.
  • Added apps/attendance/drizzle.config.ts for Drizzle ORM configuration, enabling PostgreSQL database connectivity using environment variables.

Build and Runtime Support

  • Added apps/attendance/.gitignore to exclude build artifacts, environment files, and other unnecessary files from version control.
  • Added apps/attendance/postcss.config.mjs to configure PostCSS with Tailwind CSS support.

Progressive Web App (PWA) Features

  • Added apps/attendance/public/manifest.json to define PWA metadata, icons, and display properties.
  • Added apps/attendance/public/sw.js for service worker registration and caching strategies.
  • Added apps/attendance/public/_headers to set cache-control headers for static assets.## Summary
    Explain the purpose of this PR. What problem does it solve or feature does it add?

Related Issues

Closes #21

Checklist

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

Copilot AI review requested due to automatic review settings August 11, 2025 21:21

This comment was marked as outdated.

@NathanEdg NathanEdg requested a review from Copilot August 11, 2025 21:44
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 establishes the foundational infrastructure for a new attendance tracking app within the monorepo. The changes integrate the attendance app with the existing shared packages, set up database schema for attendance records, and implement basic PWA functionality.

  • Adds new attendance app with Next.js configuration, database schema, and environment setup
  • Integrates attendance schema with shared database package and authentication system
  • Implements Progressive Web App features including service worker and manifest

Reviewed Changes

Copilot reviewed 64 out of 66 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
packages/env/src/env.shared.ts Adds ATTENDANCE_BASE_URL environment variable configuration
packages/db/src/index.ts Integrates attendance schema into shared database exports
packages/db/src/db/platforms/attendance.ts Defines attendance and shop_days database tables
packages/db/drizzle.config.ts Adds attendance schema to database migration configuration
packages/auth/src/auth.ts Updates CORS origins to include attendance app URL
apps/attendance/* Complete attendance app implementation with components, pages, and configuration

@NathanEdg NathanEdg requested a review from PillageDev August 11, 2025 22:26
@PillageDev PillageDev merged commit 6aac2e1 into staging Aug 11, 2025
11 checks passed
@PillageDev PillageDev deleted the feat/attendance-system branch August 11, 2025 23:53
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