refactor: remove breadcrumb components from program pages#2324
Conversation
Removed unused Breadcrumbs component imports and implementations from the programs page and grant pages (career-transition-grant and rapid-grants). Also simplified route configuration by removing BluedotRoute types and ROUTES imports, replacing them with simple PAGE_TITLE constants where needed.
render
Bot
temporarily deployed
to
remove-program-page-breadcrumbs - bluedot-preview PR #2324
📝 WalkthroughWalkthroughThis pull request removes Possibly related PRs
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
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. Comment |
Greptile SummaryThis PR removes the Confidence Score: 5/5Safe to merge — purely cosmetic cleanup with no logic changes or broken references. All three files retain their necessary imports, no dead code was left behind, and no functionality was altered — only breadcrumb rendering and the supporting route boilerplate were removed. No files require special attention. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[Page renders] --> B[MarketingHero]
B --> C[Page content sections]
OLD["❌ Removed: Breadcrumbs component\n(route with parentPages chain)"]
B -.->|was here before| OLD
Reviews (1): Last reviewed commit: "refactor: remove breadcrumb components f..." | Re-trigger Greptile |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
🧹 Nitpick comments (2)
apps/website/src/pages/programs/rapid-grants.tsx (1)
13-45: Optional: reusePAGE_TITLEinMarketingHero.Same observation as in
career-transition-grant.tsx—MarketingHeroon line 43 duplicates the literal'Rapid Grants'already declared asPAGE_TITLE.♻️ Proposed change
<MarketingHero - title="Rapid Grants" + title={PAGE_TITLE} subtitle="Funding for the BlueDot community to ship projects, run events, and do other concrete work on AI safety." />🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@apps/website/src/pages/programs/rapid-grants.tsx` around lines 13 - 45, The MarketingHero component is using the literal 'Rapid Grants' instead of the existing PAGE_TITLE constant; update the JSX to pass PAGE_TITLE as the title prop (replace the hard-coded string in the MarketingHero title prop with PAGE_TITLE) so the component reuses the declared constant (look for PAGE_TITLE and the MarketingHero invocation).apps/website/src/pages/programs/career-transition-grant.tsx (1)
14-33: Optional: reusePAGE_TITLEinMarketingHero.
MarketingHeroon line 31 hardcodes the same'Career Transition Grants'string already captured inPAGE_TITLE. PassingPAGE_TITLEkeeps the page heading and document title in sync if the wording ever changes.♻️ Proposed change
<MarketingHero - title="Career Transition Grants" + title={PAGE_TITLE} subtitle="Funding and support to help you work full-time on AI safety." />🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@apps/website/src/pages/programs/career-transition-grant.tsx` around lines 14 - 33, The MarketingHero title is hardcoded to "Career Transition Grants" even though PAGE_TITLE already holds that string; inside the CareerTransitionGrantPage component replace the hardcoded title prop on <MarketingHero title="Career Transition Grants" .../> with the PAGE_TITLE constant so the page heading and document title stay in sync (update the MarketingHero usage to pass title={PAGE_TITLE}).
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In `@apps/website/src/pages/programs/career-transition-grant.tsx`:
- Around line 14-33: The MarketingHero title is hardcoded to "Career Transition
Grants" even though PAGE_TITLE already holds that string; inside the
CareerTransitionGrantPage component replace the hardcoded title prop on
<MarketingHero title="Career Transition Grants" .../> with the PAGE_TITLE
constant so the page heading and document title stay in sync (update the
MarketingHero usage to pass title={PAGE_TITLE}).
In `@apps/website/src/pages/programs/rapid-grants.tsx`:
- Around line 13-45: The MarketingHero component is using the literal 'Rapid
Grants' instead of the existing PAGE_TITLE constant; update the JSX to pass
PAGE_TITLE as the title prop (replace the hard-coded string in the MarketingHero
title prop with PAGE_TITLE) so the component reuses the declared constant (look
for PAGE_TITLE and the MarketingHero invocation).
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 63c7b130-c999-4756-b5c6-0a234862612d
📒 Files selected for processing (3)
apps/website/src/pages/programs.tsxapps/website/src/pages/programs/career-transition-grant.tsxapps/website/src/pages/programs/rapid-grants.tsx
1 participant
Removed unused Breadcrumbs component imports and implementations from the programs page and grant pages (career-transition-grant and rapid-grants). Also simplified route configuration by removing BluedotRoute types and ROUTES imports, replacing them with simple PAGE_TITLE constants where needed.
Description
Issue
Fixes #
Developer checklist
Screenshot