Skip to content

feat: DR-7666 homepage#7689

Open
carlagn wants to merge 23 commits intomainfrom
feat/DR-7666-homepage
Open

feat: DR-7666 homepage#7689
carlagn wants to merge 23 commits intomainfrom
feat/DR-7666-homepage

Conversation

@carlagn
Copy link
Contributor

@carlagn carlagn commented Mar 23, 2026

Fixes #DR-7666

Summary by CodeRabbit

  • Chores
    • Updated project dependencies to add 3D rendering, HTML parsing, analytics, and CSS class-merging libraries, plus related type definitions and build tooling.
  • Navigation
    • Simplified top-level navigation: renamed a top item to "Docs", replaced an external "Get started" link with an internal "Prisma Partners" link, and removed an obsolete commented link.

@vercel
Copy link

vercel bot commented Mar 23, 2026

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

Project Deployment Actions Updated (UTC)
blog Ready Ready Preview, Comment Mar 23, 2026 5:13pm
docs Ready Ready Preview, Comment Mar 23, 2026 5:13pm
eclipse Ready Ready Preview, Comment Mar 23, 2026 5:13pm
site Ready Ready Preview, Comment Mar 23, 2026 5:13pm

Request Review

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 23, 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: 06f72730-1f1e-45e0-8de2-b9531516cd51

📥 Commits

Reviewing files that changed from the base of the PR and between 5fcac92 and 872741f.

📒 Files selected for processing (3)
  • apps/blog/src/app/(blog)/layout.tsx
  • packages/ui/src/components/navigation-menu.tsx
  • packages/ui/src/components/web-navigation.tsx
✅ Files skipped from review due to trivial changes (1)
  • apps/blog/src/app/(blog)/layout.tsx

Walkthrough

Added several runtime and dev dependencies to apps/site/package.json (Three.js ecosystem, parser, utilities, analytics, typings) and updated navigation links and prop formatting in apps/blog/src/app/(blog)/layout.tsx.

Changes

Cohort / File(s) Summary
Site package manifest
apps/site/package.json
Added runtime deps: @react-three/fiber, three, html-react-parser, tailwind-merge, posthog-js (catalog spec); added dev dep @types/three; reordered/reintroduced babel-plugin-react-compiler and typescript entries.
Blog layout / Navigation
apps/blog/src/app/(blog)/layout.tsx
Removed a commented “Management API” nav entry; replaced Resources sub-link Get started (external) with internal /partners Prisma Partners; changed top-level PartnersDocs (/docs); minor prop/formatting reflow.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 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 'feat: DR-7666 homepage' is vague and lacks descriptive detail about what actually changed in the homepage implementation. Consider a more specific title that describes the primary change, such as 'feat: update homepage navigation structure and add three.js rendering support' or similar.
✅ 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.


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

@argos-ci
Copy link

argos-ci bot commented Mar 23, 2026

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

Build Status Details Updated (UTC)
default (Inspect) ✅ No changes detected - Mar 23, 2026, 5:19 PM

carlagn added 3 commits March 23, 2026 15:37
Remove safelist and add it in global css file
Add homepage
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