The users are not able to add translations for email body as email body textfield is not available for other languages except English.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

adhariwal created an issue. See original summary.

adhariwal’s picture

adhariwal’s picture

Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, 2: 2951261-2.patch, failed testing. View results

supertony’s picture

I noticed something similar with the [node:title] token.

If, for example, I use [node:title] token, only the original language node title will be used in the email.

This is somewhat confusing if my authors are renaming content for different languages.

larowlan’s picture

This is the right fix - not sure about the failing tests though

adhariwal’s picture

Me too, not sure if the functional test requires updating.

larowlan’s picture

Issue tags: +Release blocker
abu-zakham’s picture

Version: 8.x-1.0-alpha7 » 8.x-1.x-dev
Status: Needs work » Needs review
jibran’s picture

Status: Needs review » Needs work

We need an empty update hook here to refresh the config schema for existing websites.

Lendude’s picture

Version: 8.x-1.x-dev » 2.x-dev
Status: Needs work » Needs review
FileSize
908 bytes

Now with an update hook.

We can probably lose the 'Release blocker' tag since it didn't block much ;)

  • larowlan committed 0037837 on 2.x
    Issue #2951261 by adhariwal, Lendude: There is no way to add transaltion...
larowlan’s picture

Status: Needs review » Fixed
Issue tags: -Release blocker

Thanks! will cut a 2.3.3 with this

yeah I don't do 'release blocker' tags and more, I just release early and often

Status: Fixed » Closed (fixed)

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