Skip to content

Conversation

@tgberkeley
Copy link
Collaborator

@tgberkeley tgberkeley commented Nov 19, 2025

This PR depends on reflex-dev/reflex-ui#48 in Reflex UI

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Nov 19, 2025

Greptile Summary

  • Added Google Ads conversion tracking pixel by importing and calling gtag_report_conversion with conversion ID
  • Change follows existing telemetry pattern with hardcoded tracking IDs, consistent with other trackers in the file

Confidence Score: 5/5

  • This PR is safe to merge with minimal risk
  • Simple addition of a telemetry tracker that follows the exact same pattern as existing trackers in the file, with no logic changes or complex functionality
  • No files require special attention

Sequence Diagram

sequenceDiagram
    participant User
    participant WebsiteLoader
    participant PixelModule
    participant GoogleAds
    participant OtherTrackers
    
    User->>WebsiteLoader: "Visit website"
    WebsiteLoader->>PixelModule: "Call get_pixel_website_trackers()"
    PixelModule->>GoogleAds: "Initialize gtag_report_conversion()"
    PixelModule->>OtherTrackers: "Initialize other trackers"
    GoogleAds-->>WebsiteLoader: "Conversion pixel loaded"
    OtherTrackers-->>WebsiteLoader: "Trackers loaded"
    WebsiteLoader-->>User: "Page rendered with all trackers"
Loading

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

1 file reviewed, no comments

Edit Code Review Agent Settings | Greptile
React with 👍 or 👎 to share your feedback on this new summary format

@carlosabadia carlosabadia merged commit 214425c into main Nov 20, 2025
8 of 10 checks passed
@carlosabadia carlosabadia deleted the add-google-conversion-2 branch November 20, 2025 18:53
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.

3 participants