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

signing in after selecting checkout button, will not end up to checkout page! #10834

Closed
MMaKenZi opened this issue Sep 10, 2017 · 13 comments
Closed
Labels
feature request Fixed in 2.2.x The issue has been fixed in 2.2 release line Fixed in 2.3.x The issue has been fixed in 2.3 release line Issue: Confirmed Gate 3 Passed. Manual verification of the issue completed. Issue is confirmed Issue: Format is valid Gate 1 Passed. Automatic verification of issue format passed Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development 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

Comments

@MMaKenZi
Copy link

If you open a magento store, add a product to cart, and try to checkout, and guest checkout be disabled, the login window will appear on screen,
but if you login to your account, you will be returned to homepage

while the correct page is checkout
because user clicked the checkout button before signing in

Preconditions

  1. Magento 2.1.8
  2. PHP 7.0.22

Steps to reproduce

  1. disable guest checkout
  2. open magento store
  3. add product to your cart and select checkout button
  4. fill the login window details
  5. and you should select checkout button once again after signing in

Expected result

after signing in, user should redirected to checkout page, as he want this before signing in

Actual result

will be returned to homepage

@magento-engcom-team magento-engcom-team added G1 Passed feature request Issue: Format is valid Gate 1 Passed. Automatic verification of issue format passed and removed G1 Passed labels Sep 10, 2017
@pboisvert
Copy link

@tkacheva Olena--can you consider this for backlog? I think it may make sense as a default as login at checkout is likely driven by desire to use stored address/payment info or possibly get a discount tied to customer account, but should do some research to confirm if it should configurable (to current or continue checkout).

@magento-engcom-team magento-engcom-team added Issue: Clear Description Gate 2 Passed. Manual verification of the issue description passed and removed Issue: Clear Description Gate 2 Passed. Manual verification of the issue description passed labels Sep 21, 2017
@magento-engcom-team
Copy link
Contributor

@MMaKenZi, thank you for your report.
We've created internal ticket(s) MAGETWO-80516 to track progress on the issue.

@magento-engcom-team magento-engcom-team added 2.2.x Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development Issue: Confirmed Gate 3 Passed. Manual verification of the issue completed. Issue is confirmed 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 Oct 1, 2017
@MIstvan93
Copy link

MIstvan93 commented Oct 15, 2017

I'm working on it #SQUАSHTOBERFEST

@MIstvan93
Copy link

MIstvan93 commented Oct 15, 2017

I've found that in this situation there's a configuration that can be done from stores> settings > configuration >customers > customer configuration > login options > redirect customer to account dashboard after logging in needs to be set to no like this.

http://prntscr.com/gxo0bn

@maksek
Copy link
Contributor

maksek commented Oct 15, 2017

@MIstvan93, what do you think if the "No" value will be the default one after installation?

@MIstvan93
Copy link

MIstvan93 commented Oct 15, 2017

@maksek I think it would be a good idea, but that's just me, but also there should be a condition made for this for in case it's set to yes, but if it's a checkout redirect login, the customer should be sent to checkout.

@maksek
Copy link
Contributor

maksek commented Oct 15, 2017

@MIstvan93 - create PR with changing default value, and we will process it :)

@okorshenko
Copy link
Contributor

okorshenko commented Oct 16, 2017

Hi @MIstvan93 could you please accept the invite to join magento2 repository on the GitHub?

1 similar comment
@okorshenko
Copy link
Contributor

Hi @MIstvan93 could you please accept the invite to join magento2 repository on the GitHub?

@okorshenko
Copy link
Contributor

Hi @bohemiorulo please accept the invite to join magento2 repository on the GitHub?

@bohemiorulo
Copy link

yes, I already accept it, thanks

p-bystritsky added a commit to p-bystritsky/magento2 that referenced this issue Oct 30, 2017
p-bystritsky added a commit to p-bystritsky/magento2 that referenced this issue Nov 1, 2017
p-bystritsky added a commit to p-bystritsky/magento2 that referenced this issue Nov 10, 2017
@okorshenko
Copy link
Contributor

The issue has been fixed and delivered to 2.2-develop branch. Will be available with 2.2.2 release

@okorshenko okorshenko added the Fixed in 2.2.x The issue has been fixed in 2.2 release line label Nov 11, 2017
@okorshenko
Copy link
Contributor

Hi @MMaKenZi. Thank you for your report.
The issue has been fixed in magento-engcom/magento2ce#1353 by @magento-engcom-team in 2.3-develop branch
Related commit(s):

The fix will be available with the upcoming 2.3.0 release.

@okorshenko okorshenko added the Fixed in 2.3.x The issue has been fixed in 2.3 release line label Feb 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request Fixed in 2.2.x The issue has been fixed in 2.2 release line Fixed in 2.3.x The issue has been fixed in 2.3 release line Issue: Confirmed Gate 3 Passed. Manual verification of the issue completed. Issue is confirmed Issue: Format is valid Gate 1 Passed. Automatic verification of issue format passed Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development 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

No branches or pull requests

8 participants