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

CRM-21078 Allow user to save and continue editing message template #10871

Merged
merged 3 commits into from
Aug 21, 2017

Conversation

mattwire
Copy link
Contributor

@mattwire mattwire commented Aug 17, 2017

Overview

Allow the user to continue editing a message template without exiting and then reopening the message template.
Buttons when editing message template are now (matching other workflows in CiviCRM, eg. create/edit event)
Save | Save and Done | Cancel

Additionally, cleanup FIXME, redundant code and improve performance of existing code.


Copy link
Contributor

@jitendrapurohit jitendrapurohit left a comment

Choose a reason for hiding this comment

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

Before this change, message template used to throw validation error on not filling the required field title -

image

This is no longer happening now and an empty template is saved with message The Message Template 'null' has been saved. To replicate - Apply this PR -> Add message template and save without filling any field.

@mattwire mattwire force-pushed the CRM-21078_message_template_form branch from de7252e to b93376a Compare August 18, 2017 15:14
@monishdeb
Copy link
Member

Tested, working fine.

@monishdeb monishdeb merged commit c15fad0 into civicrm:master Aug 21, 2017
@monishdeb
Copy link
Member

@mattwire in exchange can you please QA #10717 ?

@mattwire mattwire deleted the CRM-21078_message_template_form branch August 28, 2017 12:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants