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.

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

Do queued emails get removed on cart recovery? #17

Open
sunscreem opened this issue May 31, 2019 · 4 comments
Open

Do queued emails get removed on cart recovery? #17

sunscreem opened this issue May 31, 2019 · 4 comments
Labels
to implement Schedule to be implemented in an upcoming release

Comments

@sunscreem
Copy link
Contributor

Hey Myles. Lets say the first and second emails have been queued.

The first email is sent, and the user clicks through and restores their cart.

What happens to the queued second email job?

@mediabeastnz mediabeastnz added the to implement Schedule to be implemented in an upcoming release label May 31, 2019
@mediabeastnz
Copy link
Collaborator

There's a check to not actually send out an email if the order is already recovered. However I do need to change this so it removes it from the queue entirely.

@sunscreem
Copy link
Contributor Author

Got it - thanks myles. So the job remains queued, but won't actually email when it gets fired?

@mediabeastnz
Copy link
Collaborator

mediabeastnz commented Jun 2, 2019

Exactly. On my to do list though to completely remove it from the queue.

@sunscreem
Copy link
Contributor Author

Hello Sir,

Just trying to finish up my understanding of how the plug in handles restoring carts and not sending the second email. I'm sorry for the endless stream of questions.

Looks at the code I think the cart is marked as recovered when an order is placed?

So would this be right:

  1. Abandoned cart found, 1st email sent.
  2. User clicks the link and their cart is restored.
  3. They start the checkout process again, but don't complete it.
  4. The 2nd email is still still queued and sent? (Hence this open issue)
  5. The 1st email won't be triggered to be queued again because its already been sent once?

Rob

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
to implement Schedule to be implemented in an upcoming release
Projects
None yet
Development

No branches or pull requests

2 participants