Skip to content

Conversation

@gibkigonzo
Copy link
Contributor

Related Issues

closes #4132, #4133, #4136, #4138, #4139, #4140

Short Description and Why It's Useful

  • added comments in 'productsEqual' and change logic for different type of products
  • remove login user after order
  • allow changing qty for 'group' and 'bundle' products

Screenshots of Visual Changes before/after (if There Are Any)

Which Environment This Relates To

Check your case. In case of any doubts please read about Release Cycle

  • Test version (https://test.storefrontcloud.io) - this is a new feature or improvement for Vue Storefront. I've created branch from develop branch and want to merge it back to develop
  • RC version (https://next.storefrontcloud.io) - this is a stabilisation fix for Release Candidate of Vue Storefront. I've created branch from release branch and want to merge it back to release
  • Stable version (https://demo.storefrontcloud.io) - this is an important fix for current stable version. I've created branch from hotfix or master branch and want to merge it back to hotfix

Upgrade Notes and Changelog

  • No upgrade steps required (100% backward compatibility and no breaking changes)
  • I've updated the Upgrade notes and Changelog on how to port existing Vue Storefront sites with this new feature

IMPORTANT NOTICE - Remember to update CHANGELOG.md with description of your change

Contribution and Currently Important Rules Acceptance

tkostuch added 2 commits March 2, 2020 11:42
- remove login user after order
- allow changing qty for 'group' and 'bundle' products
@gibkigonzo gibkigonzo requested review from andrzejewsky and pkarw March 2, 2020 11:10
@gibkigonzo gibkigonzo added the QA - Ready for tests This is notification for testers, that improvement is ready to be tested and verified. label Mar 2, 2020
@pkarw pkarw requested a review from patzick March 2, 2020 11:38
Copy link
Collaborator

@patzick patzick left a comment

Choose a reason for hiding this comment

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

everything looks good, nice work! :)

@patzick patzick added this to the 1.11.2 milestone Mar 2, 2020
@patzick patzick merged commit a2faebc into vuestorefront:hotfix/v1.11.2 Mar 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

QA - Ready for tests This is notification for testers, that improvement is ready to be tested and verified.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants