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

feat(api): support for custom email sender trigger #234

Merged

Conversation

kylefix
Copy link
Contributor

@kylefix kylefix commented May 18, 2022

hello sir,

this PR adds (partial) support for the CustomEmailSender trigger for these trigger sources:

CustomEmailSender_SignUp
CustomEmailSender_ResendCode 
CustomEmailSender_ForgotPassword
CustomEmailSender_UpdateUserAttribute
CustomEmailSender_VerifyUserAttribute
CustomEmailSender_AdminCreateUser

also partially addresses #102 by saving the e-mail as a UserAttribute when the UserPool is configured with UsernameAttributes: ['email']

lmk if there is anything I overlooked

Thank you for this great tool, really awesome codebase!

@jagregory
Copy link
Owner

Thanks @kylefix this looks great! Sorry for the slow response, GitHub seems to periodically stop sending me notifications about this repo for some reason 🤷

I'll have a proper review tonight and will merge, but looks good so far. Thanks

@jagregory jagregory merged commit 8dcaf10 into jagregory:master May 30, 2022
@github-actions
Copy link

🎉 This PR is included in version 3.18.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants