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

feat(admin-email): Add "Email Heading" as a customizable setting for all Emails #3110

Closed
mathetos opened this issue Apr 24, 2018 · 2 comments

Comments

@mathetos
Copy link
Member

commented Apr 24, 2018

User Story

As a Give Admin, I want to be able to fully customize my email content so that each email has a header that properly describes each form/campaign.

Current Behavior

I currently can customize the content of each email just fine; but I have to use a snippet in order to customize the email heading.

Expected Behavior

I expect to be able to customize the email heading as easily as I can customize the email content.

Possible Solution

Add the setting to the global email settings like so:
image

And the individual form edit screens like so:
image

Related

Tasks

  • Add the setting to the global email settings for each individual email
  • Add the setting to the form edit screen for each individual email
  • Ensure that email tags work in those settings so users can use {donor_name}, etc. in the email heading.
  • Ensure that there are default values for the email heading in case the setting is not customized.
@kevinwhoffman

This comment has been minimized.

Copy link
Member

commented May 21, 2018

@Sidsector9 To clarify, this is the text we are looking to customize at the top of each email.

image

Note that this field should be customizable in all emails from the Settings > Emails tab.

@ravinderk

This comment has been minimized.

Copy link
Collaborator

commented May 22, 2018

@mathetos @Sidsector9 We can create this as the default setting in Email API because most of our email notification support heading.

DevinWalker pushed a commit that referenced this issue May 23, 2018
Devin Walker
Merge pull request #3247 from Sidsector9/issue-3110
feat(admin-email): Add "Email Heading" as a customizable setting for all Emails #3110
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.