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

23 - Team #44

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

23 - Team #44

wants to merge 7 commits into from

Conversation

raymondyangdev
Copy link
Collaborator

@raymondyangdev raymondyangdev commented Jul 1, 2024

Context

This change is being made to ensure that all team members' contributions to the WDCC projects are properly credited on the project's details page.

Closes #23

What Changed?

  1. Created ProjectTeam component web/src/components/projects/ProjectTeam.tsx

How To Review

  1. Spin up strapi and web
  2. Navigate to the /projects to see the example usage of the ProjectTeam component.
  3. Verify that the component is displayed correctly and data is populated as expected.

Testing\

  1. Create a dummy team in Strapi
  2. Navigate to localhost:4321/projects and ensure data from Strapi is populated correctly
  3. Manually test responsiveness of component across different screen sizes.

Notes

An example usage of the ProjectTeam component has been added to our homepage to demonstrate its functionality.
This can be moved once the Projects page has been implemented.
Once Projects page is live, the component will need to be refactored to dynamically fetch and populate the team member data and images relevant to each project.

@raymondyangdev
Copy link
Collaborator Author

  • Integrated with new Project types in Strapi.
  • Example usage on /projects for the time being.

Screenshot 2024-07-10 at 2 07 39 PM

@raymondyangdev raymondyangdev marked this pull request as ready for review July 10, 2024 02:16
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.

[Project Details] Team
1 participant