coleygroup / coleygroup.github.io

Coley Research Group website
1 stars 12 forks source link

People pages #31

Closed connorcoley closed 2 weeks ago

connorcoley commented 3 weeks ago

Convert people page to show full photos/links/bios for everyone. Note that we need to actually populate the missing information now. If the formatting looks okay, could you consider reaching out to folks to get this information?

ykevu commented 3 weeks ago

Do we want to have everyone (or most people) update this PR with their info before pulling? Or keep placeholders first

connorcoley commented 3 weeks ago

I'd like most people to update the PR with their info before we merge it in. It would be a shame to leave so many placeholders in & blank descriptions/bios

ykevu commented 2 weeks ago

Can we merge in the .yml files without the adjustments to the actual page first? I think this might reduce confusion in having every person directly add their info / photos to this PR, then we can push the page change at the end?

connorcoley commented 2 weeks ago

I think the field name changes are kind of breaking (i.e., not compatible with the old page). We can just merge this one in and then try to minimize the amount of time we have empty placeholders