Skip to content

Change Structure#6

Merged
vitusbenson merged 7 commits intomainfrom
vb/dev
Jul 16, 2025
Merged

Change Structure#6
vitusbenson merged 7 commits intomainfrom
vb/dev

Conversation

@vitusbenson
Copy link
Copy Markdown
Contributor

No description provided.

@vitusbenson vitusbenson requested a review from lazarusA July 16, 2025 14:36
@vitusbenson vitusbenson requested a review from Copilot July 16, 2025 14:43
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.ts to spotlight team and science projects and expanded resources.
  • Updated CardProfile component 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=""
Copy link

Copilot AI Jul 16, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Empty photo prop may result in broken images. Provide a valid placeholder image or remove the prop to prevent rendering issues.

Suggested change
photo=""
photo="https://via.placeholder.com/150"

Copilot uses AI. Check for mistakes.
Comment on lines +11 to +19
link: 'https://github.com/johndoe'
},
{
icon: 'linkedin',
link: 'https://linkedin.com/in/johndoe'
},
{
icon: 'twitter',
link: 'https://twitter.com/johndoe'
Copy link

Copilot AI Jul 16, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[nitpick] Placeholder social link 'johndoe' appears across multiple profiles. Update these to real links or ensure placeholders are clearly marked to avoid confusion.

Suggested change
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'

Copilot uses AI. Check for mistakes.
@vitusbenson
Copy link
Copy Markdown
Contributor Author

any idea why i can't use { in my markdown files @lazarusA ?
See ./src/app/resources/datasets/earthnet2021/page.mdx page.mdx:469:35: Could not parse expression with acorn

Copy link
Copy Markdown
Member

@lazarusA lazarusA left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Merging this now. Too many changes already.

@vitusbenson vitusbenson merged commit 4574fc7 into main Jul 16, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants