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

Feature/add express validator #17

Open
wants to merge 2 commits into
base: development
Choose a base branch
from

Conversation

Bigghead
Copy link
Collaborator

Tested portfolio routes. The reason it wasn't working as expected previously was because I had written it as a middleware function (req, res, next) => ......next(), but it expects that middleware to be in an array format for some reason:

router.post('/portfolios', [ sanitize('name') ], (req, res) => {.....}

Rewrote the reusable sanitization/validation( for later ) middleware to return an array

@Bigghead Bigghead closed this Feb 23, 2018
@Bigghead Bigghead deleted the feature/add-express-validator branch February 23, 2018 23:53
@Bigghead Bigghead restored the feature/add-express-validator branch February 24, 2018 00:33
@Bigghead Bigghead reopened this Feb 24, 2018
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