Drop the /careers page, listed roles, nav link, and About-page CTA. Redirect /careers to /about so old links do not 404.
Details
Details
- Reviewers
Fabien - Group Reviewers
Restricted Project - Commits
- rABCf128404d91a9: [e.cash] Remove careers page and job listings
- cd web/e.cash && npm test
- Confirm navbar About menu has no Careers link
- Confirm /about team section has no View Careers button
- Confirm /careers redirects to /about
Diff Detail
Diff Detail
- Repository
- rABC Bitcoin ABC
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
Comment Actions
Build Bitcoin ABC Diffs / Diff Testing (ai-review) passed.
CodeRabbit Review
Diff : committed changes only
Compare : HEAD → master
Directory : work
────────────────────────────────────────
(\(\
(• .•) Rabbits with frickin' laser beams attached to their heads. And they're pointed at your bugs.
────────────────────────────────────────
Review complete
No findings ✔
7 files reviewed:
- web/e.cash/app/careers/page.tsx
- web/e.cash/app/components/About/TeamList.tsx
- web/e.cash/app/components/Careers/CareersHero.tsx
- web/e.cash/app/components/Careers/CareersSection.tsx
- web/e.cash/app/data/careers.ts
- web/e.cash/app/data/navbarLinks.ts
- web/e.cash/app/data/redirects.ts
────────────────────────────────────────
Comment Actions
Build Bitcoin ABC Diffs / Diff Testing (preview-e.cash) passed.
Preview is available at http://51.178.130.230:41622 for the next 60 minutes.