-
Notifications
You must be signed in to change notification settings - Fork 10
chore: update CTA from book a call to get in touch, add .gitignore #135
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: update CTA from book a call to get in touch, add .gitignore #135
Conversation
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
WalkthroughEight documentation pages updated to replace CTA copy from "Book a Call" / "Talk to an expert" to "Get in touch" / "Reach out" and swap Calendly booking links for HubSpot form URLs; also added a new .gitignore rule for a Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
Comment |
Greptile OverviewGreptile SummaryThis PR updates the call-to-action (CTA) language across 8 documentation files, changing from "book a call" with Calendly links to "get in touch" with a HubSpot form that includes UTM tracking parameters. Changes Made
Files UpdatedAll 8 target files have been successfully updated with the new CTA:
Consistency NoteThe PR description states all CTAs have been updated, but there's one remaining old Calendly link in Confidence Score: 5/5
Important Files ChangedFile Analysis
Sequence DiagramsequenceDiagram
participant Author
participant Docs
participant Marketing
Marketing->>Author: Request CTA update to "get in touch"
Author->>Docs: Update 8 files with new HubSpot form URL
Author->>Docs: Change header to "Reach out"
Author->>Docs: Update CTA text with UTM parameters
Docs-->>Marketing: New CTAs with tracking enabled
|
There was a problem hiding this 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
Additional Comments (1)
Prompt To Fix With AIThis is a comment left during a code review.
Path: docs/request-network-api/batch-payments.md
Line: 37:37
Comment:
[P2] This line still contains an old Calendly link ("Book a call") that should be updated to match the new "get in touch" CTA strategy. Update this to use the new HubSpot form URL with UTM parameters for consistency across all documentation.
How can I resolve this? If you propose a fix, please make it concise. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
docs/request-network-api/batch-payments.md (1)
37-37: Missed CTA update - blocking inconsistency.Line 37 still contains the old "Book a call" CTA with a Calendly link. This contradicts the PR objective to replace all such CTAs with the standardized "get in touch" messaging and HubSpot form URL.
📜 Review details
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (8)
docs/README.mddocs/request-network-api/batch-payments.mddocs/request-network-api/create-and-pay-requests.mddocs/request-network-api/crosschain-payments.mddocs/request-network-api/crypto-to-fiat-payments.mddocs/request-network-api/easyinvoice-api-demo-app.mddocs/request-network-api/recurring-payments.mddocs/request-network-api/supported-chains-and-currencies.md
🧰 Additional context used
📓 Path-based instructions (1)
**/*
⚙️ CodeRabbit configuration file
**/*: - Only comment on issues that would block merging — ignore minor or stylistic concerns.
- Restrict feedback to errors, security risks, or functionality-breaking problems.
- Do not post comments on code style, formatting, or non-critical improvements.
- Keep reviews short: flag only issues that make the PR unsafe to merge.
- Limit review comments to 3–5 items maximum, unless additional blockers exist.
- Group similar issues into a single comment instead of posting multiple notes.
- Skip repetition — if a pattern repeats, mention it once at a summary level only.
- Do not add general suggestions; focus strictly on merge-blocking concerns.
- If there are no critical problems, respond with minimal approval (e.g., 'Looks good'). Do not add additional review.
- Avoid line-by-line commentary unless it highlights a critical bug or security hole.
- Highlight only issues that could cause runtime errors, data loss, or severe maintainability issues.
- Ignore minor optimization opportunities — focus solely on correctness and safety.
- Provide a top-level summary of critical blockers rather than detailed per-line notes.
- Comment only when the issue must be resolved before merge — otherwise, remain silent.
- When in doubt, err on the side of fewer comments — brevity and blocking issues only.
- Avoid posting any refactoring issues
Files:
docs/request-network-api/easyinvoice-api-demo-app.mddocs/request-network-api/crosschain-payments.mddocs/README.mddocs/request-network-api/create-and-pay-requests.mddocs/request-network-api/crypto-to-fiat-payments.mddocs/request-network-api/recurring-payments.mddocs/request-network-api/supported-chains-and-currencies.mddocs/request-network-api/batch-payments.md
Problem
The current CTAs across documentation use "Book a Call" with Calendly links, which needs to be updated to a unified "Get in Touch" messaging with HubSpot form URLs.
Proposed Solution
Secondary Changes
.gitignorefile with.ignore/pattern to exclude local development filesFixes RequestNetwork/public-issues#67