Repository navigation
Use shared breadcrumbs on bespoke partner pages - #1464
Conversation
|
|
No actionable comments were generated in the recent review.
|
| Layer / File(s) | Summary |
|---|---|
Partner route breadcrumb integration src/routes/partners.netlify.tsx, src/routes/partners.railway.tsx, src/routes/partners.render.tsx |
The three routes replace inline partner navigation with the shared Breadcrumbs component. Each route adds breadcrumb structured data alongside its existing partner JSON-LD. Netlify uses a fallback partner when partner data is unavailable. |
Estimated code review effort: 2 (Simple) | ~10 minutes
Change: Feature
Suggested reviewers: jhislop-design
Merge Risk: 
Minimal · up to f4549
-
The breadcrumb integration introduces no established merge-blocking correctness, availability, or data-integrity risk.
Pre-merge checks: 
4 · 
1
-
Failed checks: 1 warning Passed checks: 4 passedFull details: Docstring Coverage
-
Explanation
Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 3 functions across 3 files.
Resolution
Write docstrings for the functions missing them to satisfy the coverage threshold.
-
Finishing Touches
-
Generate docstrings
-
- Commit to this branch
- Create a new PR
Generate unit tests (beta)
-
- Commit to this branch
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
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.
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
tanstack-com | f45490d | Commit Preview URL Branch Preview URL |
Nov 01 2026, 01:58 AM |
Railway, Render and Netlify have their own partner pages, so the generic partner breadcrumb change does not reach them. Use the same semantic breadcrumb component and shared item model on those pages, including the matching WebPage BreadcrumbList.
Continues #1151 following merged #1463. Existing partner content, referral destinations and FAQ structured data are preserved. The umbrella issue stays open for the remaining library, chart and docs coverage.
Validation: full
pnpm testthrough normal commit hooks. The shared component and model were independently checked in SSR and desktop/mobile browser fixtures in #1463. All three exact-commit deployed previews passed SSR checks for canonical breadcrumbs and retained FAQ data. Railway and Render were visually checked at desktop width, Netlify at 375px with no horizontal overflow. The subsequent main integration has an identical tree, verified by an empty diff, and passed full normal commit hooks again. Exact-head CI remains required before merge.Summary by CodeRabbit