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

I had to remove these lines since they were actually breaking the desired functionality. Can you review and merge? #8

Merged
merged 2 commits into from May 14, 2013

Commits on Dec 16, 2012

  1. Removed unnecessary lines

    These lines are actually conflicting with the previous statement that merges the class level defaults with the method arguments.
    Adam Kenger committed Dec 16, 2012
    Copy the full SHA
    24ec6a2 View commit details
    Browse the repository at this point in the history

Commits on Dec 17, 2012

  1. More issues with unique_args

    The first lines were removed as they were redundant but didn't default args to a hash.
    
    In the event that the line is not called from a mail method at all, we default in the json builder so that the code runs.
    
    The behavior is now as intended.  Args can be set globally or at method level.  At method will override global.
    Adam Kenger committed Dec 17, 2012
    Copy the full SHA
    238103c View commit details
    Browse the repository at this point in the history