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

Add validators to user controller #48

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Add validators to user controller #48

wants to merge 4 commits into from

Conversation

denkomanceski
Copy link
Collaborator

@denkomanceski denkomanceski commented Nov 12, 2018 •
edited
Loading

Do not merge..

@denkomanceski denkomanceski force-pushed the dm/validators-user- branch 3 times, most recently from 9baee36 to 1fe0349 Compare November 13, 2018 14:58
@denkomanceski denkomanceski force-pushed the dm/validators-user- branch 7 times, most recently from 18f03e2 to ecaee7c Compare December 16, 2018 14:10
@denkomanceski
Copy link
Collaborator Author

denkomanceski commented Dec 16, 2018 •
edited
Loading

@turshija needs to be tested locally, because I validate exact fields

I tested: login, updating profile and creating playlists (including adding videos)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@turshija turshija Awaiting requested review from turshija

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@denkomanceski