Skip to content

chore: remove resource announcement banner - #75

Merged
creatorcluster merged 1 commit into
creatorcluster:mainfrom
Coder-soft:main
Aug 8, 2026
Merged

chore: remove resource announcement banner#75
creatorcluster merged 1 commit into
creatorcluster:mainfrom
Coder-soft:main

Conversation

@Coder-soft

@Coder-soft Coder-soft commented Aug 8, 2026

Copy link
Copy Markdown

Summary by CodeRabbit

  • Bug Fixes
    • Removed the unused announcement banner from the Resources Hub page for a cleaner, more focused layout.

@vercel

vercel Bot commented Aug 8, 2026

Copy link
Copy Markdown

@Coder-soft is attempting to deploy a commit to the yamura3's projects Team on Vercel.

A member of the Team first needs to authorize it.

@coderabbitai

coderabbitai Bot commented Aug 8, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 1271a609-4076-4228-9a4c-bccafbaf97cc

📥 Commits

Reviewing files that changed from the base of the PR and between 2173628 and ccd9109.

📒 Files selected for processing (1)
  • src/pages/ResourcesHub.tsx
💤 Files with no reviewable changes (1)
  • src/pages/ResourcesHub.tsx

📝 Walkthrough

Walkthrough

The Resources Hub no longer imports or renders ResourceAnnouncementBanner. The banner’s category navigation and smooth-scrolling entry point are removed.

Changes

Resources Hub banner removal

Layer / File(s) Summary
Remove announcement banner
src/pages/ResourcesHub.tsx
Removes the unused ResourceAnnouncementBanner import and its rendered announcement banner.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Poem

A rabbit hops past the banner today,
The resource paths now lead the way.
No unused import hides in sight,
The page is tidy, clear, and light.
Hop, hop—clean code takes flight!

🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the removal of the unused resource announcement banner from the Resources Hub page.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Create stacked PR
  • Commit on current branch
🧪 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.

@creatorcluster
creatorcluster merged commit 623ed13 into creatorcluster:main Aug 8, 2026
2 of 4 checks passed
@greptile-apps

greptile-apps Bot commented Aug 8, 2026

Copy link
Copy Markdown

Greptile Summary

Removes the retired resource announcement from the Resources Hub, including its music-tab navigation callback. Browser validation compared the page before and after the change: /resources returned successfully, rendered normally without uncaught errors, and no longer displayed the announcement. A possible page-rendering regression was disproved by this live comparison.

Confidence Score: 5/5

Safe to merge: the Resources Hub remains functional after the announcement is removed.

There are no final findings. Live Chromium validation confirmed the intended visual removal while preserving successful page rendering, and linting of the changed file passed.

Files Needing Attention: None.

Reviews (1): Last reviewed commit: "chore: remove resource announcement bann..." | Re-trigger Greptile

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.

2 participants