Skip to content
This repository has been archived by the owner on Feb 8, 2018. It is now read-only.

Add Mandrill #2313

Merged
merged 2 commits into from
Apr 22, 2014
Merged

Add Mandrill #2313

merged 2 commits into from
Apr 22, 2014

Conversation

clone1018
Copy link
Contributor

This PR is the initial stage of #2312

Not ready yet.

Example inside simplate:

message = {
 'from_email': '[email protected]',
 'from_name': 'Luke Strickland',
 'html': '<p>Hello world</p>',
 'subject': 'Mandrill Test',
 'tags': ['password-resets'],
 'text': 'Hello',
 'to': [{'email': '[email protected]',
         'name': 'Luke Strickland',
         'type': 'to'}]
}

result = website.mail.messages.send(message=message, async=False)

@clone1018 clone1018 mentioned this pull request Apr 22, 2014
5 tasks
@seanlinsley
Copy link
Contributor

Tagging this with the 'work in progress' label

@clone1018
Copy link
Contributor Author

Thanks @seanlinsley, totally meant to do that.

@clone1018
Copy link
Contributor Author

There's some other things that need to happen here. I'll join everyone in IRC tomorrow to figure it out.

@clone1018
Copy link
Contributor Author

Can I get a code standard/properness review @whit537 ?

chadwhitacre added a commit that referenced this pull request Apr 22, 2014
@chadwhitacre chadwhitacre merged commit 4299841 into master Apr 22, 2014
@chadwhitacre chadwhitacre deleted the add-mandrill branch April 22, 2014 19:58
@chadwhitacre
Copy link
Contributor

Looks good as far as it goes.

👍

@chadwhitacre
Copy link
Contributor

IRC, including design of follow-on development.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants