Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add megan's info to people.yml and hover + main images to image… #267

Merged
merged 3 commits into from
Jun 14, 2024

Conversation

YUUU23
Copy link
Contributor

@YUUU23 YUUU23 commented Jun 10, 2024

add megan's information to the website

@YUUU23 YUUU23 requested review from eldu and RobertGemmaJr June 10, 2024 16:41
Copy link

github-actions bot commented Jun 10, 2024

Visit the preview URL for this PR (updated for commit 6cf8756):

https://ccv-brown-edu--pr267-add-megan-to-website-c0gp7qt9.web.app

(expires Thu, 20 Jun 2024 21:52:04 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: e6225babbaf68dadb99b2906d33811740e5d15cb

Copy link
Member

@RobertGemmaJr RobertGemmaJr left a comment

Choose a reason for hiding this comment

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

This looks great! We're just confirming what your officially title should be. Will approve once I find that out

Copy link
Member

@RobertGemmaJr RobertGemmaJr left a comment

Choose a reason for hiding this comment

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

Looks great! Apologies for the delay 🚀

@YUUU23 YUUU23 requested review from eldu and removed request for eldu June 13, 2024 21:51
Copy link
Contributor

@eldu eldu left a comment

Choose a reason for hiding this comment

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

This looks great! Feel free to merge it in!

@YUUU23 YUUU23 merged commit 52b8cf5 into main Jun 14, 2024
12 checks passed
@YUUU23 YUUU23 deleted the add-megan-to-website branch June 14, 2024 19:46
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