Skip to content

#13157 - Last Ordered Items block - bad js code #19039

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

Merged
merged 4 commits into from
Nov 22, 2018

Conversation

omiroshnichenko
Copy link
Contributor

Description

Fix logic in last-order-items.js and Recently Ordered block.

Fixed Issues

#13157: Last Ordered Items block - bad js code

Manual testing scenarios

  1. Create order for registered customer
  2. Verify that Add to Cart button in Recently Ordered block reflects is_salable property of recently ordered product

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
Copy link
Contributor

Hi @omiroshnichenko. Thank you for your contribution
Here is some useful tips how you can test your changes using Magento test environment.
Add the comment under your pull request to deploy test or vanilla Magento instance:

  • @magento-engcom-team give me test instance - deploy test instance based on PR changes
  • @magento-engcom-team give me $VERSION instance - deploy vanilla Magento instance

For more details, please, review the Magento Contributor Assistant documentation

@ishakhsuvarov
Copy link
Contributor

@magento-engcom-team give me test instance

@magento-engcom-team
Copy link
Contributor

Hi @ishakhsuvarov. Thank you for your request. I'm working on Magento instance for you

@magento-engcom-team
Copy link
Contributor

Hi @ishakhsuvarov, here is your new Magento instance.
Admin access: http://34.228.235.121/pr-19039//admin
Login: admin Password: 123123q

@orlangur orlangur self-assigned this Nov 2, 2018
@magento-engcom-team magento-engcom-team added this to the Release: 2.3.1 milestone Nov 2, 2018
@magento-engcom-team
Copy link
Contributor

Hi @orlangur, thank you for the review.
ENGCOM-3378 has been created to process this Pull Request

@sidolov
Copy link
Contributor

sidolov commented Nov 9, 2018

Hi @omiroshnichenko , we found several issues with provided fix:

case 1: Add to Cart button disappear after product was added to cart from 'Recently Ordered' section

  1. Open admin panel
  2. Create simple product Simple1
  3. Login to Storefront as Customer
  4. Click Add to Cart
  5. Submit Order
  6. Proceed to My Account page
  7. Enable checkbox Simple1 in 'Recently Ordered' section
  8. Click Add to Cart button in 'Recently Ordered' section
  9. Check button Add to Cart in 'Recently Ordered' section
    10. Button is not visible
  10. Refresh My Account page (F5)
  11. Add to Cart is visible in 'Recently Ordered' section

case 2: Add to Cart button is not visible after product change Status Stock from "Out of Stock" to "In Stock"

  1. Open Backend
  2. Create simple product Simple1 with qty=1
  3. Login to Storefront as Customer
  4. Click Add Product to cart (qty=1)
  5. Submit Order
  6. Go to My Account page
  7. Check 'Recently Ordered' section.
  8. Button Add to Cart is not visible
  9. Open Backend
  10. Go to Catalog - Products
  11. Click Edit simple1
  12. Input Quantity=1, Status Stock= In Stock
  13. Save product
  14. Open My Account page on Storefront
  15. Check 'Recently Ordered' section

Expected:
Add to Cart is visible in 'Recently Ordered' section

Actual:
Add to Cart isn't visible in 'Recently Ordered' section

@omiroshnichenko
Copy link
Contributor Author

Hi @sidolov, first case was fixed. Second case is not related to this issue. Problem is with customer data invalidation. Currently, Magento invalidates customer data on any POST request that goes from storefront. So, I think separate issue should be created for second case.

@magento-engcom-team
Copy link
Contributor

Hi @sidolov, thank you for the review.
ENGCOM-3378 has been created to process this Pull Request

@magento-engcom-team
Copy link
Contributor

Hi @omiroshnichenko. Thank you for your contribution.
We will aim to release these changes as part of 2.3.1.
Please check the release notes for final confirmation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants