klaviyo / magento2-klaviyo

37 stars 51 forks source link

Do not load not needed backend classes #214

Closed ihor-sviziev closed 1 year ago

ihor-sviziev commented 1 year ago

Description

This PR replaces the backend context with the frontend one. Since this controller is used on frontend, it doesn't make any sense to create classes needed for backend, e.g. backend session which is part of this context.

Basically, it just improves speed of this controller and reduces session locks on checkout

Manual Testing Steps

1.

Pre-Submission Checklist:

cykolln commented 1 year ago

This has been included in the latest release 4.0.7, thank you for another contribution @ihor-sviziev - much appreciated!