Skip to content

Fix old QA feedbacks#811

Merged
tblivet merged 2 commits intoPrestaShop:developfrom
tblivet:fix/old-qa-feedbacks
Nov 14, 2025
Merged

Fix old QA feedbacks#811
tblivet merged 2 commits intoPrestaShop:developfrom
tblivet:fix/old-qa-feedbacks

Conversation

@tblivet
Copy link
Contributor

@tblivet tblivet commented Nov 13, 2025

Questions Answers
Description? Fix old QA feedbacks
Type? bug fix / improvement
BC breaks? no
Deprecations? no
Fixed ticket? #409
Sponsor company @PrestaShopCorp
How to test? --

1️⃣ First fix: Remove close option on toast to improve consistency

2️⃣ Second fix: Improve product summary

Before:
before

After:
after

@github-project-automation github-project-automation bot moved this to Ready for review in PR Dashboard Nov 13, 2025
@tblivet tblivet requested a review from ga-devfront November 13, 2025 17:08
@tblivet tblivet linked an issue Nov 13, 2025 that may be closed by this pull request
47 tasks
Copy link
Contributor

@ga-devfront ga-devfront left a comment

Choose a reason for hiding this comment

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

Beautifull ❤️

@ps-jarvis ps-jarvis moved this from Ready for review to To be tested in PR Dashboard Nov 13, 2025
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 @tblivet

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

Capture.video.du.2025-11-14.11-37-04.mp4

Tested on :
9.0.x
develop

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

Thank you

@tblivet tblivet merged commit 1469b66 into PrestaShop:develop Nov 14, 2025
6 checks passed
@github-project-automation github-project-automation bot moved this from To be tested to Merged in PR Dashboard Nov 14, 2025
@ps-jarvis ps-jarvis moved this from Merged to To be tested in PR Dashboard Nov 14, 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.

QA feedbacks - bug list

4 participants