validateProvidedData with mongoose format

[soullivaneuh]

As specified on the doc, the validate.validateProvidedData method provides the same result as validate.form.

However, I am using validate.mongoose because I prefer this format.

Because of this, I can not use the partial validation feature you propose.

Is that a reason to not support mongoose format for validateProvidedData?

Thanks

[soullivaneuh]

Just to try, I replaced form by mongoose on this line:

validate-joi/index.js

Line 107 in e914d74

const validateHook = validators.form(patchSchema, joiOptions);

Works like a charm.

A refactor may be done with ease to provide methods like this:

• validate.form
• validate.formWithProvidedData (currently validateProvidedData)
• validate.mongoose
• validate.mongooseWithProvidedData

What do you think?

I may work on a PR for this.

[soullivaneuh]

I just made a proposition in #45

[soullivaneuh]

If anyone is also stuck by this limitation, here is an hacky workaround:

# To be run by postinstall
#!/usr/bin/env sh
set -e

# Hack to be remove when both format will be supported.
# @see https://github.com/feathersjs-ecosystem/validate-joi/pull/45
sed -i 's/validators\.form/validators.mongoose/' node_modules/feathers-validate-joi/index.js

Then validate.validateProvidedData will use the mongoose format.

However, this solution does not work if you need both formats.

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Apologies if the issue could not be resolved. FeathersJS ecosystem modules are community maintained so there may be a chance that there isn't anybody available to address the issue at the moment. For other ways to get help see here.