Skip to content

Conversation

c-noonan
Copy link
Contributor

Per this ticket: https://segment.zendesk.com/agent/tickets/420194, our docs are pretty unclear on exactly what Iterable requires for the ecommerce events we map for customers. Based on https://github.com/segmentio/integrations/blob/master/integrations/iterable/lib/index.js#L163-L206 and https://github.com/segmentio/integrations/blob/master/integrations/iterable/lib/mapper.js, we pretty much just send the products array we are given to Iterable, but turns out, the endpoints we hit requires some basic fields in each product object: https://api.iterable.com/api/docs#commerce_trackPurchase and https://api.iterable.com/api/docs#commerce_updateCart. I tried to clarify these requirements better for less confusion in the future.

Per this ticket: https://segment.zendesk.com/agent/tickets/420194, our docs are pretty unclear on exactly what Iterable requires for the ecommerce events we map for customers. Based on https://github.com/segmentio/integrations/blob/master/integrations/iterable/lib/index.js#L163-L206 and https://github.com/segmentio/integrations/blob/master/integrations/iterable/lib/mapper.js, we pretty much just send the `products` array we are given to Iterable, but turns out, the endpoints we hit requires some basic fields in each product object: https://api.iterable.com/api/docs#commerce_trackPurchase and https://api.iterable.com/api/docs#commerce_updateCart. I tried to clarify these requirements better for less confusion in the future.
@c-noonan c-noonan requested a review from sanscontext March 29, 2021 18:18
@netlify
Copy link

netlify bot commented Mar 29, 2021

Deploy preview for segment-docs ready!

Built with commit b1accf9

https://deploy-preview-1486--segment-docs.netlify.app

@c-noonan c-noonan requested a review from sanscontext April 1, 2021 04:53
@sanscontext sanscontext merged commit df07540 into master Apr 7, 2021
@sanscontext sanscontext deleted the c-noonan-iterable-ecommerce-patch branch April 7, 2021 23:03
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.

2 participants