Repository navigation
Conversation
✅ Deploy Preview for footballplayer-online ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Hype76
changed the title
Hype76
changed the title
…ach-reminder-options-20261002
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.
Adds optional shared-team reminders for unanswered MATCH/TRAINING availability, squad selection alerts and Coach-selectable deadline strictness. All new automation defaults OFF and applies only to future confirmed deliveries. Automatic decisions retain distinct provenance; explicit answers win. Simon-approved durable retirement prevents last-responder unlink/relink from restoring an old deadline or replaying a notice. Existing Team Admin and scoped Coach/rank20+ including Assistant Coach access is preserved.
The current bounded repair fixes the installed adult self-recipient token guards: they required a public.users profile rejected by the adult-link validator. Both guards now resolve confirmed/current Auth identity plus verified active adult link and current team membership, retaining the Parent/contact/event/token/expiry branches, bearer-token semantics and existing ACLs. Actual MATCH/TRAINING UI wrappers save late explicit replies while keeping automatic history immutable. No broader account-model/UI/mobile/native changes.
Exact source d4fe0f4; Git/LF migration SHA25640bb7b9f5be25142541745fbd737e5b5d963a8217277e3a33cbf0b16a3b224ed. Prior017ee788 review identity is superseded; fresh independent review must assess this exact candidate. The prior narrow ledger/gate ACL correction remains intact.
Validation:77 focused tests,38 actual PostgreSQL session scenarios with797 distinct backends including queries/polls,21 full-closure authority checks,13 expanded default-OFF baseline comparisons,96 privilege assertions/9 denied ledger mutations, lint, migration441/secret2485 gates pass. All40 catalog-matching baseline triggers plus16 candidate hooks stay active. Actual adult late corrections, repeat replies, another user/team, revoked/unconfirmed/banned authority, removed membership, expired/cancelled events and OFF behavior were tested without forbidden profiles or weakened constraints. Seven response functions retain owners/security/tested grants; only the two intended guard definitions differ. Temporary PostgreSQL stopped, listener0/PID false.
Configured CI at this exact head: all9 configured jobs are terminal success. https://github.com/Hype76/footballdev/actions/runs/37069585449 and https://github.com/Hype76/footballdev/actions/runs/37069585519 . No gates weakened/skipped.
Limits: synthetic claims/local adapters/mock providers; local PostgreSQL18.3 versus installed17.6; bounded schema/function/trigger closure, extra UNIQUE(email), omitted guardian/unrelated internal FK referents. No deployed Auth/JWT/HTTP/PostgREST/native/device/provider acceptance/load proof. Independent review and separately approved dormant installation, stored OFF/ACL/RLS/function/trigger verification and remaining release checks are required before separately approved activation. Measured scheduler cadence is an operational verification, not a new user product decision.
Simon deferred app publication/OTA through Sunday4October2026 (1790978188.262589; parent1790978225.105599). Monday5October is a review point, not automatic release approval. No merge, production deployment, installation, activation, messages, attendance mutation or app publication. Native rebuild/store submission remain outside scope. Existing automatic draft-preview CI is not a production/app release. Preserve archives/worktrees/unrelated PR158/165/166. Parent owns Activity Log; denied append not retried. Final evidence and prepared readable12-column parent handoff are ready.
Verified final evidence: https://chatgpt.com/api/library/files/libfile_36edc8d4f98c81919488bb58266f5d8a/download . SHA2565A9061EA45ADFA8DBB13D25E2ED03F8461F33EEAEB348C93A52E9526A4E63EB1;482851bytes/67entries, upload/download readback verified. Includes exact tracked source, before-fix actual-wrapper denial and after-fix corrected path, full configured CI receipts, catalog/privilege/trigger/OFF evidence and prepared readable12-column parent log handoff. No denied append retry.