Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: Changes for COPPA compliance removing YOB from profile #24

Merged
merged 3 commits into from
Aug 1, 2024

Conversation

HamzaIsrar12
Copy link

Description

To ensure the app complies with COPPA regulations - Issue

Requirements:

  • Remove the Year of Birth (YOB) field from the profile section
  • If learner's age is more than 13:
    • Display toggle for Limited and Full profile
    • Allow image upload in both Limited and Full profile
    • Display all fields in both Limited and Full profile, except YOB
    • Disable the fields in Limited profile
  • If learner's age is less than 13 or not provided (requiresParentalConsent = true):
    • Do not display toggle for Limited profile
    • Do not allow image upload, hide the icon
    • Display all fields but disable them

Note:

Instead of removing the code, I've commented the code, because post-MVP we might be moving these changes behind a feature flag.

We currently don't have specific design guidelines for this request. So the changes are approved by Moiz.


COPA.Compliance.mp4

Copy link

@omerhabib26 omerhabib26 left a comment

Choose a reason for hiding this comment

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

only a minor nit

@HamzaIsrar12
Copy link
Author

@k1rill @dixidroid Please have a look at it as well. 👍🏻

@k1rill k1rill merged commit 7eea3e4 into 2U/develop Aug 1, 2024
4 checks passed
@k1rill k1rill deleted the 2U/hamza/copa_compliance branch August 1, 2024 07:18
@HamzaIsrar12 HamzaIsrar12 self-assigned this Aug 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants