Page MenuHomePhabricator

[landing] add profile pictures from s3, remove placehodlr images
AbandonedPublic

Authored by benschac on Mar 31 2022, 12:37 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, May 2, 9:24 AM
Unknown Object (File)
Thu, May 2, 9:23 AM
Unknown Object (File)
Thu, May 2, 8:35 AM
Unknown Object (File)
Sun, Apr 28, 9:53 AM
Unknown Object (File)
Mar 31 2024, 9:00 PM
Unknown Object (File)
Mar 12 2024, 1:30 AM
Unknown Object (File)
Feb 25 2024, 6:29 PM
Unknown Object (File)
Feb 18 2024, 6:32 AM
Tokens
"Doubloon" token, awarded by ashoat.

Details

Reviewers
atul
Summary

add s3 images. should render teams head shots

Test Plan

go to /team, headshot should show

Diff Detail

Repository
rCOMM Comm
Branch
team-landing
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

benschac added a parent revision: Restricted Differential Revision.Mar 31 2022, 12:54 PM
atul requested changes to this revision.Apr 5 2022, 10:26 AM

At a high level this looks about right, but there's some cleaning up to do before it's ready for review.

landing/team.react.js
12

Shouldn't have commented out code

53

Shouldn't have commented out code.

83–89

Hm, the base of this diff doesn't seem right? Couldn't find a diff in the stack where Varun's TeamProfile block was commented out.

This revision now requires changes to proceed.Apr 5 2022, 10:26 AM

"placehodlr" – a true hodler

Closing. Changes from this diff were added to a different diff.