Skip to content

LG-11209: Log event when new registered account uses disposable email domain#9747

Merged
jc-gsa merged 5 commits intomainfrom
LG-11209-log-disposable-email-registration
Dec 13, 2023
Merged

LG-11209: Log event when new registered account uses disposable email domain#9747
jc-gsa merged 5 commits intomainfrom
LG-11209-log-disposable-email-registration

Conversation

@jc-gsa
Copy link
Contributor

@jc-gsa jc-gsa commented Dec 12, 2023

🎫 Ticket

LG-11209

🛠 Summary of changes

Adds the property named disposable_email_domain to the event User registration: complete when the user registers with a known temporary email domain.

📜 Testing Plan

  • Create a new account with a domain populated in the model DisposableDomain
  • Check for the property disposable_email_domain on the event User registration: complete

@jc-gsa jc-gsa force-pushed the LG-11209-log-disposable-email-registration branch from 8dfc63f to 647fa21 Compare December 12, 2023 18:33
changelog: Internal, Analytics, Add tracking for disposable email domains
@jc-gsa jc-gsa force-pushed the LG-11209-log-disposable-email-registration branch from 647fa21 to f7a7067 Compare December 12, 2023 18:57
@jc-gsa jc-gsa requested a review from a team December 12, 2023 19:14
Copy link
Contributor

@aduth aduth left a comment

Choose a reason for hiding this comment

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

LGTM 👍

Copy link
Contributor

@aduth aduth left a comment

Choose a reason for hiding this comment

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

LGTM 👍

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