Skip to content

feat(landing): add rb2b#815

Merged
waleedlatif1 merged 1 commit intostagingfrom
feat/rb2b
Jul 29, 2025
Merged

feat(landing): add rb2b#815
waleedlatif1 merged 1 commit intostagingfrom
feat/rb2b

Conversation

@waleedlatif1
Copy link
Collaborator

Description

add rb2b only for our platform, not for self-hosted users. also modified the vercel analytics to run on our hosted app

Type of change

  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

N/A

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have added tests that prove my fix is effective or that my feature works
  • All tests pass locally and in CI (bun run test)
  • My changes generate no new warnings
  • Any dependent changes have been merged and published in downstream modules
  • I have updated version numbers as needed (if needed)
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

Security Considerations:

  • My changes do not introduce any new security vulnerabilities
  • I have considered the security implications of my changes

@vercel
Copy link

vercel bot commented Jul 29, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
sim 🔄 Building (Inspect) Visit Preview 💬 Add feedback Jul 29, 2025 3:16am
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
docs ⬜️ Skipped (Inspect) Jul 29, 2025 3:16am

@waleedlatif1 waleedlatif1 merged commit 08720d9 into staging Jul 29, 2025
3 of 4 checks passed
@waleedlatif1 waleedlatif1 deleted the feat/rb2b branch July 29, 2025 03:16
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.

Greptile Summary

This PR adds RB2B website visitor identification tracking to the hosted version of Sim Studio. RB2B is a B2B lead generation service that identifies individual website visitors by matching their digital fingerprints with LinkedIn profiles, providing sales teams with person-level identification data rather than just company-level analytics.

The implementation includes two key changes to apps/sim/app/layout.tsx:

  1. RB2B Script Integration: A new tracking script is injected into the HTML head that loads asynchronously from AWS S3. The script follows RB2B's standard snippet pattern with methods for identify and collect, and includes proper version tracking.

  2. Analytics Simplification: The complex shouldEnableAnalytics() logic has been replaced with a simpler isHosted check for both Vercel Analytics and SpeedInsights components. This creates consistency across all tracking services.

Both changes are gated behind the isHosted condition, ensuring that only the main hosted platform (not self-hosted installations) sends visitor data to these third-party services. This approach maintains user privacy for self-hosted deployments while enabling powerful lead generation capabilities for the main SaaS offering. The RB2B integration fits into the existing analytics infrastructure by following the same conditional loading pattern already established for other tracking services.

Confidence score: 4/5

• This PR is generally safe to merge with proper monitoring of the third-party script loading
• The code follows established patterns and includes appropriate privacy controls, but introduces a third-party dependency that should be monitored
• The layout.tsx file needs attention to ensure the RB2B script loads correctly and doesn't impact performance

1 file reviewed, no comments

Edit Code Review Bot Settings | Greptile

arenadeveloper02 pushed a commit to arenadeveloper02/p2-sim that referenced this pull request Sep 19, 2025
Co-authored-by: waleedlatif <waleedlatif@waleedlatifs-MacBook-Pro.local>
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.

1 participant