search

klaviyo / magento2-klaviyo

37 stars 51 forks source link

Handle newsletter subscriptions in all areas #129

Closed ihor-sviziev closed 2 years ago

ihor-sviziev commented 3 years ago

Pre-history: we have a custom Checkout extension that has built-in functionality for subscribing to the newsletter. This subscription happening on the same request as "place order". This request usually happening in the webapi_rest area. As a result, in Magento, we have subscribers, but in Klaviyo, this information isn't getting sent, so we're losing subscribers.

My Pull Request changing handling all possible areas, including webapi_rest, webapi_soap, graphql and crontab, as well as existing frontend and adminhtml.

ihor-sviziev commented 2 years ago

@klaviyojad @remstone7, could you review this Pull Request?