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.

Sign up for GitHub

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

Jump to bottom

Lock all endpoints with verifyToken #81

Closed
1 task done
BreanneHuang opened this issue Jul 11, 2020 · 2 comments · Fixed by #86
Closed
1 task done

Lock all endpoints with verifyToken #81

BreanneHuang opened this issue Jul 11, 2020 · 2 comments · Fixed by #86
Assignees
@BreanneHuang
Labels
implementation new feature
Milestone
Progress 5

Comments

@BreanneHuang
Copy link
Contributor

BreanneHuang commented Jul 11, 2020 •
edited
Loading

To-do:

  • Lock all endpoints with verifyToken except login, register, getCurrentUser
@BreanneHuang BreanneHuang added the implementation new feature label Jul 11, 2020
@BreanneHuang BreanneHuang added this to the Progress 5 milestone Jul 11, 2020
@BreanneHuang BreanneHuang self-assigned this Jul 11, 2020
@BreanneHuang BreanneHuang mentioned this issue Jul 11, 2020
Open
@BreanneHuang
Copy link
Contributor Author

Some additional things that were changed/completed:

  • Updated add song functionality so that adding a song to the universal collection, to the queue, and to the playlist is done in a single place instead of across three different files

  • Added unique constraint to prevent adding two songs with the same YouTube url

@BreanneHuang BreanneHuang linked a pull request Jul 12, 2020 that will close this issue
Locked endpoints with verifyToken #86
Merged
3 tasks
@BreanneHuang
Copy link
Contributor Author

Endpoints have been locked with verifyToken. Closing this issue.

@BreanneHuang BreanneHuang mentioned this issue Jul 25, 2020
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@BreanneHuang BreanneHuang

Labels
implementation new feature
Projects
None yet
Milestone
Progress 5
Development

Successfully merging a pull request may close this issue.

Locked endpoints with verifyToken CPSC-436I-Project/Musico
1 participant
@BreanneHuang