DynamoMTL / spree_chimpy

Spree/MailChimp Integration
BSD 3-Clause "New" or "Revised" License
35 stars 123 forks source link

Sync users from website to mailchimp #40

Open noff opened 10 years ago

noff commented 10 years ago

Gem had bug which did not sent new users to mailchimp during few weeks. We fixed it.

Now we want new users who not synced with mailchimp (changed their subscription status from false to true on website) to appears in Mailchimp list with subscribed status.

Task rake spree_chimpy:sync don't do this. No new users appears in Mailchimp.

bryanmtl commented 10 years ago

Hi @noff -

Sorry about this, we're aware of the issue. This commit is what "broke" it: https://github.com/DynamoMTL/spree_chimpy/commit/0445ec93e590a92929bb3748ced91130c022985e

This was done so that users not already appearing on the MC list are skipped over when orders are created. We're working on making this a configurable parameter. I'll update this issue as soon as that fix is in.

cc: @braidn @DynamoMTL/mickeyren

papayaah commented 10 years ago

Hi @noff

Out of curiosity can you post a gist of the fix you said you guys did?

DriesS commented 9 years ago

You could use a workaround for the moment

Spree::User.where(:subscribed => true).each do |user|
  user.subscribe
end