Skip to content

Custom SMTP and Templates - #53

Open
lohanidamodar wants to merge 12 commits into
mainfrom
feat-custom-smtp-and-email-templates
Open

Custom SMTP and Templates#53
lohanidamodar wants to merge 12 commits into
mainfrom
feat-custom-smtp-and-email-templates

Conversation

@lohanidamodar

@lohanidamodarlohanidamodar commented Mar 6, 2023•
edited
Loading

Copy link
Copy Markdown
Member

What does this PR do?

RFC for

  • custom SMTP per project
  • Custom Email templates
  • Custom SMS templates

Test Plan

(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work.)

Related PRs and Issues

(If this PR is related to any other PR or resolves any issue or related to any issue link all related PR and issues here.)

eldadfux
eldadfux requested changes Mar 6, 2023
View reviewed changes
Comment thread022-custom-smtp-and-email-templates.md Outdated
Comment thread022-custom-smtp-and-email-templates.md Outdated
Comment thread022-custom-smtp-and-email-templates.md Outdated
Comment thread022-custom-smtp-and-email-templates.md
@eldadfuxeldadfux mentioned this pull request Mar 6, 2023
Closed
2 tasks
eldadfux
eldadfux approved these changes Mar 8, 2023
View reviewed changes
Comment thread022-custom-smtp-and-email-templates.md Outdated
lohanidamodar
lohanidamodar commented Mar 9, 2023
View reviewed changes
Comment thread022-custom-smtp-and-email-templates.md Outdated
lohanidamodar
lohanidamodar commented Mar 13, 2023
View reviewed changes
Comment thread022-custom-smtp-and-email-templates.md
**PATCH /v1/projects/:projectId/template/sms/:type/:locale** - updates custom SMS templates

- **type** : string - required - template type
- **locale** : string - required (eg: en_us) - locale of the template

@lohanidamodarlohanidamodarMar 13, 2023

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

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

@eldadfux I'm having problem with the full locale en_us our locale library only accepts en. What's the purpose of having this full?

@eldadfuxeldadfuxMar 13, 2023

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

There are differences between en_us and en_uk, some might want to be more accurate.

@pietrodicaprio

pietrodicaprio commented Aug 28, 2023

Copy link
Copy Markdown

It would be awesome to have this merged, moreover would be even better to have it available in AW Cloud in order to send the emails from the project's domain instead of the global team@appwrite.io

@stnguyen90

stnguyen90 commented Oct 5, 2023

Copy link
Copy Markdown

@eldadfux can you merge this now or is there something else left to do?

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@eldadfuxeldadfuxeldadfux approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@lohanidamodar@pietrodicaprio@stnguyen90@eldadfux