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: display offline products #1408

Merged
merged 3 commits into from
Apr 5, 2023
Merged

fix: display offline products #1408

merged 3 commits into from
Apr 5, 2023

Conversation

suschneider
Copy link
Collaborator

@suschneider suschneider commented Apr 3, 2023

PR Type

[x] Bugfix

What Is the Current Behavior?

When setting a product to offline in the ICM, this product will still be displayed in the Intershop PWA under the "Recently Viewed" section. The display of the product is changed as soon as the page cache is invalidated via the ICM.

What Is the New Behavior?

The product that was set offline is no longer displayed in the "Recently Viewed" section.

Does this PR Introduce a Breaking Change?

[ ] Yes
[x] No

Other Information

This is the same for featured products and the wish list widget in my account area.

AB#85105

@suschneider suschneider added the bug Something isn't working label Apr 3, 2023
@suschneider suschneider added this to the 4.1 milestone Apr 3, 2023
@suschneider suschneider requested a review from shauke April 3, 2023 13:53
@suschneider suschneider self-assigned this Apr 3, 2023
@suschneider suschneider force-pushed the fix/display-offline-products branch 2 times, most recently from 0b66112 to b598704 Compare April 4, 2023 07:52
@shauke shauke modified the milestones: 4.1, 4.0 Apr 4, 2023
)

* if a product REST call returns with an error (e.g. if the product is set to offline in ICM or if it does not exist at all) the product is marked as failed in the PWA state management
* such failed products are now filtered and no longer rendered by the generic Products List component
* the Product Tile and the Product Row component no longer render failed products as well
* Products List component now maps the ProductContextDisplayProperties configuration as listItemConfiguration
…ecently viewed page (#1408)

* by using the generic Products List component failed products are no longer rendered
* by using the generic Products List component failed products are no longer rendered
@shauke shauke force-pushed the fix/display-offline-products branch from 24fa2c8 to de8b11b Compare April 5, 2023 10:02
@shauke shauke merged commit 3486687 into develop Apr 5, 2023
@shauke shauke deleted the fix/display-offline-products branch April 5, 2023 11:58
shauke pushed a commit that referenced this pull request Apr 5, 2023
)

* if a product REST call returns with an error (e.g. if the product is set to offline in ICM or if it does not exist at all) the product is marked as failed in the PWA state management
* such failed products are now filtered and no longer rendered by the generic Products List component
* the Product Tile and the Product Row component no longer render failed products as well
* Products List component now maps the ProductContextDisplayProperties configuration as listItemConfiguration
shauke pushed a commit that referenced this pull request Apr 5, 2023
…ecently viewed page (#1408)

* by using the generic Products List component failed products are no longer rendered
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants