Skip to content

Conversation

@KatalKavya96
Copy link

Description

  • Refactored the recognition.js file (≈457 lines) into modular, reusable components.

  • This update introduces two new files — badge-item.js and badges-data.js — to make the badges section data-driven and easier to maintain.

  • Created a reusable component for each badge entry.

  • Moved all badge information into a central badges-data.js array.

  • Replaced the manually written list of

  • tags with a clean .map() render loop.

  • Functionality and styling remain unchanged; structure and readability improved.

This PR fixes #7027

Notes for Reviewers

  • Confirm that the badges render exactly as before under /community/handbook/recognition.

  • Verify that BadgeItem handles both internal (Link) and external (a) links correctly.

  • No breaking UI or navigation changes.

@Namanv0509
Copy link
Member

@KatalKavya96 please look over the merge conflicts else it looks good to go for now

@Namanv0509
Copy link
Member

@Rajesh-Nagarajan-11 ^^ give a look if you will

@KatalKavya96
Copy link
Author

@Namanv0509 Ok i will resolve merge conflicts and recommit.Thanks for the review

@KatalKavya96
Copy link
Author

@Namanv0509 I have resolved the merge conflicts.Please see it now.Thanks

@l5io
Copy link
Contributor

l5io commented Oct 14, 2025

🚀 Preview for commit 063d683 at: https://68ee534aacea5205d8c384b5--layer5.netlify.app

@l5io
Copy link
Contributor

l5io commented Oct 16, 2025

🚀 Preview for commit 57e5526 at: https://68f0677dc733ada47fcf37c0--layer5.netlify.app

Copy link
Member

@Rajesh-Nagarajan-11 Rajesh-Nagarajan-11 left a comment

Choose a reason for hiding this comment

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

@KatalKavya96 any screenshot or video will be helpfull

@Rajesh-Nagarajan-11
Copy link
Member

  • resolve Merge Conflicts

@KatalKavya96
Copy link
Author

@Rajesh-Nagarajan-11 The preview of last commit contains all the badges : https://68f0677dc733ada47fcf37c0--layer5.netlify.app/ . Please check it once , if required i will send you screenshots from my local as well. Thanks

@l5io
Copy link
Contributor

l5io commented Oct 23, 2025

🚀 Preview for commit 3c327f1 at: https://68fa7509560fc33ae7b04f2b--layer5.netlify.app

@l5io
Copy link
Contributor

l5io commented Oct 24, 2025

🚀 Preview for commit 08c73fc at: https://68fb896555b16f6a418da302--layer5.netlify.app

@Rajesh-Nagarajan-11
Copy link
Member

@KatalKavya96 please resolve Merge Conflicts

@KatalKavya96
Copy link
Author

KatalKavya96 commented Oct 25, 2025

@Rajesh-Nagarajan-11 Resolving

Signed-off-by: Kavya Katal <kavyakatal09@gmail.com>
@KatalKavya96 KatalKavya96 force-pushed the feat/refactor-recognition-badges branch from 8a3fc9c to a0d0f70 Compare October 25, 2025 12:51
Signed-off-by: Kavya Katal <KAVYAKATAL09@GMAIL.COM>
Signed-off-by: Kavya Katal <KAVYAKATAL09@GMAIL.COM>
@l5io
Copy link
Contributor

l5io commented Oct 25, 2025

🚀 Preview for commit 71e67ac at: https://68fccecc91458179e12c8540--layer5.netlify.app

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

File Refactor into Re-usable Components

4 participants