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

Add support for identity field for email verification jobs and tickets #293

Merged
merged 6 commits into from
Sep 28, 2020

Conversation

jimmyjames
Copy link
Contributor

Changes

Adds support for specifying the (optional) identity object when creating email verification tickets or jobs. This is needed when verifying emails when the primary is social, enterprise, or passwordless. Also needed to verify secondary identities.

References

Testing

In addition to unit tests, I've also verified the changes manually using a real tenant and users.

  • This change adds test coverage
  • This change has been tested on the latest version of the platform/language or why not

Checklist

@jimmyjames jimmyjames added this to the v1-Next milestone Sep 22, 2020
@jimmyjames jimmyjames requested a review from a team September 22, 2020 13:55
Copy link
Contributor

@lbalmaceda lbalmaceda left a comment

Choose a reason for hiding this comment

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

I've not checked the tests

@jimmyjames jimmyjames force-pushed the secondary-email-verification branch from c774839 to e2859ff Compare September 25, 2020 15:42
@jimmyjames jimmyjames force-pushed the secondary-email-verification branch from c9624cd to 66be7ab Compare September 28, 2020 19:29
@lbalmaceda lbalmaceda self-requested a review September 28, 2020 19:31
@jimmyjames jimmyjames merged commit ec8b6aa into master Sep 28, 2020
@jimmyjames jimmyjames modified the milestones: v1-Next, 1.22.0 Sep 28, 2020
@jimmyjames jimmyjames deleted the secondary-email-verification branch September 29, 2020 15:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants