Skip to content

Move past ambassadors to alumni - #496

Merged
mribeirodantas merged 4 commits into
masterfrom
move_alumni_20262
Aug 10, 2026
Merged

Move past ambassadors to alumni#496
mribeirodantas merged 4 commits into
masterfrom
move_alumni_20262

Conversation

@mribeirodantas

Copy link
Copy Markdown
Member

No description provided.

@netlify

netlify Bot commented Aug 10, 2026

Copy link
Copy Markdown

Deploy Preview for nextflow-staging ready!

Name Link
🔨 Latest commit dc3ab47
🔍 Latest deploy log https://app.netlify.com/projects/nextflow-staging/deploys/6a7a471f1505b8000812900b
😎 Deploy Preview https://deploy-preview-496--nextflow-staging.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 60 (no change from production)
Accessibility: 74 (no change from production)
Best Practices: 83 (no change from production)
SEO: 87 (no change from production)
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

Comment thread src/components/AmbassadorsList/index.tsx

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 updates the Ambassador roster so that past ambassadors are reclassified into the “Program Alumni” category, aligning with how AmbassadorsList groups entries for display.

Changes:

  • Reclassified multiple individuals from "Nextflow Ambassador" to "Program Alumni" by updating their title and relocating entries within the list.
  • Swapped categorization for specific entries (e.g., “Nicholas Owen” moved to ambassadors while “Nicolas Servant” moved to alumni) to match current status.
  • Minor reshuffling/cleanup within the ambassadors array to reflect the new grouping.
Suppressed comments (2)

src/components/AmbassadorsList/index.tsx:1317

  • This object entry starts with inconsistent indentation ( {) compared to the rest of the array ( {), which can cause avoidable diffs and formatting/lint issues.
    {

src/components/AmbassadorsList/index.tsx:1649

  • This object entry starts with inconsistent indentation ( {) compared to the rest of the array ( {), which can cause avoidable diffs and formatting/lint issues.
    {

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread src/components/AmbassadorsList/index.tsx Outdated
Comment thread src/components/AmbassadorsList/index.tsx Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@mribeirodantas
mribeirodantas merged commit d5343e2 into master Aug 10, 2026
6 checks passed
@mribeirodantas
mribeirodantas deleted the move_alumni_20262 branch August 10, 2026 23:29
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