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

Previously unsubscribed guest/customer is not resubscribed when selecting groups on success page #365

Closed
bexdoggett opened this issue Aug 7, 2018 · 0 comments
Assignees
Milestone

Comments

@bexdoggett
Copy link

Issue Description

If a customer or guest has signed up for the newsletter and then unsubscribed, when they place a new order and land on the success page they can still see the groups block from Mailchimp. If they select a group and click update their groups are updated, but they are still unsubscribed. If a customer or guest who has never signed up to the newsletter follows the same steps they are subscribed to the newsletter.

Preconditions

  1. Magento CE 2.1.14 (With a fix to stop the order.success.additional.info from outputting twice)
  2. mc-magento2 1.0.28
  3. Newsletter group created with interests, set to display as checkboxes
  4. Any browser

Steps to reproduce

  1. Sign up for the newsletter
  2. Unsubscribe from the newsletter
  3. Add any product to the cart and checkout using account or email address from above
  4. Once on the success page select a group and submit

Actual and Expected result

Expected result:
When looking in the newsletter subscribers table, I expect to see the user resubscribed to the newsletter. Alternatively the groups block should not be available on the success page for unsubscribed customers/guests.

Actual result:
User is still unsubscribed.

gonzaloebiz added a commit that referenced this issue Dec 10, 2018
gonzaloebiz added a commit that referenced this issue Dec 10, 2018
@gonzaloebiz gonzaloebiz self-assigned this Dec 10, 2018
@gonzaloebiz gonzaloebiz added this to the 1.0.33 milestone Dec 10, 2018
gonzaloebiz added a commit that referenced this issue Dec 10, 2018
gonzaloebiz added a commit that referenced this issue Dec 10, 2018
gonzaloebiz added a commit that referenced this issue Dec 10, 2018
gonzaloebiz added a commit that referenced this issue Dec 10, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants