Biosecurity lander: expandable "Who this course is for" personas - #2838
Conversation
…sonas Replace the three static audience cards on the biosecurity course lander with five expandable persona panels, reusing the PersonasSection component already used by the technical AI safety lander. All panels collapsed by default. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
📝 WalkthroughWalkthroughThe Biosecurity course content now uses a Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ 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 SummaryUpdates the biosecurity course lander’s audience section.
Confidence Score: 5/5The PR appears safe to merge with no blocking or non-blocking issues identified. The persona data satisfies the existing section contract, the all-collapsed state is supported, and the new icon exports are compatible with the installed dependency.
|
| Filename | Overview |
|---|---|
| apps/website/src/components/lander/course-content/BioSecurityContent.tsx | The new persona configuration matches the existing component contract and established lander pattern, with no actionable defects identified. |
Reviews (1): Last reviewed commit: "Biosecurity lander: switch "Who this cou..." | Re-trigger Greptile
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@apps/website/src/components/lander/course-content/BioSecurityContent.tsx`:
- Line 76: Update the audience title in BioSecurityContent to use the complete
wording “High-potential people from any field,” preserving the existing title
field and surrounding content.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro Plus
Run ID: ba5c7a92-f664-4e9a-b9da-6c23e85fd49d
📒 Files selected for processing (1)
apps/website/src/components/lander/course-content/BioSecurityContent.tsx
| boldText: 'Entrepreneurs', | ||
| description: 'who want to build new pandemic defences.', | ||
| icon: PiRocketLaunch, | ||
| title: 'High-potential people, any field', |
There was a problem hiding this comment.
📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win
Use the complete audience label.
The title is grammatically incomplete. Use High-potential people from any field so the persona scope is clear.
Proposed copy fix
- title: 'High-potential people, any field',
+ title: 'High-potential people from any field',📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| title: 'High-potential people, any field', | |
| title: 'High-potential people from any field', |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@apps/website/src/components/lander/course-content/BioSecurityContent.tsx` at
line 76, Update the audience title in BioSecurityContent to use the complete
wording “High-potential people from any field,” preserving the existing title
field and surrounding content.
What
Swaps the biosecurity course lander's "Who this course is for" section from the three static audience cards (
whoIsThisFor) to five expandable persona panels (personas) — the same accordion pattern already used on the technical AI safety lander.The five profiles:
PiCpu)PiWrench)PiFlask)PiScales)PiRocketLaunch)defaultExpandedIndex: -1).BIOSECURITY_COLORS.iconBackground(#316761).How
apps/website/src/components/lander/course-content/BioSecurityContent.tsx.PersonasSectioncomponent — no component/logic/style changes.CourseLanderalready preferspersonasoverwhoIsThisFor, so no wiring changes needed.PiBriefcase/PiCompassimports; addedPiCpu/PiScales/PiWrench.Notes for reviewers
🤖 Generated with Claude Code