Avoid impacting statistics of succesfull updates with test cases#1541
Conversation
|
I tested your PR and when I do a backup with the command as you can see : Capture.video.du.2025-12-02.11-05-39.mp4Because of that, currently I can't approve this PR Waiting for feedback |
4f0e024
4f0e024 to
a7fec3e
Compare
|
|
Thank you for your PR, I tested it and I still have the segment call on web version, as you can see it : Capture.video.du.2025-12-03.11-30-36.mp4I checked if I have the good version of the PR and I still have the parameters ' PS_URL_TRACKING : 0 ' Waiting for feedback |
|
To be checked as a bug, not a regression. The change we're dealing with here is about the events sent from the backend, not the front end |
|
@AureRita I need you to test the the events sent from the back end. |
AureRita
left a comment
There was a problem hiding this comment.
Thank you for your PR, I tested it and it seems to works as you can see :
Capture.video.du.2025-12-04.17-50-51.mp4
Capture.video.du.2025-12-04.17-42-17.mp4
I checked and only js_fe_prod are submitted
Tested from :
8.21 to 9.0.1
Because the PR seems to works as expected, It's QA ✔️
Thank you
|
Waiting of the release of 7.5.0 before merging this PR |



PS_VERSION=8.2.1 docker compose up -dfrom the command line, thendocker exec -ti -u www-data prestashop php modules/autoupgrade/bin/console backup:create admin-dev. Check Segment with and without the env var.