This repository has been archived by the owner on Feb 11, 2020. It is now read-only.
Updates for Rails 3.2.3 and CreateSend 1.0.3 #7
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
In Rails 3.2.3, the ActiveRecord whitelist_attributes is true. Rather than turning this off and losing the added security, I updated the prelaunch_subscriber model so that the email attribute can be mass assigned. Without this update, you will receive a mass assignment exception. Secondly, the old version of the CreateSend gem (~0.2) emits a MultiJSON parse error; upgrading to a version of CreateSend of at least 1.0.0 fixes this issue.
This is my first pull request so apologies if I've done it all wrong! I could not get the tests to run on my environment so I was not able to update them. Please let me know if there's things I could have done better for future reference. Thanks for all your work on this gem, it's made our lives a lot easier!