Only allow users who turned 13 before this year began to opt in. #6132
Conversation
This file contains 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
|
👍 |
1 similar comment
|
👍 |
stephensanchez
pushed a commit
that referenced
this pull request
Dec 4, 2014
Only allow users who turned 13 before this year began to opt in.
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.
This is a fix for defect ECOM-719. We only track the year someone was born, not the month / day. We'll be safe by only allowing those users who turned 13 before this year began to opt in for organization emails, to avoid someone at 12 years, 1 month, to get included in the acceptable group.
Example:
Born December 2nd, 2002. Current Date January 3rd, 2015. Turns 13 on December 2nd, 2015, but is granted access 11 months early.
Now we run the risk of someone being 13 years old having to wait 11 months to opt in for emails (reversing the example)
@rlucioni @wedaly @griffresch