Skip to content

Fix structured data related to conditions#914

Merged
Hlavtox merged 1 commit intoPrestaShop:developfrom
Hlavtox:fix-structured-data
Jan 26, 2026
Merged

Fix structured data related to conditions#914
Hlavtox merged 1 commit intoPrestaShop:developfrom
Hlavtox:fix-structured-data

Conversation

@Hlavtox
Copy link
Contributor

@Hlavtox Hlavtox commented Jan 23, 2026

Questions Answers
Description? The code that shows condition in structured data doesn't work. This fixes it.
Type? refactor
BC breaks? no
Deprecations? no
Fixed ticket?
Sponsor company TRENDO s.r.o.
How to test? See below

Ze problem

Assign any condition (Used, Refurbished) to a product. Go to the product in FO, view source code. Search for itemCondition, nothing.

With PR, works.

Snímek obrazovky 2026-01-23 232335

@Hlavtox Hlavtox added this to the v2.0.0 milestone Jan 23, 2026
@github-project-automation github-project-automation bot moved this to Ready for review in PR Dashboard Jan 23, 2026
@Hlavtox
Copy link
Contributor Author

Hlavtox commented Jan 23, 2026

@kpodemski

@ps-jarvis ps-jarvis moved this from Ready for review to To be tested in PR Dashboard Jan 23, 2026
@ingridusta ingridusta self-assigned this Jan 26, 2026
Copy link

@ingridusta ingridusta left a comment

Choose a reason for hiding this comment

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

Hi @Hlavtox,

I reproduced the bug you fixed :

Image

And with your fix, itemCondition is shown in the code :

Image

No regression detected on the product page, and on the condition.

It's QA approved 🎉

@ingridusta ingridusta removed their assignment Jan 26, 2026
@Hlavtox Hlavtox merged commit 1fce4ed into PrestaShop:develop Jan 26, 2026
6 checks passed
@github-project-automation github-project-automation bot moved this from To be tested to Merged in PR Dashboard Jan 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

6 participants