Skip to content

Fix incomplete condition on order detail and improve shipment flag#849

Merged
tblivet merged 1 commit intoPrestaShop:developfrom
tblivet:fix/carrier-condition
Dec 9, 2025
Merged

Fix incomplete condition on order detail and improve shipment flag#849
tblivet merged 1 commit intoPrestaShop:developfrom
tblivet:fix/carrier-condition

Conversation

@tblivet
Copy link
Contributor

@tblivet tblivet commented Dec 9, 2025

Questions Answers
Description? Fix incomplete condition on order detail and improve shipment flag
Type? bug fix
BC breaks? no
Deprecations? no
Fixed ticket? #847
Sponsor company @PrestaShopCorp
How to test? Refer to the issue

@tblivet tblivet requested a review from ga-devfront December 9, 2025 16:47
@github-project-automation github-project-automation bot moved this to Ready for review in PR Dashboard Dec 9, 2025
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 @tblivet,

Your PR is tested and works perfectly ✨

"Carrier" line is no more displayed on virtual product only orders :

Capture d’écran 2025-12-09 à 18 49 01

No regression on orders with physical products :
SCR-20251209-qmts

And no regression on Classic theme :
Capture d’écran 2025-12-09 à 18 50 39

It's QA approved 👍 ✅

@ingridusta ingridusta self-assigned this Dec 9, 2025
@ps-jarvis ps-jarvis moved this from Ready for review to To be tested in PR Dashboard Dec 9, 2025
@tblivet tblivet merged commit 0e9a5a8 into PrestaShop:develop Dec 9, 2025
6 checks passed
@github-project-automation github-project-automation bot moved this from To be tested to Merged in PR Dashboard Dec 9, 2025
@ps-jarvis
Copy link

PR merged, well done!

Message to @PrestaShop/committers: do not forget to milestone it before the merge.

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.

[v1] Order detail page : "Carrier" is displayed for orders containing only virtual products

4 participants