DynamoMTL / spree_chimpy

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

When Creating Segment, Ensure Only ID is Returned #62

Open DanielWright opened 9 years ago

DanielWright commented 9 years ago

api_call.static_segment_add returns a Hash, not the segment's ID, so when a segment is created, @segment_id can contain an unexpected Hash, which causes an exception to be thrown when, say, subscribing a customer. Bad news bears!

DanielWright commented 9 years ago

CI spec failures seem to have been caused by 4681d8a8b8a37fb8867abbcdac0fb11283d58c0e. Shall I update the specs to reflect the change?

joshnuss commented 9 years ago

Thanks ddubya!

Yes, updating that spec would be helpful (/cc: @braidn )