refactor: navbar's social icons items#1454
Conversation
✅ Deploy Preview for reactplayio ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
Hey! contributor, thank you for opening a Pull Request 🎉.
@reactplay/maintainers will review your submission soon and give you helpful feedback.
If you're interested in continuing your contributions to open source and want to be a part of a welcoming and fantastic community, we invite you to join our ReactPlay Discord Community.
Show your support by starring ⭐ this repository. Thank you and we appreciate your contribution to open source!
Stale Marking : After 30 days of inactivity this issue/PR will be marked as stale issue/PR and it will be closed and locked in 7 days if no further activity occurs.
|
@priyankarpal, let me know if any changes are required in it. |
priyankarpal
left a comment
priyankarpal
left a comment
There was a problem hiding this comment.
rename all link to NavLink @Shivam-Katare
priyankarpal
added
waiting for reviewers
changes required
priyankarpal
changed the title
|
@priyankarpal, I renamed all |
thank you |
2 participants
Description
Refactor the navbar's social icons code. Removed the repetition part, and instead of that used the mapping to display all icons.
Fixes #1448
Type of change
How Has This Been Tested?
Checklist:
Screenshots or example output