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

Remove plain text parts from emails #942

Merged
merged 3 commits into from
Oct 23, 2018

Conversation

anttiSalmivaara
Copy link
Contributor

No description provided.

CHANGELOG.md Outdated
@@ -19,6 +19,8 @@ way to update this template, but currently, we follow a pattern:
[#939](https://github.com/sharetribe/flex-template-web/pull/939)
* [add] Added a small comment to documentation about the current state of code-splitting.
[#938](https://github.com/sharetribe/flex-template-web/pull/938)
* [remove] Removed plain text parts of email templates.
[#942](https://github.com/sharetribe/flex-template-web/pull/942)
Copy link
Contributor

Choose a reason for hiding this comment

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

@anttiSalmivaara we have kept this in chronological order, newest/current PR being the first bullet.

Copy link
Contributor

@Gnito Gnito left a comment

Choose a reason for hiding this comment

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

Looks good, but move the changelog line to be the first one (to maintain chronological order).

@anttiSalmivaara anttiSalmivaara force-pushed the remove-plain-text-parts-from-emails branch from fce5270 to b43dfc4 Compare October 23, 2018 11:30
@anttiSalmivaara anttiSalmivaara merged commit 83a18b7 into master Oct 23, 2018
@anttiSalmivaara anttiSalmivaara deleted the remove-plain-text-parts-from-emails branch October 23, 2018 11:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants