Skip to content

Added a compatibility layer for orders without shipments#906

Merged
Touxten merged 1 commit intoPrestaShop:developfrom
M0rgan01:added-compatibility-layer-for-orders-without-shipments
Jan 23, 2026
Merged

Added a compatibility layer for orders without shipments#906
Touxten merged 1 commit intoPrestaShop:developfrom
M0rgan01:added-compatibility-layer-for-orders-without-shipments

Conversation

@M0rgan01
Copy link
Contributor

Questions Answers
Description? Added a compatibility layer for orders without shipments
Type? improvement
BC breaks? no
Deprecations? no
Fixed ticket? -
Sponsor company -
How to test? -

Nakahiru
Nakahiru previously approved these changes Jan 23, 2026
@M0rgan01 M0rgan01 force-pushed the added-compatibility-layer-for-orders-without-shipments branch from 118e7eb to 4c288e1 Compare January 23, 2026 10:28
@ps-jarvis ps-jarvis moved this from Ready for review to To be tested in PR Dashboard Jan 23, 2026
Copy link
Contributor

@tblivet tblivet left a comment

Choose a reason for hiding this comment

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

👍

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 @M0rgan01,

Your fix has been tested on both Hummingbird and Classic themes.

  • Details of an order placed before FF activated are still displayed when FF is activated
  • Details of an order placed with FF activated are still displayed when FF is disabled
  • No regression detected on any displays detail order history page
  • No regression detected on returns
  • No regression detected on whole multishipment feature

It's QA approved ✅

@ingridusta ingridusta removed their assignment Jan 23, 2026
@Touxten Touxten merged commit de9890f into PrestaShop:develop Jan 23, 2026
6 checks passed
@github-project-automation github-project-automation bot moved this from To be tested to Merged in PR Dashboard Jan 23, 2026
@Touxten
Copy link
Contributor

Touxten commented Jan 23, 2026

Thank you @ingridusta && @M0rgan01

@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.

8 participants