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

fix: quick quantity change on item lists #1391

Merged
merged 6 commits into from
Mar 30, 2023
Merged

Conversation

SGrueber
Copy link
Collaborator

@SGrueber SGrueber commented Mar 21, 2023

PR Type

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build-related changes
[ ] CI-related changes
[ ] Documentation content changes
[ ] Application / infrastructure changes
[ ] Other:

Actual Behavior
When I change the quantity of an item in the shopping cart, right after I have already changed the quantity before, the price of the item is not updated

Issue Number: Closes #1380

Expected Behavior
If I have the possibility to change the item quantity, I expect a correct update of the item

Does this PR Introduce a Breaking Change?

[ ] Yes
[x] No

Other Information

AB#84676

@SGrueber SGrueber self-assigned this Mar 21, 2023
@SGrueber SGrueber added this to the 4.0 milestone Mar 21, 2023
Copy link
Collaborator Author

@SGrueber SGrueber left a comment

Choose a reason for hiding this comment

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

account-wishlist-details-page, quote-line-item-list and retailset-parts are at least missing

@Eisie96
Copy link
Contributor

Eisie96 commented Mar 27, 2023

A trackBy function is not needed for the retail set part component, because the initial data will not change, when e.g the product quantity increases.

@SGrueber SGrueber merged commit b019bb2 into develop Mar 30, 2023
@SGrueber SGrueber deleted the bugfix/quantity_change branch March 30, 2023 13:10
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.

Quick quantity change directly after a change is ignored in the shopping cart
2 participants