Skip to content

fix: hide Omar Metwally from alumni - #19

Merged
chengjiahao1234 merged 1 commit into
mainfrom
hotfix/hide-omar-alumni
Aug 8, 2026
Merged

fix: hide Omar Metwally from alumni#19
chengjiahao1234 merged 1 commit into
mainfrom
hotfix/hide-omar-alumni

Conversation

@chengjiahao1234

Copy link
Copy Markdown
Collaborator

Type of Change

  • ✅ Content Update (news, publications, team)

Summary

Commented out Omar Metwally's alumni record so he is no longer displayed on the team page while retaining his information for future reference.

Changes Made

  • Commented out Omar Metwally's complete record in _data/alumni_members.yml.
  • Retained his position and profile links in the source file.
  • Confirmed that Omar no longer appears in the rendered alumni list.

Testing Checklist

  • Changes display correctly on desktop (1920x1080 or similar)
  • Changes display correctly on tablet (768x1024 or similar)
  • Changes display correctly on mobile (375x667 or similar)
  • Tested in light theme
  • Tested in dark theme
  • No broken links or images
  • No console errors
  • Ran bundle exec jekyll serve locally without errors

Screenshots (if applicable)

Not applicable; this update removes an alumni entry from display.

Related Issues

None.

Additional Notes

  • Omar's information was commented out rather than deleted.
  • _data/alumni_members.yml parses successfully.
  • The production Jekyll build completed successfully.
  • The local /team/ page returned HTTP 200 and did not contain Omar's name.

By submitting this PR, I confirm:

  • I have tested these changes locally
  • I have followed the code standards in DEVELOPMENT.md
  • I have updated documentation if needed
  • I am ready for this to be reviewed and merged

@chengjiahao1234
chengjiahao1234 merged commit 047ed39 into main Aug 8, 2026
1 check passed
@chengjiahao1234
chengjiahao1234 deleted the hotfix/hide-omar-alumni branch August 8, 2026 23:23
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.

1 participant