Skip to content

chore: update CTA from book a demo to get in touch, add .gitignore entries#190

Merged
MantisClone merged 2 commits intomainfrom
01-12-chore_update_cta_from_book_a_demo_to_get_in_touch
Jan 15, 2026
Merged

chore: update CTA from book a demo to get in touch, add .gitignore entries#190
MantisClone merged 2 commits intomainfrom
01-12-chore_update_cta_from_book_a_demo_to_get_in_touch

Conversation

@MantisClone
Copy link
Member

@MantisClone MantisClone commented Jan 13, 2026

Problem

The current CTA uses "Book A Demo" with Calendly links, which needs to be updated to a unified "Get in Touch" messaging with HubSpot form URLs.

Proposed Solution

  • Updated header component to display "Get in touch" instead of "Book A Demo"
  • Added NEXT_PUBLIC_DEMO_MEETING environment variable with HubSpot form URL including UTM tracking parameters

Secondary Changes

  • Added .ignore/ and .github/copilot-instructions.md patterns to .gitignore to exclude local development files

Fixes RequestNetwork/public-issues#67

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 13, 2026

Walkthrough

The changes update CTA messaging and project configuration to align with a "Get in Touch" strategy. An environment variable for demo meeting URL is added to configuration, .gitignore rules are expanded with new exclusions, and header component text is updated from "Book A Demo" to "Get in touch" when a demo URL is available.

Changes

Cohort / File(s) Summary
Configuration
.env.example, .gitignore
Added NEXT_PUBLIC_DEMO_MEETING environment variable and two new .gitignore entries (.ignore/ directory and .github/copilot-instructions.md file).
Component Updates
src/components/header.tsx
Replaced "Book A Demo" text with "Get in touch" in two Link components (conditional on demo meeting URL availability).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 3
❌ Failed checks (3 warnings)
Check name Status Explanation Resolution
Linked Issues check ⚠️ Warning The PR partially addresses issue #67 by updating the header CTA text and adding the required environment variable, but only applies changes to easy-invoice without covering other products (docs.request.network, mintlify-docs, rn-checkout, invoicing-template, request-api-portal) listed in the issue requirements. Verify that this PR is scoped only to easy-invoice per a product-specific strategy, or coordinate updates across all six products specified in issue #67.
Out of Scope Changes check ⚠️ Warning The addition of .ignore/ and .github/copilot-instructions.md to .gitignore appears unrelated to the stated CTA update objective and issue #67 requirements, introducing scope beyond the specified changes. Remove the .ignore/ and .github/copilot-instructions.md entries from .gitignore or provide justification for their inclusion in this PR.
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately reflects the main changes: updating CTA text and adding gitignore entries, both of which are present in the changeset.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

Comment @coderabbitai help to get the list of available commands and usage tips.

@MantisClone MantisClone marked this pull request as ready for review January 13, 2026 02:31
@greptile-apps
Copy link

greptile-apps bot commented Jan 13, 2026

Greptile Overview

Greptile Summary

This PR updates the call-to-action (CTA) button from "Book A Demo" to "Get in touch" and replaces the Calendly link with a new HubSpot form URL that includes UTM tracking parameters.

Changes Made

  1. Header Component (src/components/header.tsx): Updated button text from "Book A Demo" to "Get in touch" in both logged-in and logged-out user states
  2. Environment Variables (.env.example): Added NEXT_PUBLIC_DEMO_MEETING with the HubSpot form URL including UTM parameters (utm_source, utm_medium, utm_campaign, utm_content)
  3. Git Ignore (.gitignore): Added .ignore/ directory and .github/copilot-instructions.md to prevent tracking of local/personal development files

Integration Notes

The environment variable is already properly validated in src/lib/env/client.mjs as an optional URL field. The header component conditionally renders the CTA only when NEXT_PUBLIC_DEMO_MEETING is defined, so the button will not appear in environments where this variable is not set.

Issue Found

There's an inconsistency in .env.example: the new NEXT_PUBLIC_DEMO_MEETING variable is uncommented while other optional variables in the same section (NEXT_PUBLIC_GTM_ID, NEXT_PUBLIC_CRYPTO_TO_FIAT_TRUSTED_ORIGINS) are commented out. This should be fixed for consistency.

Confidence Score: 4/5

  • This PR is safe to merge with one minor style fix needed
  • The changes are straightforward and well-scoped: updating button text, adding a new environment variable, and updating gitignore. The environment variable is already properly validated in the codebase. The only issue is a style inconsistency in .env.example where the new optional variable should be commented out to match the pattern of other optional variables. This is a minor issue that doesn't affect functionality but should be fixed for consistency and clarity.
  • .env.example needs the NEXT_PUBLIC_DEMO_MEETING variable to be commented out for consistency with other optional variables

Important Files Changed

File Analysis

Filename Score Overview
.env.example 3/5 Added NEXT_PUBLIC_DEMO_MEETING variable with HubSpot form URL including UTM parameters; inconsistent with other optional variables which are commented out
.gitignore 5/5 Added .ignore/ directory and .github/copilot-instructions.md to gitignore - appropriate for local/personal files
src/components/header.tsx 5/5 Updated CTA text from 'Book A Demo' to 'Get in touch' in both logged-in and logged-out states - clean change, no issues

Sequence Diagram

sequenceDiagram
    participant User
    participant Header
    participant Env as Environment Variables
    participant HubSpot as HubSpot Form

    User->>Header: Visits page
    Header->>Env: Read NEXT_PUBLIC_DEMO_MEETING
    
    alt Variable is set
        Env-->>Header: Return HubSpot URL with UTM params
        Header->>Header: Render "Get in touch" button/link
        User->>Header: Clicks "Get in touch"
        Header->>HubSpot: Navigate to form URL with UTM tracking
        Note over HubSpot: utm_source=request.network<br/>utm_medium=easy-invoice<br/>utm_campaign=evergreen<br/>utm_content=get_in_touch
        HubSpot-->>User: Display contact form
    else Variable not set
        Env-->>Header: Return undefined
        Header->>Header: Hide CTA button
        Note over Header: Button not rendered
    end
Loading

Copy link

@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, 1 comment

Edit Code Review Agent Settings | Greptile

@MantisClone MantisClone changed the title chore: update CTA from book a demo to get in touch chore: update CTA from book a demo to get in touch, add .gitignore entries Jan 13, 2026
Copy link
Contributor

@bassgeta bassgeta left a comment

Choose a reason for hiding this comment

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

Nice and easy 👌

@MantisClone MantisClone merged commit c9c273e into main Jan 15, 2026
5 checks passed
@MantisClone MantisClone deleted the 01-12-chore_update_cta_from_book_a_demo_to_get_in_touch branch January 15, 2026 03:45
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.

[CTA Update] - Update all products from 'Book a Call' to 'Get in Touch'

2 participants