Skip to content
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
<!-- Organization Logo -->
<div align="center" style="display: flex; align-items: center; justify-content: center; gap: 16px;">
<img alt="AOSSIE" src="public/aossie-logo.svg" width="175">
<img src="public/todo-project-logo.svg" width="175" />
<!-- <img src="public/todo-project-logo.svg" width="175" /> -->
</div>

&nbsp;
Expand Down Expand Up @@ -42,8 +42,8 @@
&nbsp;&nbsp;
<!-- Youtube -->
<a href="https://www.youtube.com/@AOSSIE-Org">
<img src="https://img.shields.io/youtube/channel/subscribers/UCKVVLbawY7Gej_3o2WKsoiA?style=flat&logo=youtube&logoColor=white%20&logoSize=auto&labelColor=FF0000&color=FF0000
" alt="Youtube Badge"></a>
<img src="https://img.shields.io/badge/subscribers-73-FF0000?style=flat&logo=youtube&logoColor=white&labelColor=FF0000" alt="YouTube Badge">
</a>
Comment on lines +45 to +46
Copy link
Contributor

@coderabbitai coderabbitai bot Feb 22, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟡 Minor

Two issues with the new YouTube badge: hardcoded stale count and invisible label contrast.

  1. Hardcoded subscriber count will go stale. The static value 73 in subscribers-73-FF0000 is frozen at the time of commit. Shields.io supports a native YouTube Channel Subscribers badge that pulls live data — consider switching to that instead.

  2. labelColor=FF0000 matches the badge color, eliminating contrast. Both the label section (left, labelColor) and the message section (right, badge color) are FF0000, so the whole badge renders as a uniform red block with no visual separator between "subscribers" and "73".

If a live subscriber count is desired, the shields.io YouTube Channel Subscribers badge (/youtube/channel/subscribers/<CHANNEL_ID>) provides a dynamic count without requiring a hardcoded value.

If a static CTA is preferred (simpler and never goes stale), reverting to the previous "Subscribe" badge is a better choice:

🛠️ Option A — Dynamic subscriber badge (preferred)
-  <img src="https://img.shields.io/badge/subscribers-73-FF0000?style=flat&logo=youtube&logoColor=white&labelColor=FF0000" alt="YouTube Badge">
+  <img src="https://img.shields.io/youtube/channel/subscribers/<CHANNEL_ID>?style=flat&logo=youtube&logoColor=white&label=Subscribe&color=FF0000" alt="YouTube Badge">

Replace <CHANNEL_ID> with the AOSSIE-Org YouTube channel ID.

🛠️ Option B — Static Subscribe CTA (no stale data)
-  <img src="https://img.shields.io/badge/subscribers-73-FF0000?style=flat&logo=youtube&logoColor=white&labelColor=FF0000" alt="YouTube Badge">
+  <img src="https://img.shields.io/badge/YouTube-Subscribe-FF0000?style=flat&logo=youtube&logoColor=white" alt="YouTube Badge">
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
<img src="https://img.shields.io/badge/subscribers-73-FF0000?style=flat&logo=youtube&logoColor=white&labelColor=FF0000" alt="YouTube Badge">
</a>
<img src="https://img.shields.io/badge/YouTube-Subscribe-FF0000?style=flat&logo=youtube&logoColor=white" alt="YouTube Badge">
</a>
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@README.md` around lines 45 - 46, The YouTube badge currently uses a hardcoded
count and identical label/message color; update the img src in the <img> tag to
use shields.io's dynamic YouTube subscribers endpoint
(/youtube/channel/subscribers/<CHANNEL_ID>) replacing the static
"subscribers-73-FF0000" segment with "youtube/channel/subscribers/<CHANNEL_ID>"
and set a contrasting labelColor (e.g., labelColor=ffffff or label="Subscribe")
so the left label is readable, or alternatively revert the <img> to the previous
static "Subscribe" CTA badge if you prefer a non-dynamic, never-stale badge.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@hamzanazir27 , could you fix this?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems like the humans are having a chat. I'll hop back into my burrow for now. If you need me again, just tag @coderabbitai in a new comment, and I'll come hopping out!

</p>

---
Expand Down