Skip to content

Conversation

@StasKozar
Copy link
Contributor

@StasKozar StasKozar commented Dec 15, 2017

Fix Back to Sign in url on confirmation form

Description

Added login url to Back to Sign in link

Fixed Issues (if relevant)

  1. Storefront Back to Sign in button does not work as expected #12715: Issue title Storefront Back to Sign in button does not work as expected

Manual testing scenarios

Preconditions

Magento is installed from 2.2-develop branch
Set Require Emails Confirmation = Yes at Stores > Configuration > Customers > Customer Configuration
Steps to reproduce

Go to Storefront as a guest
Start creating new customer
Fill in all the required fields
Click Create an Account
Click click here from message `Please check your email for the confirmation link or click here for a new link
Click Back to Sign In button
Expected result

Customer is redirected to Sign In form

Actual result

Page is just reloaded

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 bugfix Component: Customer mageconf Reproduced on 2.2.x The issue has been reproduced on latest 2.2 release labels Dec 15, 2017
@sidolov sidolov self-assigned this Dec 16, 2017
@sidolov sidolov added this to the December 2017 milestone Dec 16, 2017
@sidolov
Copy link
Contributor

sidolov commented Dec 18, 2017

Hi @StasKozar , please, sign license CLA and I will continue delivery process

@StasKozar
Copy link
Contributor Author

Hi @sidolov , done.

@sidolov
Copy link
Contributor

sidolov commented Dec 19, 2017

Hi @StasKozar , consider to port this solution for 2.3 version

@magento-team magento-team merged commit faee0fc into magento:2.2-develop Dec 22, 2017
@magento-engcom-team magento-engcom-team added the Fixed in 2.2.x The issue has been fixed in 2.2 release line label Dec 22, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix Component: Customer Fixed in 2.2.x The issue has been fixed in 2.2 release line Progress: accept Release Line: 2.2 Reproduced on 2.2.x The issue has been reproduced on latest 2.2 release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants