Skip to content

225 - Updated staff card component#228

Merged
Vaibhav978 merged 3 commits into
mainfrom
225-update-staff-card
Jun 15, 2026
Merged

225 - Updated staff card component#228
Vaibhav978 merged 3 commits into
mainfrom
225-update-staff-card

Conversation

@mehanana

@mehanana mehanana commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

ℹ️ Issue

Closes #225

📝 Description

Updated the staff card component to match the new figma design

Briefly list the changes made to the code:

  1. Added new email & title props to display the staff's email and role
  2. Changed the layout of the card to match the new figma design
  3. Added new tests and modified existing tests for this component

✔️ Verification

Ran the tests to make sure everything passed. Also looked at the staff card to ensure it visually matched the figma.
How it looks on local host:
Screenshot 2026-06-08 225147
Tests for Staff Card Component:
Screenshot 2026-06-08 230108

Provide screenshots of any new components, styling changes, or pages.

🏕️ (Optional) Future Work / Notes

Currently, the props for title and image are optional, but I was wondering if the title should be required for every staff member?

@mehanana mehanana requested a review from nourshoreibah as a code owner June 9, 2026 03:07
@github-actions github-actions Bot requested a review from Vaibhav978 June 9, 2026 03:07
github-actions Bot added a commit that referenced this pull request Jun 9, 2026
mehanana added 2 commits June 14, 2026 17:41
…ctured accountspage into its own page folder instead of keeping it in the components folder
@Vaibhav978 Vaibhav978 added this pull request to the merge queue Jun 15, 2026
github-actions Bot added a commit that referenced this pull request Jun 15, 2026
Merged via the queue into main with commit b423404 Jun 15, 2026
15 checks passed
@Vaibhav978 Vaibhav978 deleted the 225-update-staff-card branch June 15, 2026 02:06
github-actions Bot added a commit that referenced this pull request Jun 15, 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.

Update the staff card component

3 participants