Adding companies section to the landing page#464
Adding companies section to the landing page#464Akshat-Raj wants to merge 6 commits intopointblank-club:stagingfrom
Conversation
There was a problem hiding this comment.
Pull request overview
This PR adds a "Partner Companies" section to the landing page showcasing six major tech companies (Apple, Google, Amazon, Microsoft, Visa, AMD) that collaborate with the organization through mentorship, hackathons, and student programs.
Key Changes:
- Created a new
Companiescomponent with company logos and descriptions - Integrated the component into the landing page between Founder and Achievements sections
- Added company logo assets for all six partner companies
Reviewed changes
Copilot reviewed 2 out of 9 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
components/companies.tsx |
New component displaying partner companies with logos, names, and descriptions. Includes error handling with fallback to initials. |
app/(default)/page.tsx |
Added Companies component import and rendered it in the page layout |
public/images/companies/*.png/*.webp |
Added logo images for Apple, Google, Amazon, Microsoft, Visa, and AMD |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| {companyData.map((company, index) => ( | ||
| <div key={index} className="w-full sm:w-1/2 md:w-1/3 lg:w-1/4 flex-1 text-center space-y-4"> |
There was a problem hiding this comment.
Using array index as the key prop is an anti-pattern when the list is static and won't be reordered. While not causing bugs here, it's better practice to use a stable unique identifier like the company name instead of the index.
| {companyData.map((company, index) => ( | |
| <div key={index} className="w-full sm:w-1/2 md:w-1/3 lg:w-1/4 flex-1 text-center space-y-4"> | |
| {companyData.map((company) => ( | |
| <div key={company.name} className="w-full sm:w-1/2 md:w-1/3 lg:w-1/4 flex-1 text-center space-y-4"> |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
|
Logo sizes varying too much, why is amazon logo in a pill make them all of same size |
|
LGTM for me |

Summary: This PR adds a section for the companies PB is partnered with on the landing page
Which issue(s) this PR fixes:
Changes Made
Describe the changes you've made in this PR:
Type of Change
How Has This Been Tested?
Describe the tests you ran:
Screenshots (if applicable)