Repository navigation
Connect teacher schedule and roster to attendance check-in - #100
Draft
TaylorONeal wants to merge 4 commits into
Draft
TaylorONeal wants to merge 4 commits into
TaylorONeal wants to merge 4 commits into
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Replace teacher schedule fixtures with assigned-class reads, roster display, and server-confirmed attendance check-in. Narrow database RPCs enforce active staff membership and assignment and reuse existing booking/time/status checks. Add member cancellation retry coverage and awaited sign-out error handling.
Validation: 526 application and 32 script tests, typecheck, production build, and 23 browser tests pass, including mobile attendance and lost-response recovery. Full local PostgreSQL16 migration/SQL suite passes in a fresh isolated cluster: 251 PASS lines, including all 15 teacher authorization cases, payments, attribution and concurrency races. Temporary server and data were cleaned up. CI run39176946297 passed all four jobs (database, edge typecheck, browser E2E, web checks).
The initial combined two-app local runner reused a cluster-global service_role with incompatible fixture attributes. Running each app in its own fresh cluster fixed the test environment; no payment code or production role change was needed.
Transactional migration00044 and read-only rollout verification scripts are included. The teacher frontend must follow the reviewed database migration. No hosted migration, credential change, merge, or manual deployment has been performed.