Skip to content

Update Our Team Page #1862 #1865

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

franciscusagnew
Copy link
Contributor

Description of changes

Updated Our Team Page info as requested in Issue #1862

Note:

  • Not able to update the photos in the request because someone with access to our Amazon S3 service needs to add them first. Once Cynthia's photo is added, its current link should work. However, for Glomani Bravo-Lopez the link will need to be added after the photo is uploaded.

  • Having bios of varying lengths causes the page to look inconsistent and disjointed. I will submit a request for a redesign to handle this problem in the interim.

Issue Resolved

Fixes #1862

Screenshots/GIFs

Our Team

Copy link

vercel bot commented May 3, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
operation-code ❌ Failed (Inspect) May 23, 2025 8:12pm
storybook ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 23, 2025 8:12pm

@kylemh
Copy link
Member

kylemh commented May 3, 2025

Copy link

codeclimate bot commented May 3, 2025

Code Climate has analyzed commit f70d615 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (90% is the threshold).

This pull request will bring the total coverage in the repository to 81.1% (-4.4% change).

View more on Code Climate.

Copy link

cypress bot commented May 3, 2025

operation_code    Run #4861

Run Properties:  status check passed Passed #4861  •  git commit a34b806f2d: Merge branch 'main' into main
Project operation_code
Branch Review pull/1865
Run status status check passed Passed #4861
Run duration 03m 14s
Commit git commit a34b806f2d: Merge branch 'main' into main
Committer Cynthia S. Kao
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 25
View all changes introduced in this branch ↗︎

@datphotogurl
Copy link

datphotogurl commented May 23, 2025

Hi folks @franciscusagnew Glomani Bravo-Lopez is not the Executive Director, please remove that title and add his photo if you can. Please also move him to the bottom as the newest member, and move Ali Cipolla-Taylor to beside me since she is the Vice-Chair. Thank you!

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.

Update Our Team Page
3 participants