-
-
Notifications
You must be signed in to change notification settings - Fork 3.9k
Conversation
Addition of new profile mohit1607.
Merged the Branch created for adding a profile
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's great having you contribute to this project
Welcome to the community 🤓If you would like to continue contributing to open source and would like to do it with an awesome inclusive community, you should join our Discord chat and our GitHub Organisation - we help and encourage each other to contribute to open source little and often 🤓 . Any questions let us know.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great.
data/mohit1607.json
Outdated
"tags": ["javascript", "typescript", "react.js", "node.js", "mongodb", "fullstack"], | ||
"socials": [ | ||
{ "icon": "twitter", "url": "https://twitter.com/ashliya_mohit" }, | ||
{ "icon": "github", "url": "https://github.com/mohit1607" } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These are not icons, please use icons available, for example FaTwitter
https://linkfree.eddiehub.io/icons
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Icons need updating, inline comment added
Resolved the issue using the appropriate icons in socials array.
Hi Eddie thanks for reviewing my profile and Now I look forward to solve more issues. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to merge. Thank you for your contribution! 🎉
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good 👍
Addition of new profile mohit1607.
Fixes Issue
Changes proposed
Check List (Check all the applicable boxes)
Screenshots
Note to reviewers