chore: remove COPPA age requirement warnings and references#49
Merged
Conversation
- Update frontend .env.example to use login.dev.kernelworx.app - Change deploy.sh to discover the Cognito custom domain from OpenTofu output first, then AWS CLI describe-user-pool, instead of the legacy CloudFormation lookup - Add cognito_domain output to tofu/application/environments/dev/main.tf for parity with prod
Removes COPPA-related copy from the UI, docs, and project guidance: - LandingPage, LoginPage, SignupPage warning alerts/blocks - PrivacyPolicyPage Children's Privacy (COPPA) section - README, branding guide, and copilot instructions references - Related LandingPage test The signup form age-confirmation checkbox remains functional.
Contributor
There was a problem hiding this comment.
Pull request overview
This PR removes COPPA/children’s privacy messaging from the UI and documentation, while also updating deployment/infrastructure outputs to better support discovering the Cognito hosted UI domain during frontend deployment.
Changes:
- Removed COPPA-related warning copy from public-facing frontend pages, tests, and documentation.
- Updated frontend deployment logic to derive the Cognito domain via OpenTofu outputs (with AWS CLI / constructed fallbacks).
- Added/standardized OpenTofu outputs for the Cognito domain and updated the frontend
.env.exampleto reflect the custom domain.
Reviewed changes
Copilot reviewed 14 out of 14 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| tofu/application/environments/dev/main.tf | Reformatting + adds cognito_domain output for deployment tooling. |
| tests/e2e/test_smoke_signup.py | Rewords age checkbox comment (COPPA reference removed). |
| README.md | Removes COPPA warning bullet(s) from project overview/compliance section. |
| frontend/tests/LandingPage.test.tsx | Removes assertion for COPPA warning block that no longer renders. |
| frontend/src/pages/SignupPage.tsx | Removes COPPA warning copy/component from signup UI (age checkbox remains). |
| frontend/src/pages/PrivacyPolicyPage.tsx | Removes “Children’s Privacy (COPPA)” section from privacy policy page. |
| frontend/src/pages/LoginPage.tsx | Removes COPPA warning alert from login page. |
| frontend/src/pages/LandingPage.tsx | Removes COPPA warning panel from landing page content. |
| frontend/deploy.sh | Switches Cognito domain discovery to prefer OpenTofu output with AWS CLI fallback. |
| frontend/.env.example | Updates example Cognito domain to match custom login domain pattern. |
| docs/branding/README.md | Removes COPPA compliance bullet from branding asset notes. |
| docs/BRANDING_GUIDE.md | Removes COPPA warning example section from branding guide. |
| docs/BRANDING_GUIDE.html | Removes COPPA-related CSS and example markup from branding guide HTML. |
| .github/copilot-instructions.md | Removes COPPA warning guidance from repo Copilot instructions. |
| fi | ||
| COGNITO_DOMAIN=$(aws cognito-idp describe-user-pool \ | ||
| --user-pool-id "$USER_POOL_ID" \ | ||
| --query 'UserPool.CustomDomain' \ |
Comment on lines
+95
to
+99
| # OpenTofu environment directory (for output lookups) | ||
| TOFU_DIR="$SCRIPT_DIR/../tofu/application/environments/$ENVIRONMENT" | ||
|
|
||
| # Get Cognito custom domain | ||
| # 1. Prefer OpenTofu output (matches the managed infrastructure config) |
Resolved conflicts: - frontend/src/pages/LandingPage.tsx - frontend/tests/LandingPage.test.tsx Kept main's rewritten landing page and tests, which no longer contain the COPPA warning.
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
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.
Removes COPPA-related copy from the UI, docs, and project guidance.
Changes:
The signup form age-confirmation checkbox remains functional.