-
Notifications
You must be signed in to change notification settings - Fork 54
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
docs: extend SMTP providers guide #3074
Conversation
SMTP language edit
Update Smtp providers
…_providers # Conflicts: # astro/src/content/docs/customize/email-and-messages/configure-email.mdx
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Doc looks good, if we could add a few extra in vale
Sinch | ||
Brevo | ||
MailerSend | ||
Mailchimp |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we add a few more here?
mailcow
Haraka
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you also probably want to merge in main
because the location of accept.txt
moved due to this PR: #3178
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
totally missed this as well, @sixhobbits if you could merge and update these in
config/vale/styles/config/vocabularies/FusionAuth/accept.txt
@sixhobbits I think we are probably hitting our own updates here but can you merge main again? |
@sixhobbits conflicts 🙏 |
No description provided.