feat: Add Farmvaidya partner and update Partners component #22
Conversation
There was a problem hiding this comment.
Pull request overview
This PR adds Farmvaidya as a new sponsor partner to the Partners component by adding their logo image and creating a new partner card in the component.
Key Changes:
- Added a new sponsor section for Farmvaidya in the Partners component
- Added the Farmvaidya logo image file (farmvaidya.jpeg) to the public/partners directory
Reviewed changes
Copilot reviewed 1 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| src/components/Partners.tsx | Added new partner card for Farmvaidya sponsor with logo and label |
| public/partners/farmvaidya.jpeg | Added Farmvaidya logo image file (binary JPEG) |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| <h4 className="opacity-50 mt-4 md:mt-[26.6px] text-sm md:text-xl">Sponsor</h4> | ||
| <div className="w-[140px] md:w-[220px] h-[90px] md:h-[140px] flex items-center justify-center"> | ||
| <img | ||
| src="./partners/farmvaidya.jpeg" |
There was a problem hiding this comment.
The image source path is inconsistent with other partner images in this file. Other local images use an absolute path starting with "/" (e.g., "/partners/gitam-logo-black.png", "/partners/ananta-logo-black.png"), but this uses a relative path "./partners/farmvaidya.jpeg". For consistency and to avoid potential path resolution issues, this should use "/partners/farmvaidya.jpeg" instead.
| src="./partners/farmvaidya.jpeg" | |
| src="/partners/farmvaidya.jpeg" |
| <img | ||
| src="./partners/farmvaidya.jpeg" | ||
| className="max-w-full max-h-full object-contain" | ||
| alt="GCGC" |
There was a problem hiding this comment.
The alt text is incorrect. It says "GCGC" but this is the Farmvaidya sponsor logo. The alt text should describe the actual partner being displayed, which is "Farmvaidya".
| alt="GCGC" | |
| alt="Farmvaidya" |
No description provided.