-
Notifications
You must be signed in to change notification settings - Fork 256
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adds routes for Skills API #650
Conversation
@ankushdharkar can you review it? |
f8a3c27
to
77eceb4
Compare
123b24d
to
14979c5
Compare
const { skillName } = req.params; | ||
const skillWithoutSpecialChar = checkDashUnderscore.test(skillName) | ||
? skillName.replace(checkDashUnderscore, " ") | ||
: skillName; |
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.
If there is some error here, how is that handled?
*/ | ||
async function awardSkill(skillData, userName) { | ||
try { | ||
const skillSnapshot = await skillsCollection.add(skillData); |
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.
Won't this create multiple records for the same skills?
Closing this as this feature is being built as a stand-alone product, in Skill-tree https://github.com/Real-Dev-Squad/skill-tree-frontend/ |
The API Contract and Data Model should be finalised before moving ahead with this feature
Description
This feature adds routes, controllers, and models for these endpoints
/skills/:username
/skills
/skills/:username
/users/skill/:skillName
This feature will help fetch the skills of users dynamically from the backend instead of current mock data on members site. These skill/s will be given to the user from the dashboard site
Closes #455