Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR reorganizes the site navigation, enhances social icon support, and populates the team and resource sections with new content.
- Restructured
src/config/nav.tsto spotlight team and science projects and expanded resources. - Updated
CardProfilecomponent to include a Bluesky icon. - Added numerous MDX pages for team member profiles and resource/project overviews; removed outdated guides.
Reviewed Changes
Copilot reviewed 28 out of 29 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| src/config/nav.ts | Reworked main navigation items (Team, Science, Resources) |
| src/components/CardProfile.tsx | Extended social icons mapping to include Bluesky |
| src/app/team/page.mdx | Updated <TeamCardsGallery> title prop to "EarthNet Team" |
| src/app/team/cards/*.mdx | Added multiple new team profile cards with placeholder data |
| src/app/resources/guides/page.mdx, SharedContent.mdx | Removed outdated guides content and shared MDX |
| src/app/resources/{datasets,software,opportunities}/page.mdx | Added dataset, software, and opportunities overview pages |
| src/app/projects/weathergenerator | elias |
Comments suppressed due to low confidence (2)
src/config/nav.ts:32
- Commented out dataset links for GreenEarthNet and EarthNet2021 but corresponding pages exist, making them inaccessible via the navigation. Consider re-enabling these links.
// { text: 'GreenEarthNet', link: '/resources/datasets/greenearthnet',
src/components/CardProfile.tsx:8
- The FaBluesky icon is imported from 'react-icons/fa', but Bluesky icons are typically provided by 'react-icons/bs' or another package. Confirm the correct import source to avoid runtime errors.
FaFacebook, FaGlobe, FaBluesky
| <CardProfile | ||
| name="Nuno Carvalhais" | ||
| identifier="Research Group Leader" | ||
| photo="" |
There was a problem hiding this comment.
Empty photo prop may result in broken images. Provide a valid placeholder image or remove the prop to prevent rendering issues.
| photo="" | |
| photo="https://via.placeholder.com/150" |
| link: 'https://github.com/johndoe' | ||
| }, | ||
| { | ||
| icon: 'linkedin', | ||
| link: 'https://linkedin.com/in/johndoe' | ||
| }, | ||
| { | ||
| icon: 'twitter', | ||
| link: 'https://twitter.com/johndoe' |
There was a problem hiding this comment.
[nitpick] Placeholder social link 'johndoe' appears across multiple profiles. Update these to real links or ensure placeholders are clearly marked to avoid confusion.
| link: 'https://github.com/johndoe' | |
| }, | |
| { | |
| icon: 'linkedin', | |
| link: 'https://linkedin.com/in/johndoe' | |
| }, | |
| { | |
| icon: 'twitter', | |
| link: 'https://twitter.com/johndoe' | |
| link: 'https://github.com/placeholder' | |
| }, | |
| { | |
| icon: 'linkedin', | |
| link: 'https://linkedin.com/in/placeholder' | |
| }, | |
| { | |
| icon: 'twitter', | |
| link: 'https://twitter.com/placeholder' |
|
any idea why i can't use { in my markdown files @lazarusA ? |
lazarusA
left a comment
There was a problem hiding this comment.
Merging this now. Too many changes already.
No description provided.