-
-
Notifications
You must be signed in to change notification settings - Fork 3.9k
Conversation
Co-authored-by: sital002 <[email protected]>
Co-authored-by: sital002 <[email protected]>
Co-authored-by: sital002 <[email protected]>
Corrected JSON: Fixed "Follow me on LinkedI" to "Follow me on LinkedIn" and quoted "FaLink" and "FaGithub".
Update Ousman Ethmane's Profile Information
Important Auto Review SkippedReview was skipped due to path filters Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit's AI:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
data/Ousman37.json
Outdated
{ | ||
"name": "LinkedIn Follow me", | ||
"url": "https://www.linkedin.com/in/ethmane-ousman-34885844/", | ||
"icon": "fab fa-linkedin", |
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.
This is not a valid icon and I have made a suggestion as to what I believe you are looking to achieve:
"icon": "fab fa-linkedin", | |
"icon": "FaLinkedin", |
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.
Thank you, @SaraJaoude for your feedback and suggestion. I appreciate your input. I've made the necessary changes to use the custom icon format 'FaLinkedin' as you suggested. Please let me know if there are any further improvements or adjustments needed.
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.
You do not appear to have used valid icons - please see inline comment and update throughout.
You may wish to take a look at the Available Icons page in our Documentation.
Updated JSON data with correct icons.
data/Ousman37.json
Outdated
"evenTags": [ | ||
"Front-End Development", | ||
"UI/UX Design", | ||
"Responsive Web Design", | ||
"Version Control", | ||
"Web Development Tools", | ||
"Problem Solving", | ||
"Debugging", | ||
"Team Collaboration" | ||
], |
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.
I don't think if evenTags
is a valid attribute, Please remove this from your json
. You can add these tags to tags
attribute if you want.
"evenTags": [ | |
"Front-End Development", | |
"UI/UX Design", | |
"Responsive Web Design", | |
"Version Control", | |
"Web Development Tools", | |
"Problem Solving", | |
"Debugging", | |
"Team Collaboration" | |
], |
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.
Updated JSON structure: Removed 'evenTags' attribute and added tags to 'tags' attribute as suggested.
"testimonials": [ | ||
{ | ||
"author": "John Doe", | ||
"position": "CEO, XYZ Company", | ||
"quote": "Ousman is an incredibly talented front-end developer. His work has greatly improved our website's user experience." | ||
}, | ||
{ | ||
"author": "Jane Smith", | ||
"position": "UI/UX Designer, ABC Agency", | ||
"quote": "Working with Ousman has been a pleasure. He has a keen eye for design and pays great attention to detail." | ||
} | ||
], |
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 doesn't seem to be the right way to add testimonials & may not be displayed on your profile. Please check out these docs to know more about testimonials, and remove this from your json
.
"testimonials": [ | |
{ | |
"author": "John Doe", | |
"position": "CEO, XYZ Company", | |
"quote": "Ousman is an incredibly talented front-end developer. His work has greatly improved our website's user experience." | |
}, | |
{ | |
"author": "Jane Smith", | |
"position": "UI/UX Designer, ABC Agency", | |
"quote": "Working with Ousman has been a pleasure. He has a keen eye for design and pays great attention to detail." | |
} | |
], |
], | ||
"callToAction": { | ||
"text": "Contact Me", | ||
"url": "mailto:[email protected]" | ||
} |
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.
callToAction
doesn't seem to be a valid attribute. You can add mail url in the links
array as a link.
], | |
"callToAction": { | |
"text": "Contact Me", | |
"url": "mailto:[email protected]" | |
} | |
] |
chore: Remove 'evenTags' attribute and add tags to 'tags' attribute
Remove 'evenTags' and Update 'tags' in JSON - Eliminated invalid 'evenTags' attribute. - Merged its contents into the 'tags' attribute.
We look forward to seeing your Profile on BioDrop, but as there is no recent activity from you to make the changes suggested I am going to close this pull request. We hope you will raise a new pull request to create your Profile and when you do, you might want to take a look at these Docs which will help with the suggested changes the maintainers have proposed. Please remember you can also create your Profile using Forms. |
Fixes Issue
Changes proposed
Check List (Check all the applicable boxes)
Screenshots
Note to reviewers