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

Accept custom email_message and email_subject for code verification #48

Open
jfharden opened this issue Apr 12, 2021 · 2 comments
Open
Assignees
Labels
documentation Improvements or additions to documentation

Comments

@jfharden
Copy link
Contributor

The email configuration options for verification message customisation only accept email_message_by_link and email_subject_by_link. This means you cannot customise the email subject and message when using CONFIRM_BY_CODE type.

It would be good to also accept email_message and email_subject parameters so the verification message when using CONFIRM_BY_CODE could also be set.

@jfharden
Copy link
Contributor Author

I've just realised I can set them with the input email_verification_message and email_verification_subject. It would be good if they could also be part of the verification_message_template as email_message and email_subject too, although definitely not required and adds extra complexity (so maybe just a readme update to mention where the setting is)

@lgallard lgallard self-assigned this Apr 12, 2021
@lgallard lgallard added the documentation Improvements or additions to documentation label Apr 12, 2021
@rufusnufus
Copy link

rufusnufus commented May 18, 2022

Hello! I also have the problem with email_message.
According to the terraform docs about Cognito User Pool resource email_message attribute doesn't require to contain {username} placeholder. However, in the code you assigning to email_verification_message attribute the variable that must contain {username} and {####} placeholders. In our project, we can't pass the username to {username} placeholder in the email verification message. See first note here
image

It would be good to separate verification_message_template and invite_message_template attributes in different variables.

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

No branches or pull requests

3 participants