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

Dashboard panel keeps re-opening during checkout #2010

Merged
merged 3 commits into from
Mar 22, 2017

Conversation

impactmass
Copy link
Contributor

@impactmass impactmass commented Mar 17, 2017

Resolves #1953.

Change made: Use proper check for enable shipping on sidebar display

How to test:

  1. Do a reaction reset before starting the app.
  2. First, configure shipping, then add an item to cart and proceed to checkout.
  3. On the checkout page, close the sidebar (if open). Then, add shipping address and save.
  4. Confirm that the sidebar doesn't pop out (since you already have shipping configured.
  5. Go back and disable shipping. Go back to checkout (close your sidebar, if opened).
  6. Delete your earlier shipping address (if there).
  7. Re-add shipping address and save. Confirm that the sidebar opens to shipping settings (since you don't have it enabled)

You can do multiple variations of trying to checkout with shipping configured and when it is not.

Copy link
Collaborator

@brent-hoover brent-hoover left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested. Works as expected.

Copy link
Member

@mikemurray mikemurray left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does't seem to work for me.

I have shipping enabled and configured, but the shipping panel keeps opening up every time I go to the checkout page, and when I delete / re-add address.

  • Did a full reset and everything.
  • Im only using flat rate shipping with the first 2 shipping methods enabled.

@impactmass
Copy link
Contributor Author

Ok. I'll go over this again to spot what could be going on.

@impactmass
Copy link
Contributor Author

OK.... Found the issue... So. this branch seun-fix-checkout-sidebar-1956 which I'm making the PR from was updated. I guess this was done in order to bring it up to the state development is.

But that update is affecting what I've done in this PR, causing @mikemurray to get unexpected results.

I'll do a rebase for this branch.

@impactmass
Copy link
Contributor Author

This is good for a second pass @mikemurray

Copy link
Member

@mikemurray mikemurray left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@mikemurray mikemurray merged commit 9e32c11 into development Mar 22, 2017
@mikemurray mikemurray deleted the seun-fix-checkout-sidebar-1956 branch March 22, 2017 16:23
@mikemurray mikemurray removed the review label Mar 22, 2017
jshimko added a commit to evereveofficial/reaction that referenced this pull request Mar 27, 2017
* development:
  fix elasticsearch errors
  skip npm install outside of container on CI
  wait longer on CI for Docker test
  update CI config for Evereve
  add ReactionPublicCustomFolder
  update data_import
  update data_import
  update data_import
  add submodules to circle config
  add plugin submodules
  Don't reload when the main product is not in tbe modified products (reactioncommerce#2033)
  If default provider is not available, then add it. (reactioncommerce#2025)
  Dashboard panel keeps re-opening during checkout (reactioncommerce#2010)
  fixing issue 2008 getting 404 error on variant option (reactioncommerce#2021)
  Create "Tax Detail" view in Orders Dashboard (reactioncommerce#2005)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants