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
fix: pass context for isDefined and custom validators #296
Conversation
Since we need typestack#296 to be able to use context with isDefined, ran a `npm run-script build` and then moved all the files from build/package into the root, which mimics an npm published build. Also, we cannot simply add the branch to package.json because yarnpkg/yarn#1671
5e622ea
to
78e1b30
Compare
Since we need typestack#296 to be able to use context with isDefined, ran a `npm run-script build` and then moved all the files from build/package into the root, which mimics an npm published build. Also, we cannot simply add the branch to package.json because yarnpkg/yarn#1671
@amirsaber Would you be able to help review this? Thanks! |
just discovered this problem today, thanks for the PR ! |
When this will be merged and available on NPM :) ? |
Any update on this? |
Any update on this? Just bumped into this issue myself, would be handy if context on custom validators would be passed. |
I'm happy to resolve the conflicts, if someone is willing to review? |
Sure. Please resolve conflicts, cover changes with tests and I am happy to review this. |
Context wasn't being passed in for isDefined and custom validators. typestack#292
73512b9
to
8d5c466
Compare
@vlapo should be ready for review, thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good for me. Just small request about tests.
Thank you for your patient and perseverance. |
This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Context wasn't being passed in for isDefined and custom validators.
#292