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

Fix Newsletter Subscribe Workflow #13044

Merged
merged 2 commits into from
Jan 11, 2018
Merged

Fix Newsletter Subscribe Workflow #13044

merged 2 commits into from
Jan 11, 2018

Conversation

torhoehn
Copy link
Contributor

@torhoehn torhoehn commented Jan 7, 2018

Description

This pull request fixes that the newsletter confirmation email will be sent multiple times and that the Confirmation Succes and Unsubscription Email will be sent at the correct states.

Fixed Issues (if relevant)

  1. Multiple newsletter confirmation emails sent #12876: Multiple newsletter confirmation emails sent

Manual testing scenarios

  1. Change configuration under Stores > Configuration > Customers > Newsletter > Subscription Options > Need to Confirm to Yes

  2. Sign up for Newsletter on frontend as a guest

  3. Follow confirmation link in email

  4. Create an account and check Sign Up for Newsletter within the form and use the same email as in Step 2

  5. Go to your customer account edit

  6. Save your customer account without changing anything

Contribution checklist

  • Pull request has a meaningful description of its purpose
  • All commits are accompanied by meaningful commit messages
  • All new or changed code is covered with unit/integration tests (if applicable)
  • All automated tests passed successfully (all builds on Travis CI are green)

@magento-engcom-team magento-engcom-team added Reproduced on 2.2.x The issue has been reproduced on latest 2.2 release Reproduced on 2.3.x The issue has been reproduced on latest 2.3 release labels Jan 7, 2018
@dmanners
Copy link
Contributor

Hi @torhoehn thank you for this pull request. Currently the static tests are failing in travis. Could you take a look at the following issue for me.

There was 1 failure:
1) Magento\Test\Php\LiveCodeTest::testCodeStyle
PHP Code Sniffer detected 2 violation(s): 
FILE: ...is/build/magento/magento2/app/code/Magento/Newsletter/Model/Subscriber.php
--------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
--------------------------------------------------------------------------------
 617 | ERROR | [x] Inline control structures are not allowed
--------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------

Thank you

@dmanners dmanners self-assigned this Jan 10, 2018
@dmanners dmanners added this to the January 2018 milestone Jan 10, 2018
@dmanners
Copy link
Contributor

@torhoehn thank you for the quick update. I will process this PR now for you.

@magento-team magento-team merged commit 93453a8 into magento:2.2-develop Jan 11, 2018
@magento-engcom-team magento-engcom-team added the Fixed in 2.2.x The issue has been fixed in 2.2 release line label Jan 11, 2018
@torhoehn torhoehn deleted the newsletter-fix branch January 11, 2018 19:34
@ihor-sviziev ihor-sviziev added partners-contribution Pull Request is created by Magento Partner and removed partner-contribution labels Jun 5, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fixed in 2.2.x The issue has been fixed in 2.2 release line partners-contribution Pull Request is created by Magento Partner Progress: accept Release Line: 2.2 Reproduced on 2.2.x The issue has been reproduced on latest 2.2 release Reproduced on 2.3.x The issue has been reproduced on latest 2.3 release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants