Skip to content

Comments

Revert tracking#313

Merged
nk-ag merged 3 commits intoexospherehost:mainfrom
nk-ag:reverttracking
Aug 28, 2025
Merged

Revert tracking#313
nk-ag merged 3 commits intoexospherehost:mainfrom
nk-ag:reverttracking

Conversation

@nk-ag
Copy link
Contributor

@nk-ag nk-ag commented Aug 28, 2025

No description provided.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Aug 28, 2025

📝 Walkthrough

Summary by CodeRabbit

  • Performance
    • Slightly faster initial page loads by removing an unnecessary third‑party script.
  • Privacy
    • Reduced third‑party tracking surface area; only core analytics remain.
  • Chores
    • Removed a legacy loader from the app layout to streamline the landing page.

Walkthrough

RB2BLoader component was removed and its import/usage deleted from the app layout. This eliminates client-side injection of the RB2B script on pathname changes. GoogleAnalytics remains in place. No exported/public API in layout changed.

Changes

Cohort / File(s) Summary of Changes
Layout update
landing-page/src/app/layout.tsx
Removed import and JSX usage of RB2BLoader; no other layout modifications.
Component removal
landing-page/src/components/RB2BLoader.tsx
Deleted RB2BLoader component that injected the rb2b script on route changes.

Sequence Diagram(s)

sequenceDiagram
  autonumber
  actor User
  participant App as Next.js App Router
  participant Layout as RootLayout
  participant GA as GoogleAnalytics
  participant RB as RB2B Script Loader (removed)

  rect rgb(240,248,255)
  note over User,App: Before (previous behavior)
  User->>App: Navigate / route change
  App->>Layout: Render RootLayout
  Layout->>GA: Initialize analytics
  Layout->>RB: useEffect on pathname<br/>remove old script, append new rb2b script
  RB-->>Layout: Script loaded asynchronously
  end

  rect rgb(245,245,245)
  note over User,App: After (current behavior)
  User->>App: Navigate / route change
  App->>Layout: Render RootLayout
  Layout->>GA: Initialize analytics
  note right of Layout: RB2B script injection no longer executed
  end
Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

A bunny once loaded a script with glee,
Now lighter it hops, as nimble as can be.
No tags to append, no DOM to tweak—
Just carrots, GA, and a simpler streak.
Thump-thump! Onward to shipping spree. 🥕✨


📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: ASSERTIVE

Plan: Pro

💡 Knowledge Base configuration:

  • MCP integration is disabled by default for public repositories
  • Jira integration is disabled by default for public repositories
  • Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 518c9bb and b487523.

📒 Files selected for processing (2)
  • landing-page/src/app/layout.tsx (0 hunks)
  • landing-page/src/components/RB2BLoader.tsx (0 hunks)
💤 Files with no reviewable changes (2)
  • landing-page/src/app/layout.tsx
  • landing-page/src/components/RB2BLoader.tsx
✨ Finishing Touches
  • 📝 Generate Docstrings
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share
🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

  • Review comments: Directly reply to a review comment made by CodeRabbit. Example:
    • I pushed a fix in commit <commit_id>, please review it.
    • Open a follow-up GitHub issue for this discussion.
  • Files and specific lines of code (under the "Files changed" tab): Tag @coderabbit in a new review comment at the desired location with your query.
  • PR comments: Tag @coderabbit in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
    • @coderabbit gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
    • @coderabbit read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.

Support

Need help? Create a ticket on our support page for assistance with any issues or questions.

CodeRabbit Commands (Invoked using PR/Issue comments)

Type @coderabbit help to get the list of available commands.

Other keywords and placeholders

  • Add @coderabbit ignore or @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
  • Add @coderabbit summary or @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
  • Add @coderabbit or @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (.coderabbit.yaml)

  • You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
  • Please see the configuration documentation for more information.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Status, Documentation and Community

  • Visit our Status Page to check the current availability of CodeRabbit.
  • Visit our Documentation for detailed information on how to use CodeRabbit.
  • Join our Discord Community to get help, request features, and share feedback.
  • Follow us on X/Twitter for updates and announcements.

Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Summary of Changes

Hello @nk-ag, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

This pull request reverts a previous change by removing the RB2BLoader component and its integration from the landing page application. This effectively discontinues the tracking functionality associated with this component.

Highlights

  • Tracking Component Removal: The RB2BLoader component, responsible for tracking, has been entirely removed from the codebase.
  • Layout Integration Cleanup: The landing-page/src/app/layout.tsx file has been updated to remove all references and usage of the RB2BLoader.
Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point in your pull request via creating an issue comment (i.e. comment on the pull request page) using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in issue comments and review comments.
Help /gemini help Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution.

Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request removes the RB2BLoader component to revert tracking functionality. The changes are straightforward and correctly implemented. My review includes one suggestion for a follow-up action to ensure all related configuration is also cleaned up, which will improve project maintainability.


const script = document.createElement('script');
script.id = 'rb2b-script';
script.src = `https://ddwl4m2hdecbv.cloudfront.net/b/${process.env.NEXT_PUBLIC_RB2B_ID}/${process.env.NEXT_PUBLIC_RB2B_ID}.js.gz`;
Copy link
Contributor

Choose a reason for hiding this comment

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

medium

With the removal of this component, the NEXT_PUBLIC_RB2B_ID environment variable is likely no longer used. To keep the project configuration clean and prevent future confusion, it's recommended to remove this variable from your environment settings (e.g., .env files or your deployment platform's configuration).

@nk-ag nk-ag merged commit bdf949a into exospherehost:main Aug 28, 2025
3 checks passed
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