Skip to content

fix: link project cards on homepage to their project pages (#726)#727

Closed
AnupamKumar-1 wants to merge 1 commit into
AOSSIE-Org:mainfrom
AnupamKumar-1:fix/project-card-links
Closed

fix: link project cards on homepage to their project pages (#726)#727
AnupamKumar-1 wants to merge 1 commit into
AOSSIE-Org:mainfrom
AnupamKumar-1:fix/project-card-links

Conversation

@AnupamKumar-1
Copy link
Copy Markdown

@AnupamKumar-1 AnupamKumar-1 commented May 26, 2026

Addressed Issues:

Fixes #726

Additional Notes:

Added href prop to CardEffect.jsx and passed project.link.href from src/app/page.jsx

Files Changed:

  • src/app/page.jsx
  • src/components/home/CardEffect.jsx

AI Usage Disclosure:

  • This PR does not contain AI-generated code at all.

I have used the following AI models and tools: None

Checklist

  • My PR addresses a single issue, fixes a single bug or makes a single improvement.
  • My code follows the project's code style and conventions
  • If applicable, I have made corresponding changes or additions to the documentation
  • If applicable, I have made corresponding changes or additions to tests
  • My changes generate no new warnings or errors
  • I have joined the Discord server and I will share a link to this PR with the project maintainers there
  • I have read the Contribution Guidelines
  • Once I submit my PR, CodeRabbit AI will automatically review it and I will address CodeRabbit's comments.
  • I have filled this PR template completely and carefully, and I understand that my PR may be closed without review otherwise.

Summary by CodeRabbit

  • New Features
    • Project cards in the grid now include functional navigation links to project details.

Review Change Stack

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 26, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 54bb097a-893d-4c5f-ae22-99d3b996e757

📥 Commits

Reviewing files that changed from the base of the PR and between 895772c and dcc46f0.

📒 Files selected for processing (2)
  • src/app/page.jsx
  • src/components/home/CardEffect.jsx

📝 Walkthrough

Walkthrough

The PR enables navigation on project cards by updating the CardEffect component to accept and use an href prop, then passing project.link.href from the home page grid to each rendered card instance.

Changes

Project Card Link Navigation

Layer / File(s) Summary
Card link routing
src/components/home/CardEffect.jsx, src/app/page.jsx
CardEffect now accepts an href prop in its component signature and applies it to the motion.a element. The home page projects grid passes project.link.href to each CardEffect instance, enabling navigation to project pages when cards are clicked.

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly Related PRs

  • AOSSIE-Org/Website#696: Updates project link values in projects.js, which directly determines the destination URLs passed through the new href prop.

Suggested Reviewers

  • Zahnentferner

Poem

A rabbit hops through links so bright,
Each card now leads to projects right,
No more static, frozen still—
Navigation's climbing up the hill! 🐰✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and specifically describes the main change: linking project cards on the homepage to their project pages, which matches the primary objective.
Linked Issues check ✅ Passed The code changes successfully implement the required fix by adding an href prop to CardEffect and passing project.link.href from page.jsx, enabling navigation from project cards to project pages.
Out of Scope Changes check ✅ Passed All changes are directly related to fixing issue #726; no unrelated or out-of-scope modifications are present in the pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@AnupamKumar-1 AnupamKumar-1 deleted the fix/project-card-links branch May 28, 2026 14:45
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.

[BUG]: Project should be linked to it's project page

1 participant