Skip to content

Add profile picture for Raphina Liu - #9

Merged
monperrus merged 2 commits into
masterfrom
add-raphina-profile
Aug 24, 2026
Merged

Add profile picture for Raphina Liu#9
monperrus merged 2 commits into
masterfrom
add-raphina-profile

Conversation

@sofiabobadilla

Copy link
Copy Markdown
Contributor

Adds img/people/raphina.jpeg (200x200, 9.8 KB) and the matching image: key in _data/people.yml, so the People page renders the photo instead of the initials placeholder.

The image is already square, so it drops straight into the 70px circle with no cropping needed.

Note on #8

This branches off master, so it keeps Raphina's current role: engineer. #8 changes that same entry to role: grad — whichever merges second may need a small conflict resolution on those adjacent lines. The intended end state is both: role: grad and the image: key.

Test plan

Verified with a local jekyll serve: the entry renders as <img class="rounded-circle profile"> and the image serves 200.

🤖 Generated with Claude Code

Add img/people/raphina.jpeg (200x200) and reference it from people.yml so the People page shows the photo instead of the initials placeholder.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@monperrus

Copy link
Copy Markdown
Contributor

@copilot resolve the merge conflicts in this pull request please make no mistake

Co-authored-by: monperrus <803666+monperrus@users.noreply.github.com>

Copilot AI commented Aug 24, 2026

Copy link
Copy Markdown

@copilot resolve the merge conflicts in this pull request please make no mistake

Done. Merged master into this branch and resolved the conflict in _data/people.yml by keeping both role: grad (from #8) and image: "/img/people/raphina.jpeg" (from this PR). Merge commit: d8acf88.

Copilot AI requested a review from monperrus August 24, 2026 07:35
@monperrus
monperrus merged commit 08e6204 into master Aug 24, 2026
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