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

Delete default builtin mail templates #983

Merged
merged 3 commits into from
Dec 19, 2018

Conversation

anttiSalmivaara
Copy link
Contributor

Other mail templates than those related to transactions can now be edited in console, so they are removed from the repository. At the same time the context variables are updated to match console and README is updated.

This directory contains the default templates for all mails sent by default as result of activity in
the marketplace.
This directory contains the default templates for mails defined in the default transaction process.
Templates for other notifications can be edited in Console.
Copy link
Contributor

Choose a reason for hiding this comment

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

We could stick to same terminology that is used here: https://flex-docs.sharetribe.com/email-templates.html. So built-in emails and transaction emails.

@anttiSalmivaara anttiSalmivaara force-pushed the delete-default-builtin-templates branch from c2e3260 to e353d7f Compare December 19, 2018 08:26
Copy link
Member

@rap1ds rap1ds left a comment

Choose a reason for hiding this comment

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

👍

@rap1ds rap1ds merged commit 043ddaf into master Dec 19, 2018
@rap1ds rap1ds deleted the delete-default-builtin-templates branch December 19, 2018 10:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants