Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

mad mimi mailer wants all our mails to send out via madmimi #4

Open
aka47 opened this issue May 11, 2010 · 3 comments
Open

mad mimi mailer wants all our mails to send out via madmimi #4

aka47 opened this issue May 11, 2010 · 3 comments

Comments

@aka47
Copy link

aka47 commented May 11, 2010

Hi,

with the update to the latest version, mad-mini wants to send out all emails via mad-mimi. Does not matter if they have mimi_ in their name or not.

What are we doing wrong, since your commit tries exactly to fix this...
We downgraded for now to 0.0.9 and when i have time i will have a closer look.

@madmimi
Copy link

madmimi commented May 11, 2010

Were you seeing this issue on 0.1.2.1?

@aka47
Copy link
Author

aka47 commented May 11, 2010

yes, 0.1.2.1. I did not try it in production, but our specs are failing because of it.

I another project, where every mail gets send out via mad-mimi, the problem was that it set the promotion not correct, because method_prefix was nil. (They use it still via inherintence not as mix-in.)

@mech
Copy link

mech commented May 15, 2010

I have similar problem. During the 0.0.8 version, I need to specify mimi_ prefix on my Mailer class and without the mimi_ prefix on my exact Promotion name, now I need to delete all my mimi_ prefix method. Is this the intended behavior?

previously

def mimi_notify_candidate
end

now, because the promotion name is "notify_candidate"

def notify_candidate
end

Can someone point me to where MadMimi has make such changes or some announcement of it :)

Thanks

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

No branches or pull requests

2 participants