Git repo, pull requests and 'merge_language'

  • Hey MC

    The client / install was already using the plugin so it makes sense to just use wpMandrill::mail. However, we need {{ handlebars }} in the templates on the Mandrill end.

    In short I added merge_language to static function mail(). Simple. It works ?? But now I’m afraid an update to this plugin will over-write that enhancement. Is there a repo for this plugin and I can submit a pull request?

    What’s the best way to get my update into your code?

    Thanks
    mfs

    https://www.ads-software.com/plugins/wpmandrill/

Viewing 1 replies (of 1 total)
  • Thread Starter ChiefAlchemist

    (@chiefalchemist)

    1) So i take it merge_language isn’t in this latest release ??

    2) I’m curious. Is there a reason why this plugin doesn’t use the official Mandrill library for PHP? I’m looking at that and this plugin and it feels like the plugin could / should just be a wrapper (and some other WP-centric stuff) for the PHP class(es).

    3) Any chance of getting either of these onto GitHub? I’d prefer to do a pull request as opposed to keep rehacking the plugin with my merge_language fixes (which btw are minor at best.)

Viewing 1 replies (of 1 total)
  • The topic ‘Git repo, pull requests and 'merge_language'’ is closed to new replies.