Skip to content

Fix Rails autoloading, add ConsentToTrack option that defaults to true (required by CM)#4

Open
le-martre wants to merge 32 commits into
madetech:masterfrom
le-martre:master
Open

Fix Rails autoloading, add ConsentToTrack option that defaults to true (required by CM)#4
le-martre wants to merge 32 commits into
madetech:masterfrom
le-martre:master

Conversation

@le-martre
Copy link
Copy Markdown

Hello!

In the case that someone is still checking on this sometimes, here is a PR that unbreaks the gem.

Gem autoloading in Rails was not functionning anymore, and it wasn't possible to send SmartEmails anymore because of the new "ConsentToTrack" required attribute in CampaignMonitor's API with createsend 5.0.0.

I decided to put it at "yes" by default, based on this part of the Campaign Monitor API doc :

If an email address has no value stored for ConsentToTrack, it is assumed that the subscriber has given consent.

The changes are not the cleanest but I've already spent quite some time on this and I'd like to get going on my actual product!

If I get no answer in a while I'll probably just publish the gem to RubyGems myself (no biggie), because it doesn't seem to be on there anymore!

Have a great day

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.

1 participant