Problem/Motivation

There is an extra comma rendered in email template for which user greetings are disabled.

Steps to reproduce

- Install Open Social
- Go to admin/config/opensocial/swiftmail and add for example "register_no_approval_required" under "Disabled email message keys with user greeting"
- Sign up with new account so you will trigger sending of email with template "register_no_approval_required"
- The email will be send with an extra comma just below the header

screenshot

Proposed resolution

Do not render heading in email template when it is empty.

Remaining tasks

This changes need to be applied also for related issue #3259374: Create a new template for the new email engine

User interface changes

N/A

API changes

N/A

Data model changes

N/A

CommentFileSizeAuthor
Screenshot 2023-07-14 at 16.30.50.png53.43 KBzanvidmar

Issue fork socialbase-3374705

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

zanvidmar created an issue. See original summary.

zanvidmar’s picture

Title: Remove heading in swiftmailer when it is empty » Remove heading in swiftmailer email template when heading it is empty

zanvidmar’s picture

Issue summary: View changes
Status: Active » Needs review
zanvidmar’s picture

Status: Needs review » Reviewed & tested by the community

  • c9b9507d committed on 2.5.x
    Issue #3374705 by zanvidmar: Remove heading in swiftmailer email...
robertragas’s picture

Status: Reviewed & tested by the community » Fixed
zanvidmar’s picture

Status: Fixed » Reviewed & tested by the community
zanvidmar’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

tbsiqueira’s picture