fix(docs): wire correct icons to docs landing cards (Closes #5699) - #5702
fix(docs): wire correct icons to docs landing cards (Closes #5699)#5702waterWang wants to merge 1 commit into
Conversation
✅ Deploy Preview for asyncapi-website ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify project configuration. |
There was a problem hiding this comment.
Welcome to AsyncAPI. Thanks a lot for creating your first pull request. Please check out our contributors guide useful for opening a pull request.
Keep in mind there are also other channels you can use to interact with AsyncAPI community. For more details check out this issue.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughThe change renames the migration icon component and updates the ChangesIcon mapping updates
Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: ⚪ Minimal · up to This PR makes localized documentation icon and export-name corrections without introducing an actionable merge-blocking risk; it is merge-ready after normal checks and review. Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 ESLint
ESLint install failed: dependency version conflict. Check your lock file or package.json. 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. Comment |
|
|
⚡️ Lighthouse report for the changes in this PR:
Lighthouse ran on https://deploy-preview-5702--asyncapi-website.netlify.app/ |



Summary
Fixes #5699 — Docs / Community card icons in the landing page Header section use icons that don't match their sections.
Changes
components/data/buckets.tsIconUseCases(pen/quill — reads as "writing/authoring") → now usesIconTools(wrench/tools metaphor matching the AsyncAPI tools ecosystem docs).IconExplorer(sitemap node with plus sign — reads as "add a node") → now usesIconLoupe(magnifier, matching the "explore the spec" intent).IconExplorerimport.components/icons/Migration.tsxIconUserswhile living inMigration.tsx, which misleads any consumer importing it asIconMigrationand shadows the realUsersicon's role. Renamed the export toIconMigrationto match the file name and its actual migration icon content.Verification
components/icons/Tools.tsxandcomponents/icons/Loupe.tsxalready existed (verified via repo tree) — no new icons needed to be authored.buckets.tsimport list now includesIconToolsandIconLoupe, and no longer referencesIconExplorer/IconUseCasesin the card mappings.Usersicon (components/icons/Users.tsx) still exportsIconUsersand is untouched — Community card remains correct.Summary by CodeRabbit