Skip to content

Conversation

sade-wusi
Copy link
Contributor

Proposed changes

Provided clarification of how page events are transformed into track events and sent downstream.

Merge timing

  • ASAP

@sade-wusi sade-wusi requested a review from a team as a code owner August 6, 2025 18:43
@sade-wusi sade-wusi added the fixed-content fix to content that has outdated or wrong information label Aug 6, 2025
@sade-wusi sade-wusi requested a review from pwseg August 6, 2025 18:43
pwseg
pwseg previously approved these changes Aug 7, 2025
Copy link
Contributor

@pwseg pwseg left a comment

Choose a reason for hiding this comment

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

Handful of suggestions, but LGTM.

Co-authored-by: pwseg <86626706+pwseg@users.noreply.github.com>
@sade-wusi sade-wusi requested a review from pwseg August 7, 2025 13:25
@sade-wusi sade-wusi merged commit 504ea71 into develop Aug 7, 2025
2 checks passed
@sade-wusi sade-wusi deleted the optimizely-page-events branch August 7, 2025 17:46
Copy link
Contributor

github-actions bot commented Aug 7, 2025

Thank you for your contribution! Your pull request is merged, but may take a day or two to appear on the site.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fixed-content fix to content that has outdated or wrong information
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants