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

Allow adding custom transform filters #99

Open
JaneJeon opened this issue Jun 4, 2019 · 0 comments · May be fixed by #262
Open

Allow adding custom transform filters #99

JaneJeon opened this issue Jun 4, 2019 · 0 comments · May be fixed by #262

Comments

@JaneJeon
Copy link

JaneJeon commented Jun 4, 2019

dynamicDefaults allow users to add their own defaults functions by exporting DEFAULTS in the keyword, as so: https://github.com/epoberezkin/ajv-keywords/blob/master/keywords/dynamicDefaults.js#L49

However, transform does not export the transform variable - https://github.com/epoberezkin/ajv-keywords/blob/master/keywords/transform.js#L4 - and so there's literally no way for users to add custom transformations.

I think you should export the transform variable like you do in dynamicDefaults and allow people to add their own transformations, rather than bolting in other transformations like in #95 or #66.

Thanks!

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

Successfully merging a pull request may close this issue.

2 participants