Skip to content

Conversation

@ergo
Copy link
Member

@ergo ergo commented Dec 7, 2016

This fixes #80 problem.

@ergo ergo changed the title Fixes #80 Fixes problem with incorrect arguments passed to constructors #80 Dec 7, 2016
@mmerickel
Copy link
Member

The issue is (more than likely) that you are pinned to repoze.sendmail 4.1 which does not support this feature, however 4.2 does. I think you should instead lobby for a release to repoze.sendmail for whatever bug you are worried about, or you should pin pyramid_mailer as well.

@papachoco
Copy link

I monkey-patched repoze-sendmail since it has been more than a year since we have had a release of that package

@tseaver
Copy link
Member

tseaver commented Dec 7, 2016

@ergo
Copy link
Member Author

ergo commented Dec 7, 2016

Indeed , closing this for now.

@ergo ergo closed this Dec 7, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

QueuedMailDelivery nor DirectMailDelivery do not accept transaction_manager keyword

4 participants