Skip to content

[FIX] Toggle off events on web when requested#1589

Merged
Quetzacoalt91 merged 11 commits intoPrestaShop:devfrom
ga-devfront:fix/disable-segments-events-on-web
Jan 9, 2026
Merged

[FIX] Toggle off events on web when requested#1589
Quetzacoalt91 merged 11 commits intoPrestaShop:devfrom
ga-devfront:fix/disable-segments-events-on-web

Conversation

@ga-devfront
Copy link
Contributor

@ga-devfront ga-devfront commented Jan 7, 2026

Questions Answers
Description? disable segments events on web
Type? bug fix
BC breaks? no
Deprecations? no
Fixed ticket? N/A
Sponsor company @PrestaShopCorp
How to test? Check below this table :O

How to reproduce

Step 1: inside module folder /tests/UI launch "PS_VERSION=8.2.1 docker compose up -d"
Step 2: Go to BO from the new shop
Step 3: Go to module Manager
Step 4: Search Update assistant
Step 5: Install it
Step 6: Clic on Configure
Step 7: Clic on Update your store and get started
Step 8: Select Prestashop v9.0.1 and Next
Step 9: Click on Next and Launch Backup
Step 10: Do a right click, click on Inspect and select Network
Step 11: Click on Start Backup

Expected result after fix

Step 1: Docker is started to create your shop
Step 2: BO is displayed
Step 3: Module manager page is displayed
Step 4: Update assistant module is displayed
Step 5: Button Install is replaced by Configure
Step 6: Welcome page of update assistant is displayed
Step 7: Version choice page is displayed
Step 8: Option choice page is displayed
Step 9: Modale Launch update is displayed
Step 10: None call on Network is displayed
Step 11: Many call is displayed but none with domain : api.segment.io

Obtained result before fix

Step 1: Docker is started to create your shop
Step 2: BO is displayed
Step 3: Module manager page is displayed
Step 4: Update assistant module is displayed
Step 5: Button Install is replaced by Configure
Step 6: Welcome page of update assistant is displayed
Step 7: Version choice page is displayed
Step 8: Option choice page is displayed
Step 9: Modale Launch update is displayed
Step 10: None call on Network is displayed
Step 11: Many call is displayed but one have domain : api.segment.io

@ga-devfront ga-devfront added the bug Type: Bug fix label Jan 7, 2026
@github-project-automation github-project-automation bot moved this to Ready for review in PR Dashboard Jan 7, 2026
@ga-devfront ga-devfront force-pushed the fix/disable-segments-events-on-web branch from ebf1156 to 668fa31 Compare January 7, 2026 16:09
@Quetzacoalt91 Quetzacoalt91 changed the title [FIX] disable segments events on web [FIX] Toggle off events on web when requested Jan 8, 2026
Quetzacoalt91
Quetzacoalt91 previously approved these changes Jan 8, 2026
@ga-devfront ga-devfront added this to the 7.6.0 milestone Jan 8, 2026
@AureRita AureRita self-assigned this Jan 9, 2026
@sonarqubecloud
Copy link

sonarqubecloud bot commented Jan 9, 2026

Copy link
Contributor

@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 @ga-devfront

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

Capture.video.du.2026-01-09.14-46-33.mp4
Capture d’écran du 2026-01-09 11-54-39 Capture d’écran du 2026-01-09 12-00-04

Tested on :
8.2.1

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

Thank you

@AureRita AureRita added the QA ✔️ Status: Check done, Code approved label Jan 9, 2026
@Quetzacoalt91 Quetzacoalt91 merged commit 881a5ee into PrestaShop:dev Jan 9, 2026
56 checks passed
@github-project-automation github-project-automation bot moved this from Ready for review to Merged in PR Dashboard Jan 9, 2026
@ps-jarvis ps-jarvis moved this from Merged to Ready for review in PR Dashboard Jan 9, 2026
@ga-devfront ga-devfront deleted the fix/disable-segments-events-on-web branch January 9, 2026 14:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Type: Bug fix QA ✔️ Status: Check done, Code approved

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants