Skip to content

fix(site): enterprise page#7773

Merged
mhartington merged 1 commit intomainfrom
enterprise-page-fix
Apr 3, 2026
Merged

fix(site): enterprise page#7773
mhartington merged 1 commit intomainfrom
enterprise-page-fix

Conversation

@mhartington
Copy link
Copy Markdown
Member

@mhartington mhartington commented Apr 3, 2026

Summary by CodeRabbit

  • Enhancement
    • Updated the "Get in touch" button to navigate to a contact section within the page instead of opening an external link in a new tab, streamlining user navigation and keeping the experience on-page.

@vercel
Copy link
Copy Markdown

vercel bot commented Apr 3, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
blog Ready Ready Preview, Comment Apr 3, 2026 3:12am
docs Ready Ready Preview, Comment Apr 3, 2026 3:12am
eclipse Ready Ready Preview, Comment Apr 3, 2026 3:12am
site Ready Ready Preview, Comment Apr 3, 2026 3:12am

Request Review

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 3, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: d9e4c85d-7981-4f42-9247-709b46701a74

📥 Commits

Reviewing files that changed from the base of the PR and between 0bb5c43 and a52bc3f.

📒 Files selected for processing (1)
  • apps/site/src/app/enterprise/page.tsx

Walkthrough

The enterprise page was updated to convert an external call-to-action button link into an internal page anchor reference. The "Get in touch" button now navigates to a "Connect with us" section on the same page via the #contact-us anchor, with the corresponding section id added for routing. One minor whitespace normalization was applied.

Changes

Cohort / File(s) Summary
Enterprise Page Navigation
apps/site/src/app/enterprise/page.tsx
Updated "Get in touch" button href from external Prisma URL to internal #contact-us anchor; removed target="_blank" and rel="noopener noreferrer" attributes. Added id="contact-us" to "Connect with us" section header. Normalized JSX whitespace.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name Status Explanation Resolution
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.
Title check ❓ Inconclusive The title is vague and generic, referring only to 'enterprise page' without specifying what was actually fixed. Consider a more descriptive title like 'fix(site): update enterprise page contact link to anchor' that clarifies the specific fix being applied.
✅ Passed checks (1 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

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


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

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

@argos-ci
Copy link
Copy Markdown

argos-ci bot commented Apr 3, 2026

The latest updates on your projects. Learn more about Argos notifications ↗︎

Build Status Details Updated (UTC)
default (Inspect) ✅ No changes detected - Apr 3, 2026, 3:18 AM

@mhartington mhartington merged commit 6ddbbee into main Apr 3, 2026
17 of 18 checks passed
@mhartington mhartington deleted the enterprise-page-fix branch April 3, 2026 03:18
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