Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update: update VWO SmartCode to v2.1 #816

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

zeeshan-vwo
Copy link
Contributor

@zeeshan-vwo zeeshan-vwo commented Mar 3, 2025

What does this PR do?
This PR updates the VWO SmartCode in the project to the latest version (v2.1), replacing the outdated version to ensure compatibility and access to the latest features and improvements. This PR also resolves the issue found in PR #808

Are there breaking changes in this PR?
No, there are no breaking changes expected in this PR. The updated SmartCode is backward-compatible with existing functionality.

Testing
End-to-end testing is required to verify the successful loading of SmartCode v2.1 into the page.
Note: Require assistance from Segment engineering team to deploy it in the given testing source: Testing Source

Any background context you want to provide?
The current project contains a very old version of the VWO SmartCode. Upgrading to version 2.1 ensures alignment with VWO’s latest updates and feature set.
Also this is a continuation of PR #808 which was reverted due to issues and bugs found while initializing the SmartCode.

Is there parity with the server-side/android/iOS integration components (if applicable)?
This PR is not directly related to server-side, Android, or iOS components.

Does this require a new integration setting? If so, please explain how the new setting works
No new integration settings are required for this update.

Testing not required as this is a test and will only be deployed to one source.

@rvadera12 rvadera12 mentioned this pull request Mar 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants