At the moment, module uses template names to identity Mandrill Email Templates. That is error-prone way to do it since template names can be accidentally changed on Mandrill web site.

This is what Mandrill's API documentation says about IDs (in /messages/send-template.json query):
template_name*
the immutable name or slug of a template that exists in the user's account. For backwards-compatibility, the template name may also be used but the immutable slug is preferred.

It would be a good idea to start using slugs.

CommentFileSizeAuthor
#2 mandrill-use-slug-instead-of-template-name-2243459-1.patch1.14 KBvaartio
#1 mandrill-use-slug-instead-of-template-name-707484-1.patch1.14 KBvaartio

Comments

vaartio’s picture

vaartio commented
Version: 7.x-2.x-dev » 7.x-1.x-dev
StatusFileSize
new mandrill-use-slug-instead-of-template-name-707484-1.patch1.14 KB

Attached is a patch that does the trick.

Btw, it is a bit confusing that there are dev releases for 1.x and 2.x branches, but 2.x branch actually doesn't exist. I assume that they have been combined and 1.x is the HEAD of development. Please, correct if I'm wrong.

vaartio’s picture

vaartio commented
StatusFileSize
new mandrill-use-slug-instead-of-template-name-2243459-1.patch1.14 KB

Changed correct issue number in the patch file name. Here's a new one.

  • Commit 809ed96 on 7.x-1.x authored by vaartio, committed by gcb: 
    #2243459 - Use slug instead of template display name.
gcb’s picture

gcb
he/him
Location Portland, OR
commented
Status: Active » Closed (fixed)

Thanks! Applied in dev.

  • Commit 809ed96 on 7.x-1.x, 7.x-2.x authored by vaartio, committed by gcb: 
    #2243459 - Use slug instead of template display name.