You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
With Rails 5.0.0.1 not supplying id option to expose results in Decent Exposure expecting to have a params object available, which of course doesn't exist in mailers.
Since DE is able to hook into the execution chain between the mailer class and instance action method being invoked, it should be possible to use naming conventions to pluck the needed parameters automatically without need for additional boilerplate assignments.
Sorry, I don't have time to dig into the issue and provide a patch right now. But would love to mention DE in my Action Mailer chapter in The Rails 5 Way if possible.
The text was updated successfully, but these errors were encountered:
With Rails 5.0.0.1 not supplying
id
option toexpose
results in Decent Exposure expecting to have a params object available, which of course doesn't exist in mailers.Since DE is able to hook into the execution chain between the mailer class and instance action method being invoked, it should be possible to use naming conventions to pluck the needed parameters automatically without need for additional boilerplate assignments.
Sorry, I don't have time to dig into the issue and provide a patch right now. But would love to mention DE in my Action Mailer chapter in The Rails 5 Way if possible.
The text was updated successfully, but these errors were encountered: