Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Export orders and carts only for already exported customers #49

Merged
merged 1 commit into from Apr 20, 2022
Merged

Export orders and carts only for already exported customers #49

merged 1 commit into from Apr 20, 2022

Conversation

lruozzi9
Copy link
Member

Changes proposed in this pull request:

  • Implements the OrderRepository to only exports orders and abandoned carts owned by a customer already exported to ActiveCampaign

@webgriffe/wg-devs what do you think about this PR :shipit:?

@lruozzi9 lruozzi9 added bug Something isn't working enhancement New feature or request labels Apr 19, 2022
@lruozzi9 lruozzi9 self-assigned this Apr 19, 2022
@lruozzi9 lruozzi9 changed the title Export orders and carts only for already exported customers (#49) Export orders and carts only for already exported customers Apr 19, 2022
@mmenozzi
Copy link
Member

Changes proposed in this pull request:

  • Implements the OrderRepository to only exports orders and abandoned carts owned by a customer already exported to ActiveCampaign

@webgriffe/wg-devs what do you think about this PR :shipit:?

Looks good to me.

LucaGallinari
LucaGallinari approved these changes Apr 20, 2022
View reviewed changes
Copy link
Member

@LucaGallinari LucaGallinari left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great!

@lruozzi9 lruozzi9 merged commit cbc1671 into master Apr 20, 2022
@lruozzi9 lruozzi9 deleted the fix-order-repository branch April 20, 2022 12:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@LucaGallinari LucaGallinari LucaGallinari approved these changes

Assignees

@lruozzi9 lruozzi9

Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@lruozzi9 @mmenozzi @LucaGallinari