Skip to content
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

Skills API #9

Merged
merged 26 commits into from
Feb 2, 2023
Merged

Skills API #9

merged 26 commits into from
Feb 2, 2023

Conversation

philip-peiffer
Copy link
Collaborator

Background:

Implemented the following:

Backend Work:

  • /skills - shared skills amongst users, in file routes/skills.js
    • POST, GET, method not allowed on PUT, PATCH, DELETE
    • method not allowed on /skills/:skill_id routes
  • /users/:user_id/skills - user specific skills, in file routes/userSkills
    • GET - commented out until user verification implemented
    • method not allowed on POST, PUT, PATCH, DELETE
  • /users/:user_id/skills/:skill_id - user specific skill, in file routes/userSkills
    • GET, DELETE, PUT - commented out until user verification implemented
    • method not allowed on POST, PATCH

Frontend Work:

Made it so that skills page displays the shared skills table (not user specific), just to show that it is fetching correctly and items are being created.

@philip-peiffer philip-peiffer merged commit b6cc1ff into main Feb 2, 2023
@philip-peiffer philip-peiffer deleted the peiffer-skills-api branch February 2, 2023 19:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant