Update Twitter logo to 'X' in the website #519

[PAGARE-MANGESH]

Summary by CodeRabbit

• Chores

  • Updated FontAwesome icon library dependencies to version 7.1.0 and React FontAwesome to version 3.1.1 for improved compatibility and features.

• Style

  • Updated social media icons throughout the application to reflect current branding standards.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[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

FontAwesome dependencies upgraded from version 6 to 7, and Twitter icon references replaced with X icon across frontend components. Package.json updated alongside component files to reflect dependency version changes and icon symbol updates.

Changes

Cohort / File(s)	Summary
Dependency Updates package.json	Updated @fortawesome/free-brands-svg-icons from ^6.2.1 to ^7.1.0 and @fortawesome/react-fontawesome from ^0.2.0 to ^3.1.1
Icon Replacements src/components/Footer.jsx, src/pages/index.jsx	Replaced faTwitter icon import and usage with faXTwitter across both files to reflect brand icon change

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 A hop and skip through FontAwesome's delight,
From Twitter to X, our icons take flight!
Version seven brings us closer to the stars,
Components dance with their new avatar—
The web looks fresh, the icons burn bright! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 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 (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately summarizes the main change: updating Twitter logo references to 'X' across the codebase by replacing faTwitter with faXTwitter icons and updating FontAwesome dependencies.

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

✨ Finishing touches

• 📝 Generate docstrings

---

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.}

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (4)

src/components/Footer.jsx (2)

6-6: Fix formatting: remove extra space in import statement.

There's a double space after faGitlab, in the import statement.

✨ Proposed fix

-import { faDiscord, faGithub, faGitlab,  faXTwitter } from '@fortawesome/free-brands-svg-icons'
+import { faDiscord, faGithub, faGitlab, faXTwitter } from '@fortawesome/free-brands-svg-icons'

---

48-49: Update aria-label and fix icon spacing.

The aria-label still references "Twitter" and should be updated to reflect the X rebrand. Additionally, there's inconsistent spacing in the icon prop (icon={ faXTwitter} has a space after the opening brace).

♻️ Proposed fix

-                <Link aria-label="Follow on Twitter" className=' text-zinc-400 hover:text-[#00843D] dark:text-zinc-400 dark:hover:text-yellow-400 transition' href='https://twitter.com/aossie_org'>
-                  <FontAwesomeIcon icon={ faXTwitter} size='xl' />
+                <Link aria-label="Follow on X" className=' text-zinc-400 hover:text-[#00843D] dark:text-zinc-400 dark:hover:text-yellow-400 transition' href='https://twitter.com/aossie_org'>
+                  <FontAwesomeIcon icon={faXTwitter} size='xl' />
                </Link>

src/pages/index.jsx (2)

20-20: Remove extraneous blank line.

An extra blank line was introduced at line 20.

✨ Proposed fix

-
 export default function Home() {

---

92-96: Update aria-label to reflect X rebrand.

The aria-label still references "Twitter" and should be updated to "X" for consistency with the icon change and to accurately describe the link for screen reader users.

♻️ Proposed fix

                 <Link
-                  aria-label="Follow on Twitter"
+                  aria-label="Follow on X"
                   className="text-zinc-500 transition hover:text-[#00843D] dark:text-zinc-400 dark:hover:text-yellow-400"
                   href="https://twitter.com/aossie_org"
                 >

📜 Review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between dfb8668 and 6ec02ab.

⛔ Files ignored due to path filters (1)

• package-lock.json is excluded by !**/package-lock.json

📒 Files selected for processing (3)

• package.json
• src/components/Footer.jsx
• src/pages/index.jsx

🔇 Additional comments (1)

package.json (1)

17-21: No major compatibility issue exists with the current FontAwesome package versions. @fortawesome/react-fontawesome@3.1.1 explicitly supports mixing v6 and v7 packages via its peer dependency constraint ~6 || ~7 for fontawesome-svg-core. Font Awesome documentation confirms that v7 packages maintain backward compatibility with v6 icon packages, and incremental package updates are supported. The version configuration is valid.

Likely an incorrect or invalid review comment.