Skip to content

Conversation

@M0rgan01
Copy link
Contributor

Questions Answers
Description? This can be confusing for the user. What they're interested in is the total shipping cost, which is already included in the totals. Furthermore, after splitting and merging shipments, it's currently impossible to have accurate values ​​in this table.
Type? improvement
BC breaks? no
Deprecations? no
Fixed ticket? -
Sponsor company -
How to test? -

@github-project-automation github-project-automation bot moved this to Ready for review in PR Dashboard Jan 29, 2026
@ps-jarvis ps-jarvis added the Waiting for QA Status: Waiting for QA feedback label Jan 29, 2026
@ps-jarvis ps-jarvis moved this from Ready for review to To be tested in PR Dashboard Jan 29, 2026
@AureRita AureRita self-assigned this Jan 30, 2026
Copy link

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

Thank you for your PR, I tested it and it seems to works as you can see :

Image

Because the PR seems to works as expected, It's QA ✔️

Thank you

@AureRita AureRita added QA ✔️ Status: QA-Approved and removed Waiting for QA Status: Waiting for QA feedback labels Jan 30, 2026
@Touxten Touxten merged commit 314513a into PrestaShop:develop Jan 30, 2026
6 checks passed
@github-project-automation github-project-automation bot moved this from To be tested to Merged in PR Dashboard Jan 30, 2026
@ps-jarvis
Copy link

PR merged, well done!

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

@Touxten
Copy link
Contributor

Touxten commented Jan 30, 2026

Thnak you @AureRita && @M0rgan01

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

QA ✔️ Status: QA-Approved

Projects

Status: Merged

Development

Successfully merging this pull request may close these issues.

6 participants