Fix project links not clickable on /projects page

[tubajavedd]

This PR fixes the issue where project links on the /projects page were not clickable.

Changes:

• Replaced the

  wrapper with an tag for project links

• Used project.link.href properly
• Added target="_blank" and rel="noopener noreferrer"

Note:
The project "OpenChat" still has href "#" in data, so it does not navigate.

Fixes: #481

Summary by CodeRabbit

• Refactor

  • Updated link components for improved accessibility and semantic structure across the application (project links now use proper anchor behavior).

• Style

  • Minor formatting and whitespace adjustments in the banner component with no visible or functional changes.

[vercel]

Someone is attempting to deploy a commit to the AOSSIE Team on Vercel.

A member of the Team first needs to authorize it.

[coderabbitai]

📝 Walkthrough

Walkthrough

Banner file has only formatting/whitespace adjustments with no functional or DOM changes; projects page replaces a non-interactive paragraph with a semantic <a> element using href, target="_blank", and rel="noopener noreferrer" to restore clickable project links.

Changes

Cohort / File(s)	Summary
Banner formatting only src/components/Banner.jsx	Whitespace/formatting changes around the button/Link block; no DOM, prop, or behavioral changes.
Projects page link fix src/pages/projects.jsx	Replaces a non-interactive <p> (link label + icon) with a semantic <a> that uses project.link.href, target="_blank", and rel="noopener noreferrer", restoring clickable external links and hover styles.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰
I hopped through markup, nibbled a bug,
Turned text to a link with a cheerful shrug.
Now clicks bloom bright where labels once sat,
A tiny fix — imagine that! ✨

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

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.
Out of Scope Changes check	❓ Inconclusive	The changes in Banner.jsx appear to be formatting/whitespace-only with no functional changes, but are unrelated to the /projects page link issue. This represents a minor scope creep that should be clarified.	Clarify whether Banner.jsx whitespace changes are intentional and necessary, or should be removed to keep this PR focused solely on fixing project links.

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and accurately summarizes the main change: fixing non-clickable project links on the /projects page by converting them to proper anchor elements.
Linked Issues check	✅ Passed	The pull request implements all required changes from issue #481: replacing wrapper with tag, using project.link.href, adding target="_blank" and rel="noopener noreferrer" attributes, and maintaining styling.

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

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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